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

efl/eet: Fix certifictate file path detection for eet tests.

SVN revision: 81453

efl: Tentative fix to find the ecore_x_version.h also in the ecore tests.

Shows up on buildbot out of tree builds.

SVN revision: 81440

Simple tutorial for Eo. Supposed to help you to write code with Eo and
prevent eventual bugs.
Feel free to ask, comment, modify this tutorial.

My last commit before the end of the world.

Signed-off-by: Daniel Zaoui <daniel.zaoui@samsung.com>
SVN revision: 81425

edje_entry: check NULL parameter

SVN revision: 81418

edje_entry: NULL checking should be processed before accessing

SVN revision: 81417

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

efl/evas_object_image_is_inside: fixed implementation.

This function was basically never working correctly. Everything was
fixed by simulating the evas_object_image_render() workflow, but
instead of actually draw we just check the pixel transparency.

Bugs fixed:

 * fails when image is scaled up (could segv) or down (incorrect values);
 * fails when image is moved to negative x,y;
 * fails when border was being used.

Now everything is fixed and seems to work properly, except I'm not
handling the map and get_pixels() cases, these are marked with ERR()
so we can fix them if someone needs.

SVN revision: 81410

efl/evas_cache: copied images should be considered loaded.

Whenever we copy an image, making it write-able
(evas_object_image_data_get(o, 1)) or just start painting a pristine
buffer (evas_object_image_size_set(o, w, h)), we must mark the image
as loaded to avoid trying to load it (and failing, marking the whole
thing as EVAS_LOAD_ERROR_GENERIC).

SVN revision: 81409

Edje: delete real part object after swallows got handled

Case the real part object(rp->object) is an smart object it start to delete
the whole smart object hierarchy and a child object may be a swallow of this
real part. So just delete the rp->object after swallows got handled.

SVN revision: 81403

EPhysics: on velocity example unregister callback on restart

Unregister callback on restart and avoid trying to use an to be deleted
body.

SVN revision: 81401

efl/evas: Fix XCB/Xlib crash when closing applications

We need to wait draw threads finishing their stuff before freeing canvas.

Signed-off-by: Paulo Alcantara <pcacjr@profusion.mobi>
SVN revision: 81395

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

Efl: Fix FriBidiChar size detection.

It's better than nothing, I don't know what the proper fix is.
FRIBIDI_CFLAGS just doesn't exist. Doesn't that macro we use call
PKG_CHECK_MODULES?

SVN revision: 81385

evas/engines: Introduce render mode

The render mode should be useful for engines other than software one.

Signed-off-by: Paulo Alcantara <pcacjr@profusion.mobi>
SVN revision: 81384

fix 24bpp rendering problem and pixel fetch problem found in qemu/kvm.

SVN revision: 81378

Fix some minor formatting in Alex Wu's cpu patch.

Signed-off-by: Christopher Michael <cp.michael@samsung.com>
SVN revision: 81361

ecore-wayland: (version 2)Fix monitoring ECORE_FD_WRITE defaultly on wayland display fd lead to 100% cpu usage

In ecore_wl_init(), adding wayland display fd with ECORE_FD_WRITE
flag make CPU usage 100%. The proper way to monitor the ECORE_FD_WRITE
is when the wl_display_flush() return value < 0 and errno == EAGAIN.
And if wl_display_flush() return, we remove ECORE_FD_WRITE flag from
the display fd.

Change from v1:
Add idle enterer destroy code into _ecore_wl_shutdown() to avoid
using freed wl_display.

SVN revision: 81360

evas: fixing example

Signed-off-by: Yakov Goldberg <yakov.g@samsung.com>
SVN revision: 81359

edje: Fix memleak in codegen.

Free apiname before we return.

SVN revision: 81335

efl/evas_bidi: Fix memleak.

Make sure we also free udelmin when we retrun from from the for loop early.

SVN revision: 81334

evas: Eo-styled Evas example added.

Signed-off-by: Yakov Goldberg <yakov.g@samsung.com>
SVN revision: 81328

cleaning: remove unneeded $(top_builddir)

SVN revision: 81324

fix returned value

SVN revision: 81323

oops-- -> call c func not mmx funf .. in the c only render path!

SVN revision: 81315

efl/ecore_evas_extn: add release notes about fixed memory bugs

SVN revision: 81314

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

ethumb: better naming for signal arguments

having the same argument names in a D-Bus signal/method is the equivalent
in C to have a function with this signature:

int my_func(int a, int a, int a, int a);

Don't.

SVN revision: 81309

ethumb: don't use edbus_service_signal_new()

this function is being removed from edbus.

SVN revision: 81308

EPhysics: add soft body light implementation

Do not use the evas map 3d light util function for every slice and
avoid the unexpected "mosaic" effect.

SVN revision: 81306

efl/ecore_evas_extn: don't leak bdata.

PS: I cannot backport this to 1.7.x, could someone do this for me?

SVN revision: 81305

efl/ecore_evas_extn: fix memory errors.

 - EVAS_CALLBACK_FREE and EVAS_CALLBACK_DEL were doing the same thing
   at different stages, causing a segv due double free.

 - extn->file.updates and its Ipc_Data_Update were leaking.

PS: I can't backport this to 1.7, but the problem is still
there. Could someone look into those?

SVN revision: 81304

evas/render: Avoid race when sending render data and updates_cb

Signed-off-by: Paulo Alcantara <pcacjr@profusion.mobi>
SVN revision: 81303

efl/ecore_evas: be loud on errors.

SVN revision: 81302

efl/ecore_evas: abort when object does not contain Ecore_Evas.

SVN revision: 81301

efl/ecore_evas: do not silently fail on incorrect interface.

SVN revision: 81300

ethumb: rename edbus functions

SVN revision: 81298

efreet: rename edbus functions

SVN revision: 81296

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

efl: Created Ecore group and added existing Ecore groups to it.

SVN revision: 81293

efl: Created Evil group and added existing Evil groups to it.

SVN revision: 81292

efl: Created Embryo group and added existing Embryo groups to it.

SVN revision: 81291

efl: Created Eina group and added existing Eina groups to it.

SVN revision: 81290

efl: Created Eet group and added existing Eet groups to it.

SVN revision: 81289

efl: Created Evas group and added all existing evas groups to it.

SVN revision: 81288

evas: Update ChangeLog

SVN revision: 81285

evas: Modify software_generic and gl_x11 to with with threaded renderer

SVN revision: 81284

ecore-evas: Async render

SVN revision: 81283

evas: Async render

SVN revision: 81282

evas: Add function to ref an Image_Entry

SVN revision: 81281

evas: Add thread threaded render queue

SVN revision: 81280

efl: Modified section names as doxygen reuses titles for same named sections in different pages.

SVN revision: 81279

efl: Give different names for example pages in different libs.

SVN revision: 81278

efl: Unifying authors page.

SVN revision: 81277

efl: Adding Evas doc images.

SVN revision: 81276

efl: Adding Ecore doc images.

SVN revision: 81275

efl: Adding *.dox files from various libs.

SVN revision: 81274

efl: Adding a unified main page that links to the "mainpage" of libs.

SVN revision: 81273

efl: Add ChangeLog, NEWS and @since for ecore_audio

Vtorri is right of course.

Signed-off-by: Daniel Willmann <d.willmann@samsung.com>
SVN revision: 81268

sorry vincent. i know you dont like thus, but with this commit efl
tree simply is broken and doesnt compile. error here:

...
src/Makefile_Evas.am:1809: unterminated conditionals: HAVE_WINDOWS_TRUE
src/Makefile.am:24:   src/Makefile_Evas.am' included from here
src/Makefile.am:128: unterminated conditionals: HAVE_WINDOWS_TRUE
src/Makefile.am: installing ./depcomp'
automake: ####################
automake: ## Internal Error ##
automake: ####################
automake: undefined condition TRUE' for RECURSIVE_TARGETS'
automake: RECURSIVE_TARGETS:
automake:   {
automake:     HAVE_WINDOWS => {
automake:       type: +=
automake:       where: /usr/share/automake-1.11/am/texinfos.am:
automake:       comment:
automake:       value: dvi-recursive html-recursive info-recursive
pdf-recursive ps-recursive \
automake:   install-dvi-recursive \
automake:   install-html-recursive \
automake:   install-info-recursive \
automake:   install-pdf-recursive \
automake:   install-ps-recursive all-recursive check-recursive
installcheck-recursive
automake:       owner: Automake
automake:     }
automake:   }
automake:
automake: Please contact <bug-automake@gnu.org>.
 at /usr/share/automake-1.11/Automake/Channels.pm line 657
         Automake::Channels::msg('automake', '', 'undefined condition
TRUE\' for RECURSIVE_TARGETS\'\x{a}RECURSIV...') called at
/usr/share/automake-1.11/Automake/ChannelDefs.pm line 208
        Automake::ChannelDefs::prog_error('undefined condition TRUE\'
for RECURSIVE_TARGETS\'\x{a}RECURSIV...') called at
/usr/share/automake-1.11/Automake/Item.pm line 94
        Automake::Item::rdef('Automake::Variable=HASH(0x38cbe20)',
'Automake::Condition=HASH(0x2832a48)') called at /usr/bin/automake
line 4102
        Automake::handle_subdirs() called at /usr/bin/automake line 8305
                Automake::generate_makefile('src/Makefile.am',
'src/Makefile.in') called at /usr/bin/automake line 8602
        Automake::handle_makefile('src/Makefile.in') called at
/usr/bin/automake line 8616
        Automake::handle_makefiles_serial() called at
/usr/bin/automake line 8769
autoreconf: automake failed with exit status: 255
...

i looked at the HAVE_WINDOWS if's and it seems fine to me - i couldnt
find what was missing, so i had to resort to a revert instead of fix :(
sorry :(

SVN revision: 81267

no need to search headers in builddir

SVN revision: 81258

Fix for edje_cc that didn't like insertion of element before the first element

Signed-off-by: Daniel Zaoui <daniel.zaoui@samsung.com>
SVN revision: 81252

[edje/edje_entry] Before cursor is moved by mouse_down, imf_context_reset should be called with current cursor position.

SVN revision: 81209

edje: ecore_imf cursor infomation update is not necessary in focus-out event

SVN revision: 81201

efl: another leak-- in Evas_Object_Text.

SVN revision: 81198

updating esperanto translations

SVN revision: 81195

evas: Update ChangeLog

SVN revision: 81190

evas/common: Prepare soil to land map code for threaded render

SVN revision: 81189

evas/common: Prepare soil to land scaling code for threaded render

SVN revision: 81188

evas/common: Prepare soil to land polygon drawing code for threaded render

SVN revision: 81187

evas/common: Prepare soil to land rect drawing code for threaded render

SVN revision: 81186

evas/common: Prepare soil to land line drawing code for threaded render

SVN revision: 81185

evas: Do not unload pixels from RGBA_Image if cache refcount > 0

SVN revision: 81184

evas: Use refcounted glyph arrays

This is in preparation for threaded render landing: the render thread will
hold a reference to a text object's glyphs while it hasn't been rendered
yet (and will drop that reference after drawing). This changes the internal
API a little bit (evas_common_font_rgba_draw() now takes an Evas_Glyph_Array
instead of an Evas_Text_Props).

SVN revision: 81183

evas: Get rid of RGBA_Image ref member

(It's not by anything in the code.)

SVN revision: 81182

Fix documentation

SVN revision: 81181

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

efreet: rename DISCONNECTED event after edbus change

SVN revision: 81178

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

efl: fix some of the memory leak.

SVN revision: 81136

Fix docs

Signed-off-by: Daniel Zaoui <daniel.zaoui@samsung.com>
SVN revision: 81130

efl: detect all text change properly and don't destroy text if we don't relayout it.

SVN revision: 81061

small change - dont let update rect list for image object become a
runaway endless list if evas doenst come around and render (pick it
up) any time soon - limit to 512 update rects.

SVN revision: 81039

warn-- .. but this is suspicios as we goto the end where is_v and
was_v are used... and never do set them.

SVN revision: 81038

evas - reverted changelog that was not inteded.

SVN revision: 81026