eldbus: Remove unnecessary function - _eldbus_connection_unref()

Neither internal function call this function.

eldbus: Handle removal of last reference of connection inside of callback

Updated Eldbus connect_address example

See description on the header of the source file

Signed-off-by: Eduardo Lima (Etrunko) <eduardo.lima@intel.com>

eldbus - better fix for dbus_conn becoming null in dispatch

eldbus - only unref client conns that are not null after dispatch

eldbus: Missing dbus_connection_unref() in error path

Signed-off-by: Eduardo Lima (Etrunko) <eduardo.lima@intel.com>

eldbus: Call dbus_bus_register() for TYPE_ADDRESS connections

From D-Bus documentation:

http://dbus.freedesktop.org/doc/api/html/group__DBusBus.html

dbus_bus_register():

    If you open a bus connection with dbus_connection_open() or
    dbus_connection_open_private() you will have to dbus_bus_register()
    yourself, or make the appropriate registration method calls yourself.

Signed-off-by: Eduardo Lima (Etrunko) <eduardo.lima@intel.com>

eldbus: Add proxy call to org.freedesktop.DBus.Hello

Signed-off-by: Eduardo Lima (Etrunko) <eduardo.lima@intel.com>

eldbus: error message iters should be initted as readonly.

more descriptive error message in eldbus when property fetching fails

inform users when eldbus-codegen is cowardly refusing to generate code

use sizeof for eldbus-codegen internal define lengths

s/LENGHT/LENGTH

eldbus: Fix crash when eldbus is shutdown and we have a live connection of type address

Also show info about each connection of type address live.

Example for eldbus_address_connection_get() functions

Signed-off-by: Eduardo Lima (Etrunko) <eduardo.lima@intel.com>

eldbus: Welcome two new APIs and one connection type

eldbus_address_connection_get() and eldbus_private_address_connection_get()
are similar to the respective _connection_get() counterparts, but enables
users to connect to buses other than system or session.

The new type introduced for those connections is ELDBUS_CONNECTION_TYPE_ADDRESS
and they require an additional address parameter, which will be passed to
dbus_connection_open().

Signed-off-by: Eduardo Lima (Etrunko) <eduardo.lima@intel.com>

eldbus: New internal function _eldbus_connection_free()

This is actually a split on the _eldbus_connection_unref() function, that
will be called either when the refcount reaches 0 or to force the deletion
of an Eldus_Connection.

The second use case will be contemplated in a following patch.

Signed-off-by: Eduardo Lima (Etrunko) <eduardo.lima@intel.com>

eldbus: Reset value of fd_handler after deletion

Signed-off-by: Eduardo Lima (Etrunko) <eduardo.lima@intel.com>

fix eldbus generator memleak when failing to fetch args

eldbus_service: explicitly set flags to 0 in static _Signal and _Method structs

eldbus: Fix interface_unregister()

eldbus: Fix eldbus_service_*_get() functions over FDO interfaces

eldbus: Fix eldbus_pending_destination_get(), it was returnig the method name

eldbus: Add EINA_ARG_NONNULL to new API function

eldbus: Add eldbus_message_iter_fixed_array_append()

Add missing files in previous backport

Revert "Backport fixes from EFL tree"

This reverts commit 7de696d9d258c372d503c8040d79fd061dc3765a.

eldbus: use the right variable name for version.

Backport fixes from EFL tree

Backport changes from EFL

There were lots of changes in eldbus inside EFL. This commit imports all
of them by simply copying the headers and source files over the old
ones. This is the lazy way to do but there were way too many missing
commits, including renames and cleanups.

Docs are probably broken - NOT TESTED. Use the EFL version if you are
interested in the documentation.

build-sys: Define version as in EFL

Don't use deprecated AM_CONFIG_HEADER

Rename edbus->eldbus

Spec file fix.

SVN revision: 81704

edbus: Fix make distcheck.

SVN revision: 81515

edbus codegen: rename tool and move it to the right dir

SVN revision: 81496

edbus codegen: fix check for eina + ecore

This way we get a spurious line in configure, resulting in this message:

/home/lucas/p/edbus/configure: line 14196: ecore: command not found

SVN revision: 81495

edbus: Mark instrospectable as dirty when needed

Patch by: José Roberto de Souza  <zehortigoza@profusion.mobi>

SVN revision: 81494

edbus: Fix edbus_proxy_property_set

Fix the function and add support for complex types, in which case a
Eina_Value is expected.

Patch by: José Roberto de Souza  <zehortigoza@profusion.mobi>

SVN revision: 81493

edbus codegen: Initial commit

Add codegen tool to generate client code.

Patch by: José Roberto de Souza  <zehortigoza@profusion.mobi>

SVN revision: 81492

edbus: add helper function edbus_message_iter_struct_like_to_eina_value()

Patch by: José Roberto de Souza  <zehortigoza@profusion.mobi>

SVN revision: 81489

edbus: add _message_iter_from_eina_value_struct to private header

Patch by: José Roberto de Souza  <zehortigoza@profusion.mobi>

SVN revision: 81488

edbus: avoid cyclic unref

If we are freeing a EDBUS_Connection_Name its name_owner_changed signal
handler may hold a pointer and try to unref it when deleting the signal
handler. We can't simply make the signal handler hold a reference to the
connection name, otherwise edbus_connection_name_gc will never be
triggered because of cyclic references.

Thus, just set the cn->name_owner_changed->bus to NULL before trying to
delete the signal handler.

Related log found by Lucas Jóia:

==20607== Invalid read of size 4
==20607==    at 0x6FE29EE: edbus_connection_name_gc.isra.3 (edbus_core.c:375)
==20607==    by 0x6FE4287: edbus_connection_unref (edbus_core.c:1028)
==20607==    by 0x4C8D94: e_msgbus_shutdown (e_msgbus.c:167)
==20607==    by 0x436194: _e_main_shutdown (e_main.c:1136)
==20607==    by 0x434F25: main (e_main.c:1074)
==20607==  Address 0x1461ba68 is 24 bytes inside a block of size 64 free'd
==20607==    at 0x4C2A739: free (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==20607==    by 0x6FF0E78: edbus_signal_handler_unref (edbus_signal_handler.c:269)
==20607==    by 0x6FE2A48: edbus_connection_name_gc.isra.3 (edbus_core.c:384)
==20607==    by 0x6FE4287: edbus_connection_unref (edbus_core.c:1028)
==20607==    by 0x4C8D94: e_msgbus_shutdown (e_msgbus.c:167)
==20607==    by 0x436194: _e_main_shutdown (e_main.c:1136)
==20607==    by 0x434F25: main (e_main.c:1074)

SVN revision: 81463

edbus: Do not modify cn->names hash while walking it

Bug triggered by Lucas Jóia:

==10042== Invalid read of size 8
==10042==    at 0x6B86626: _eina_rbtree_iterator_next (eina_rbtree.c:165)
==10042==    by 0x6B7228D: _eina_hash_iterator_next (eina_hash.c:622)
==10042==    by 0x6FE41DC: edbus_connection_unref (edbus_core.c:1015)
==10042==    by 0x4C8D94: e_msgbus_shutdown (e_msgbus.c:167)
==10042==    by 0x436194: _e_main_shutdown (e_main.c:1136)
==10042==    by 0x434F25: main (e_main.c:1074)
==10042==  Address 0x15c1b958 is 40 bytes inside a block of size 96 free'd
==10042==    at 0x4C2A739: free (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==10042==    by 0x6B71CB7: _eina_hash_del_by_hash_el (eina_hash.c:441)
==10042==    by 0x6FE2A1E: edbus_connection_name_gc.isra.2 (edbus_core.c:385)
==10042==    by 0x6FE4217: edbus_connection_unref (edbus_core.c:1026)
==10042==    by 0x4C8D94: e_msgbus_shutdown (e_msgbus.c:167)
==10042==    by 0x436194: _e_main_shutdown (e_main.c:1136)
==10042==    by 0x434F25: main (e_main.c:1074)

SVN revision: 81462

edbus: no need for this auxiliary variables, and check refcount first

SVN revision: 81461

edbus: Remove object path of service if ObjectManager is the last optional iface

Patch by: José Roberto de Souza  <zehortigoza@profusion.mobi>

SVN revision: 81416

edbus: Fix undefined ref. to 'EINA_LOG_DOMAIN_GLOBAL' in examples

Patch by: José Roberto de Souza  <zehortigoza@profusion.mobi>

SVN revision: 81415

edbus: Only create one ref of proxys when listen object manager

Patch by: José Roberto de Souza  <zehortigoza@profusion.mobi>

SVN revision: 81414

edbus: (de)yodafy free cb functions

edbus_connection_cb_free_add     -> edbus_connection_free_cb_add
edbus_connection_cb_free_del     -> edbus_connection_free_cb_del
edbus_object_cb_free_add         -> edbus_object_free_cb_add
edbus_object_cb_free_del         -> edbus_object_free_cb_del
edbus_pending_cb_free_add        -> edbus_pending_free_cb_add
edbus_pending_cb_free_del        -> edbus_pending_free_cb_del
edbus_proxy_cb_free_add          -> edbus_proxy_free_cb_add
edbus_proxy_cb_free_del          -> edbus_proxy_free_cb_del
edbus_signal_handler_cb_free_add -> edbus_signal_handler_free_cb_add
edbus_signal_handler_cb_free_del -> edbus_signal_handler_free_cb_del

SVN revision: 81389

edbus: Fix leftover unref after edbus_service_signal_send change

SVN revision: 81388

edbus: unexport edbus_message_signal_new()

Signals need to be sent with edbus_service_signal_emit() -- for basic
messages -- or edbus_service_signal_new() + edbus_service_signal_send --
for complex messages.  Otherwise it's possible to send signals that are
not in the service introspection or that have different signatures by
mistake/typo.

SVN revision: 81311

edbus: unref sent signal, also for complex types

SVN revision: 81310

edbus: rename functions edbus_message{_iter,}_arguments_set

Rename these functions since they do not set the data in the
iterator/message but rather they append the data.

Also improve the documentation of edbus_message_iter_arguments_append()
to clarify its usage.

SVN revision: 81295

edbus: Fix invalid access memory in edbus_service_interface_unregister()

_object_unregister is called synchronized by libdbus, so when _interface_free() ran
your object its already freed.

==30579== Invalid read of size 4
==30579==    at 0x4775190: _find_object_manager_parent (edbus_service.c:803)
==30579==    by 0x4775292: _interface_free (edbus_service.c:1011)
==30579==    by 0x4777F1D: edbus_service_interface_unregister (edbus_service.c:1101)
==30579==    by 0x40CBD28: elm_dbus_menu_delete (elm_dbus_menu.c:128)
==30579==    by 0x414552F: _elm_menu_smart_del (elm_menu.c:562)
==30579==    by 0x4810F39: _eo_op_internal (eo.c:363)
==30579==    by 0x4812E1B: eo_do_internal (eo.c:403)
==30579==    by 0x4279D02: evas_object_smart_del (evas_object_smart.c:1080)

Patch by: José Roberto de Souza  <zehortigoza@profusion.mobi>

SVN revision: 81180

edbus: fix missing blank lines

Apparently the previous sed script didn't caught all missing blank
lines.

SVN revision: 81179

edbus: rename DISCONNECTED event

SVN revision: 81177

edbus: Fix formatting

SVN revision: 81176

edbus: simplify end condition check in loop

SVN revision: 81175

edbus: avoid using dbus_signature_iter_get_signature when not needed

SVN revision: 81174

edbus: Use eina_log in client example and generalize expected values

Keep expected values in a struct. It would be ideal to have all values
and compare functions in an array, so we would be able to set the same
callback function for all methods. But it's already short enough so keep
it as is.

The usage of eina_log here allows us to easily catch which test failed.

SVN revision: 81173

edbus: Fix typo in error message

SVN revision: 81172

edbus: Don't use "bool" as a variable name

Using bool as variable name is asking for trouble in C99. If we end up
including stdbool.h, this would fail.

SVN revision: 81171

edbus: Let arguments_get fail if getting less args than passed

If user passed a string with more elements, return EINA_FALSE on
edbus_message_arguments_get() so he knows not all elements are
initialized. Before this patch, we would notify user of its error if he
did something like:

i) edbus_message_arguments_get(msg, "uu", &a)
ii) edbus_message_arguments_get(msg, "uu", &a, &b)

And "msg" containing only 1 argument.

This also fixes the case in which user is getting the elements of an
array iterator and the array is empty. We were previously returning
EINA_TRUE, even if we were not filling the variable.

Last but not least, if the user was calling
edbus_message_iter_arguments_get() in an empty array, we would return
EINA_FALSE, even if we didn't actually get any element.

SVN revision: 81170

edbus: do not use dbus_signature_iter_get_signature in edbus_message_iter_arguments_vget

we are only interested in the first char of the signature so we can use
dbus_signature_iter_get_current_type and:

a) avoid the allocation of the signature for each complete type
b) simplify the check for struct and dict, since *_get_current_type()
   does TheRightThing (TM)

This also rename some variables to clarify the new semantics:

 iter_type -> iter
 sig_type  -> sig_iter

SVN revision: 81169

edbus: Improve doc of edbus_{private_,}connection_get

SVN revision: 80968

edbus: add edbus.spec to gitignore

SVN revision: 80967

edbus: Remove useless defines

These values are used only using they directly are more meaninful than
those defines.

SVN revision: 80966

edbus: remove unneeded code

SVN revision: 80965

edbus: if you goto end, you don't use else

SVN revision: 80964

edbus: Add missing EINA_WARN_UNUSED_RESULT to service API

SVN revision: 80963

edbus: Add blank lines to separate function docs

find . -name '*.h' -exec \
     sed -i '/^\s*$/b; N; /\n\/\*\*/{P;x;p;x;D}' {} \;

This ensures to add a blank line above a doxygen comment if and only if
there wasn't one.

SVN revision: 80962

edbus: Give proper doc to edbus_service_property_changed()

SVN revision: 80961

edbus: Dont create proxy of FDO.Instropectable

Patch by: José Roberto de Souza  <zehortigoza@profusion.mobi>

SVN revision: 80712

edbus: Dont create proxy of FDO.Peer

Patch by: José Roberto de Souza  <zehortigoza@profusion.mobi>

SVN revision: 80711

edbus: Fix typo in object manager

Patch by: José Roberto de Souza  <zehortigoza@profusion.mobi>

SVN revision: 80710

edbus: Dont create proxy of Object Manager on edbus_object_managed_objects_get()

Patch by: José Roberto de Souza  <zehortigoza@profusion.mobi>

SVN revision: 80709

edbus: Only one ref of Properties interface in each object

Patch by: José Roberto de Souza  <zehortigoza@profusion.mobi>

SVN revision: 80708

edbus: Better naming for FDO proxy in connection

Patch by: Lucas De Marchi <lucas.de.marchi@gmail.com>

SVN revision: 80707

edbus: Keep only one reference of object and proxy of dbus daemon interface per connection

Patch by: José Roberto de Souza  <zehortigoza@profusion.mobi>

SVN revision: 80706

edbus: simplify returning new ref of obj/proxy

Patch by: Lucas De Marchi <lucas.de.marchi@gmail.com>

SVN revision: 80705

edbus: Add ref to object and proxy in edbus_object/proxy_get

Patch by: José Roberto de Souza  <zehortigoza@profusion.mobi>

SVN revision: 80704

edbus: Fix typo on doc of edbus_message_arguments_get

Patch by: José Roberto de Souza  <zehortigoza@profusion.mobi>

SVN revision: 80703

edbus: Add object method call

Patch by: José Roberto de Souza  <zehortigoza@profusion.mobi>

SVN revision: 80702

edbus: Add missing break in edbus_message_from_eina_value

Patch by: José Roberto de Souza  <zehortigoza@profusion.mobi>

SVN revision: 80701

edbus: Add example for reading str array inside a variant using eina_value

Patch by: José Roberto de Souza  <zehortigoza@profusion.mobi>

SVN revision: 80700

edbus: Change some function parameters to const

Patch by: José Roberto de Souza  <zehortigoza@profusion.mobi>

SVN revision: 80699

edbus: Signal property changed after a successful property-set in complex server example

Patch by: José Roberto de Souza  <zehortigoza@profusion.mobi>

SVN revision: 80698

edbus: Fix signature check in edbus_message_iter_fixed_array_get()

Patch by: José Roberto de Souza  <zehortigoza@profusion.mobi>

SVN revision: 80697

edbus: Add Variant to Property set function

Patch by: José Roberto de Souza  <zehortigoza@profusion.mobi>

SVN revision: 80696

edbus: Add proxy to Proxy_Event_Property_Changed

Patch by: José Roberto de Souza  <zehortigoza@profusion.mobi>

SVN revision: 80695

edbus: Remove signal_handler_del over a already freed signal

Patch by: José Roberto de Souza  <zehortigoza@profusion.mobi>

SVN revision: 80694

edbus: Add Disconnected event in connection

Patch by: José Roberto de Souza  <zehortigoza@profusion.mobi>

SVN revision: 80693

edbus: Remove unnecessary connection events

Patch by: José Roberto de Souza  <zehortigoza@profusion.mobi>

SVN revision: 80692

edbus: Add edbus_private_connection_get()

Patch by: José Roberto de Souza  <zehortigoza@profusion.mobi>

SVN revision: 80691

edbus: Refactor examples

Patch by: José Roberto de Souza  <zehortigoza@profusion.mobi>

SVN revision: 80690

edbus: Support '(' and '{' in edbus_message_iter_get_and_next()

Patch by: José Roberto de Souza  <zehortigoza@profusion.mobi>

SVN revision: 80689

edbus: Allow to monitor unique id in edbus_name_owner_changed_callback_add

This allows to monitor when a client exits. Clients in general don't
register a name in the bus and in some cases it's even not allowed to do
(for example an agent talking to bluetoothd, that runs in system bus).

Patch by: José Roberto de Souza  <zehortigoza@profusion.mobi>

SVN revision: 80687

edbus: Fix bug found in refactored banshee example

Refactor edbus_signal_handler_add() so internal signal handlers don't
set the connection free callback. This fixes the bug in which
EDBus_Connection was freeing the signal handler of EDBus_Conenction_Name

==22814== Invalid read of size 4
==22814==    at 0x40564B0: edbus_signal_handler_del (edbus_signal_handler.c:278)
==22814==    by 0x4040E65: _edbus_connection_name_unref (edbus_core.c:507)
==22814==    by 0x404106B: edbus_connection_name_owner_monitor (edbus_core.c:520)
==22814==    by 0x4055F63: _edbus_signal_handler_clean (edbus_signal_handler.c:217)
==22814==    by 0x40564F8: edbus_signal_handler_del (edbus_signal_handler.c:279)
==22814==    by 0x4043088: _edbus_connection_unref (edbus_core.c:1045)
==22814==    by 0x404352F: edbus_connection_unref (edbus_core.c:1105)
==22814==    by 0x80498AA: main (banshee.c:233)
==22814==  Address 0x44bea48 is 0 bytes inside a block of size 72 free'd
==22814==    at 0x402C06C: free (in /usr/lib/valgrind/vgpreload_memcheck-x86-linux.so)
==22814==    by 0x4056118: _edbus_signal_handler_del (edbus_signal_handler.c:249)
==22814==    by 0x4056401: edbus_signal_handler_unref (edbus_signal_handler.c:272)
==22814==    by 0x4056503: edbus_signal_handler_del (edbus_signal_handler.c:280)
==22814==    by 0x4043088: _edbus_connection_unref (edbus_core.c:1045)
==22814==    by 0x404352F: edbus_connection_unref (edbus_core.c:1105)
==22814==    by 0x80498AA: main (banshee.c:233)
==22814==
CRI<22814>: src/lib/edbus_signal_handler.c:278 edbus_signal_handler_del() *** Eina Magic Check Failed !!!
    Input handle has already been freed!
    *** NAUGHTY PROGRAMMER!!!
    *** SPANK SPANK SPANK!!!
    *** Now go fix your code. Tut tut tut!

Patch by: José Roberto de Souza  <zehortigoza@profusion.mobi>

SVN revision: 80686

edbus: Refactor banshee example

Patch by: José Roberto de Souza  <zehortigoza@profusion.mobi>

SVN revision: 80685

edbus: Refactor edbus_connection_name

Simplifications on "monitor" of NameOwnerChanged
Life cycle tied with refcount, objs, and name_owner_changed list.

Patch by: José Roberto de Souza  <zehortigoza@profusion.mobi>

SVN revision: 80684