aboutsummaryrefslogtreecommitdiffstats
path: root/src/lib/ecore_con (follow)
Commit message (Collapse)AuthorAgeFilesLines
* ecore_con: fix error handling case.Cedric BAIL2015-06-031-2/+0
| | | | | | | | | | | Since the move to Eo, in case of error we do not need to cleanup anything when finalize finish. The kill function was arranged to do so, but it was still emiting an event exposing the invalid Eo object. Looking closely at the code, that should have not happened, as there was no symetrical ADD event and the function would do nothing else. So removing it. This prevent any potential SEGV from a function that would catch the DEL event. @fix
* ecore_con: add eo_event handler to efl_networ_url.Srivardhan Hebbar2015-05-293-38/+141
| | | | | | | | | | | | Summary: Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com> Reviewers: cedric Subscribers: cedric Differential Revision: https://phab.enlightenment.org/D2556 Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>
* eolian: disallow cyclic dependencies between .eo filesDaniel Kolesa2015-05-221-1/+2
| | | | | | | | | | Eo files will now fail to compile if a cycle is detected. This required some temporary changes in existing eo files (we had 2 cycles) for which I added a FIXME (they do not affect C generation). @feature
* Eo: Remove eo_error_set() and clean up finalizer()Tom Hacohen2015-05-201-4/+3
| | | | | | | | | | | | This is another cleanup in perparation for the Eo stable release. This is no longer needed thanks to the proper error reporting with eo_constructor()'s new return value. The finalizer change cleans it up a bit so it catches more cases/issues. This also means that the finalizer cleans up the object in all cases, and not only some. @feature.
* Ecore con url: Fix wrong object destruction.Tom Hacohen2015-05-201-11/+25
| | | | | | | | | Because we defer the deletion of this object until we get all the responses from the network, we must manually free it. A better solution would possibly be to just free the requests upon object deletion instead of deferring the deletion. @fix
* Ecore con url: correctly destruct the object.Tom Hacohen2015-05-201-0/+1
|
* Eo: Add a return value to eo_constructor().Tom Hacohen2015-05-202-6/+9
| | | | | | | | | | | From now on, constructors should return a value, usually the object being worked on, or NULL (if the constructor failed). This can also be used for implementing singletons, by just always returning the same object from the constructor. This is one of the final steps towards stabilizing Eo. @feature
* eolian: new syntax for params/values/keysDaniel Kolesa2015-05-184-18/+18
| | | | | | | | | | | Instead of "@in type name;" we now use "@in name: type;". This change is done because of consistency with the rest of Eolian; pretty much every other part of Eolian syntax uses the latter form. This is a big breaking change in the .eo format, so please update your .eo files accordingly and compile Elementary together with the EFL. @feature
* ecore_con: changed Ecore_Con_Url to Efl_Network_Url.Srivardhan Hebbar2015-05-185-43/+92
| | | | | | | | | | | | | | Summary: This is just the beginning. I tried for one class to check. Tell me if this is fine, I'll change in other classes also. The goal is to simplify and make our API clearer to understand to new comers. Reviewers: cedric Subscribers: cedric Differential Revision: https://phab.enlightenment.org/D2468 Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>
* ecore_con: use portable environment lookup.Cedric BAIL2015-05-141-15/+6
|
* ecore_con: it doesn't make sense to get port on local protocol.Cedric BAIL2015-05-141-0/+9
|
* ecore_con: add missing declarations on Windows.Vincent Torri2015-05-141-2/+4
| | | | Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>
* eolian: buttsDaniel Kolesa2015-05-121-2/+2
|
* eolian: fix doc comments across the treeDaniel Kolesa2015-05-112-4/+4
|
* eolian: change all EFL .eo files to use new syntax for propertiesDaniel Kolesa2015-05-074-18/+16
|
* ecore: remove the need to order the header correctly for Windows.Cedric BAIL2015-05-071-0/+3
|
* ecore_con: fix double free issue in ecore_con_eet.Srivardhan Hebbar2015-04-281-1/+1
| | | | | | | | | | | | | | | | | | Summary: While creating ecore_con_eet_server, if client is not set to false, then while deleting, it is checking for client and it is by default not false. So set it. ecore_con_reply object is being deleted twice. So removed code to delete the reply object as it would be deleted after the callback. Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com> Reviewers: cedric Subscribers: cedric Maniphest Tasks: T2372 Differential Revision: https://phab.enlightenment.org/D2445 Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>
* ecore_con: fix typos and improve document consistence.Thiep Ha2015-04-251-90/+90
| | | | | | | | | | | | | | | | Summary: This patch fixes typos and uses primitive verb in brief section to improve document consistence for ecore_con. Reviewers: Hermet, cedric Reviewed By: cedric Subscribers: cedric Differential Revision: https://phab.enlightenment.org/D2424 Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>
* ecore-con: try to load libcurl.so.5 before .so.4Raoul Hecky2015-04-241-0/+4
| | | | | | | Curl is using a strange soname bumping where soname version is changed by configure when it detects some differences with off_t. Explanations here: http://curl.haxx.se/dev/readme-curl_off_t.html As we are not using curl_off_t in our code, we can simply load libcurl.so.5 if present.
* ecore_con: fix dns lookup crash issue.Srivardhan Hebbar2015-04-221-2/+0
| | | | | | | | | | | | | | | Summary: When executed ecore_con_lookup_example function, we were getting a crash. This is the fix for the crash. I analysed it in the valgrind and found the error. Valgrind reported invalid free at this place. svr->name is getting freed in _ecore_con_server_eo_base_destructor function and svr is getting freed in ecore_con_shutdown function. Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com> Reviewers: cedric Subscribers: cedric Differential Revision: https://phab.enlightenment.org/D2398 Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>
* Ecore_Con: Fix clang warningsJean-Philippe Andre2015-04-211-2/+2
|
* ecore_con: fix stack overflow in ecore_con_url_timer issue.Srivardhan Hebbar2015-04-101-21/+14
| | | | | | | | | | | | | | | | | Summary: last_ms is not needed at all. If curl_multi_timeout returns non zero, then its error, if it returns zero, then its fine. If ms is zero, then call curl_multi_perform immediately. Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com> Reviewers: cedric Subscribers: cedric Maniphest Tasks: T2200 Differential Revision: https://phab.enlightenment.org/D2294 Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>
* ecore_con: fix ftp upload function to follow documentation.Srivardhan Hebbar2015-03-271-2/+2
| | | | | | | | | | | | | | | | | Summary: While creating a Ecore_con_url object, the url is given in this format "ftp://ftp.example.com". While uploading a file, this function was prefixing "ftp://" to this url which resulted in DNS failure, and upload fail. So corrected the issue. Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com> Reviewers: cedric Reviewed By: cedric Subscribers: cedric Differential Revision: https://phab.enlightenment.org/D2221 Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>
* ecore_con: do not try to load /etc/resolv.conf when on windows.Nicolas Aguirre2015-03-261-0/+2
| | | | Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>
* ecore_con: clean up, refactor and unpdate documentation.Srivardhan Hebbar2015-03-232-54/+79
| | | | | | | | | | | | | | | | | | | | | Summary: 1. Have refactored code in ecore_con_local.c 2. Changed env variable from "TMP" to "TMPDIR". 3. Added check for negetive port number when type is ECORE_CON_LOCAL_USER. 4. Added check to check TMPDIR before assuming /tmp as temp directory. 5. Updated documentation in Ecore_Con.h explaining about local socket and port number. Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com> Reviewers: cedric Reviewed By: cedric Subscribers: cedric Differential Revision: https://phab.enlightenment.org/D2194 Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>
* ecore_con: fix ECORE_CON_LOCAL_SYSTEM use with negative port number.Srivardhan Hebbar2015-03-181-22/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Summary: The socket can be created even with negative port number, but in that case the port is ignored so that you can connect to non Ecore_Con based IPC. This patch remove that test to make the client and server match. @fix Bug: While creating ECORE_CON_LOCAL_SYSTEM server and client pair, when the socket name was "test_socket" and port number "-8" (Any negative number). Then while creating listening socket, the code would go to line no 291 and socket is created in tmp by ".ecore_servicetest_socket|-8". When the same is passed to bind then the code would go to line 118 and the socket it would try to bind would be ".ecore_servicetest_socket" !! So the bind would fail. Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com> Reviewers: cedric Reviewed By: cedric Subscribers: cedric Differential Revision: https://phab.enlightenment.org/D2186 Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>
* ecore_con: fix build with -DISCOMFITOR and OpenSSL.Cedric BAIL2015-03-171-1/+1
|
* ecore_con: remove deprecated use of Eina_Binbuf functions.Cedric BAIL2015-03-171-4/+4
|
* ecore_con: unbreak GNUTLS support.Cedric BAIL2015-03-171-1/+1
|
* emile: Add SSL support.Cedric BAIL2015-03-171-2/+0
|
* ecore_con: initialize emile cipher support on demand.Cedric BAIL2015-03-171-0/+3
|
* ecore_con: depend on emile for initializing crypto library.Cedric BAIL2015-03-171-65/+15
|
* ecore_con: move Socks API documentation to Ecore_Con.hSrivardhan Hebbar2015-03-132-166/+186
| | | | | | | | | | | | | | Summary: Reviewers: cedric Reviewed By: cedric Subscribers: cedric Differential Revision: https://phab.enlightenment.org/D2138 Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>
* ecore_con: fix rebase issue.Cedric BAIL2015-03-071-2/+2
| | | | Catched by Coverity CID 1287124.
* ecore_con: Not returning error if port is less than zero for local sockets.Srivardhan Hebbar2015-03-061-2/+9
| | | | | | | | | | | | | | Summary: If the socket is local, then there is no need to through error if the port is less than 0 (i.e., negetive). The behavior is same in _ecore_con_connector_eo_base_finalize funtion. So applied the same here. There is code in ecore_con_local.c to handle if port is less than zero for local sockets. In _ecore_con_connector_eo_base_finalize function, I've added a space just so that it shows difference in phabricator and would be easy for you to review. Reviewers: cedric Reviewed By: cedric Subscribers: cedric Differential Revision: https://phab.enlightenment.org/D2089
* ecore_con: calling ecore_con_local_init/shutdown from ecore_con_init/shutdown.Srivardhan Hebbar2015-03-061-0/+2
| | | | | | | | | | | | | | | | | Summary: ecore_con_local doesn't currently require any initialization, but it might need that on some platform and not having the init make the source code non obvious. This is just a patch to improve future maintenability. Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com> Reviewers: cedric Subscribers: cedric Differential Revision: https://phab.enlightenment.org/D2098 Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>
* Fix code to conform to recent Eo changes.Tom Hacohen2015-02-231-15/+30
|
* ecore_con: move documentation of ssl functions from ecore_con_ssl.c to ↵Srivardhan Hebbar2015-02-202-126/+125
| | | | | | | | | | | | | | Ecore_Con.h Summary: Moved documentation of ssl functions from ecore_con_ssl.c to Ecore_Con.h. Reviewers: cedric Subscribers: cedric Differential Revision: https://phab.enlightenment.org/D1994 Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>
* ecore_con_url: add ecore_con_url_head()Boris Faure2015-02-192-2/+32
| | | | | | Just like ecore_con_url_get() or ecore_con_url_post() but to a HTTP HEAD. @feature
* ecore_con_url: use an enum instead of multiple definesBoris Faure2015-02-191-8/+13
|
* ecore_con: updatet documentation about Ecore_Con events.Srivardhan Hebbar2015-02-161-9/+14
| | | | | | | | | | | | | | Summary: Ordered events into server side events, client side events and update the text. Reviewers: cedric Reviewed By: cedric Subscribers: cedric Differential Revision: https://phab.enlightenment.org/D1976 Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>
* ecore_con: EAGAIN is replaced by WSAEWOULDBLOCK on Windows.Guillaume Friloux2015-02-111-0/+4
|
* ecore: improve network events handling on Windows.Guillaume Friloux2015-02-111-2/+2
|
* ecore_con: do not load /etc/hosts on Windows.Guillaume Friloux2015-02-111-2/+3
|
* ecore_con: make disconnects work on Windows.Guillaume Friloux2015-02-111-0/+7
|
* ecore_con: fix use of FormatMessage for error reporting on windows.Guillaume Friloux2015-02-111-3/+4
| | | | @fix
* ecore_con: silent warning with send() usage.Guillaume Friloux2015-02-111-8/+8
|
* ecore_con: use send() instead of write() for portability.Guillaume Friloux2015-02-111-5/+1
|
* ecore_con: use recv() instead of read for better portability.Guillaume Friloux2015-02-111-4/+6
|
* ecore_con: fix _ecore_con_server_flush on Windows.Guillaume Friloux2015-02-111-5/+35
| | | | @fix