Ecore_xcb_keymap: Fix memory leak in _ecore_xcb_keymap_finilize

Summary:
xcb_get_modifier_mapping_reply_t *reply is obtained from
xcb_get_modifier_mapping_reply and should be freed after use.

Reviewers: raster, Hermet, tasn, zmike

Subscribers: singh.amitesh, yashu21985, alok25, sachin.dev, cedric

Differential Revision: https://phab.enlightenment.org/D3167

@fix

eolian-cxx: Fixed possible exception escaping main

Added try/catch around main to show the reason for the generation failure.

@fix CID 1265600

eolian-cxx: Fixed assert which was a tautology

Fixed the assert to the correct check

@fix CID 1316018

Evas engine wayland_shm: pass the correct enum value.

Following this patch, the correct enum type is now passed to this
function. It's also passing the same value (0), but now from the correct
enum. This doesn't change behaviour, and looks like what was intended.

This fixes the clang warning.

ecore-wayland: Comment out unused functions

Summary: As we do not listen for surface enter/leave events, we do not
need these functions so comment them out.

Thanks to Tom for the report :)

@fix

Signed-off-by: Chris Michael <cp.michael@samsung.com>

ecore-wayland: Comment out unused functions for session listener

Summary: As session listener is currently disabled in Ecore_Wayland,
we don't need these functions defined so comment them out.

Thanks to Tom for the report :)

@fix

Signed-off-by: Chris Michael <cp.michael@samsung.com>

Evas engine software generic: Fix clang warning.

Double parenthesis comparison is reserved for evaluation for assignment, not just comparison.
So for example you should do ((x = 2)) and (x == 2), but not ((x == 2)) or (x = 2).

Ector renderer: fix return value to be of the right type.

This function expects a pointer as a return value, not a bool.

@fix.

Ethumb client: Fix shadow issues (warnings).

Eeze sensor udev: Fix memory leak and possible crash.

Due to a misplaced semicolon, EINA_LIST_FREE was doing nothing and only
the last part was freed if there was one, and probably a crash if there
wasn't.

This shows again that having statements without {} is dangerous. We need
to force {} and add a lint rule, or at the very least, add a rule to put
the ; in a new line (like clang warns about by default anyway).

(Same as issue found in previous commit)

@fix

Edje cc handlers: Fix memory leak and possible crash.

Due to a misplaced semicolon, EINA_LIST_FREE was doing nothing and only
the last part was freed if there was one, and probably a crash if there
wasn't.

This shows again that having statements without {} is dangerous. We need
to force {} and add a lint rule, or at the very least, add a rule to put
the ; in a new line (like clang warns about by default anyway).

@fix

Ector renderer: fix return value to be of the right type.

This function expects a pointer as a return value, not a bool.

@fix.

Eio model test: initialise pointer (clang warning).

This is probably not an issue because the function should always
return a value, but initialising this variable silence clang's
warning and is not harmful anyway.

Eo: Fix Eo on Windows.

We use function names instead of function pointers of Windows, because
of dll import/export issues (more in a comment in eo.c). Before this
commit we were comparing the pointers to the strings instead of the
content in some of the places, which caused op desc lookup not to work.
This fixes that.

Thanks to vtorri for his assistance.

@fix

evas_canvas3d: fix mesh being added twice added to scene data

Summary: @fix

Reviewers: cedric, Hermet

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D3143

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

ecore_con: preventing possible NULL pointer dereference.

Summary:
Checking if client is NULL before freeing is not of much use. If it is NULL, it would have crashed before when it was dereferenced in printf. So checking NULL before dereferencing.
Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com>
Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D3151

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

ecore_exe: correcting spelling mistake in comment.

Summary:
Corrected the spelling mistake of mechanism.

Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com>
Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D3152

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

ecor_xcb: removing useless assignment.

Summary:
The assignment of NULL will have no effect on the caller. So removed that statment.

Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com>
Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D3153

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

ecore_xcb: initializing the uninitialized variable.

Summary:
realwin is uninitialized and is being checked against 0 at 384. It is set at 377 only if win2 == win, else it remains uninitialized. So initializing it to NULL.
Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com>
Reviewers: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D3154

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

ecore_xcb: handle realloc failure.

Summary:
If realloc fails, then lst would be NULL, so assigning lst[i] only if realloc is successful, else assigning lst to previous memory location.

Signed-off-by: Srivardhan Hebbar <sri.hebbar@samsung.com>
Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D3155

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

eio_monitor: fix memory leak in eio monitor module

Summary:
Free the allocated memory in eio monitor win32 module.

Signed-off-by: Vivek Ellur <vivek.ellur@samsung.com>
Reviewers: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D3160

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

eio_monitor: fix memory leak issue

Summary:
@fix

Signed-off-by: Vivek Ellur <vivek.ellur@samsung.com>
Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D3161

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

ecore_fd: add _ecore_main_fdh_poll_del() when fd handler is deleted.

Summary:
When fd handler is deleted by ECORE_CALLBACK_CANCEL, _ecore_main_fdh_poll_del() is not called.
So fd still exists in epoll's event pool.

Reviewers: raster, seoz, woohyun, Hermet, cedric

Reviewed By: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D3131

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

ecore-evas x - solve internal window size flickrs in e

so there is an issue that e brings out where configure events get
queued and deferred AND e ends up requesting a new size, but new size
is wrong as its read from an old event (requested size is updated) and
in the end ecore-evas doesnt request the actual new size because
current w/h is "the same" even though it isn't... bah - it's complex
and a self-feeding event issue. just doing the move/resize solves it.

@fix

ecore-evas-wayland: Set ee->draw_ok appropriately based on visibility

Summary: A previous code change to the common code which processes
updates is relying on having the ecore_evas->draw_ok flag set in order
to process updates. This change makes sure that ee->draw_ok is in sync
with canvas visibility.

@fix

Signed-off-by: Chris Michael <cp.michael@samsung.com>

evas-wayland-shm: Fix silly typo

Summary: OOpppss, this should be eina_rectangle_free

Signed-off-by: Chris Michael <cp.michael@samsung.com>

ecore-evas-wayland-shm: Check for fullscreen and override windows when
processing updates

Summary: When processing render updates, we should be checking if the
Ecore_Evas "should be visible" property is set.

@fix

Signed-off-by: Chris Michael <cp.michael@samsung.com>

evas-wayland-shm: Don't leak rectangles during flush

Summary: As we loop the regions of the output buffer and accumulate
changes to post, we should be freeing the rectangles when we are done
with them.

@fix

Signed-off-by: Chris Michael <cp.michael@samsung.com>

evas-wayland-shm: Fix to add support for MERGE_MODE Smart

Summary: Seems we were never setting any merge_mode for the
wayland-shm engine. This fix implements setting merge_mode

@fix

Signed-off-by: Chris Michael <cp.michael@samsung.com>

Evas filters: Fix crash with async sw rendering

If the filtered object (text or image object) was deleted, its
output image (cached inside the filter data) would be freed
immediately. This could cause crashes in case of async rendering.

@fix

eet - be robust about garbage at the end that looks like an idenity

if there is an identity signaure at the end, ONLY check it if it looks
like a real one (correct magic number, cert and sig size fields are
sane etc.). this means eet opens dont fail for files that may have
trailing garbage or padding that is not an eet identity signature.

ethumb - fix client free issue with pending dbus messages

this fixes T2769

@fix

eet - fix signature alignment and position

Evas textblock: Force relayout during proxy render

If the textblock object was not visible in the main canvas, but
still needs to be rendered in a proxy surface, then _relayout may
not have been called. This forces generation of paragraphs based on
the current geometry.

This patch is ugly. I know. This is evas render :)

Evas render: Fix another bad proxy clipping issue

This is an ugly hack to fix an issue reported in D3114. I don't
understand how the proposed patch could even fix anything given
the current situation.

Test case:
 - Create edje object with textblock inside
 - Clip out edje object (--> all children become not visible)
 - Take textblock from edje and set it as source of a proxy
 - Mark proxy as source_clip

Result: Nothing visible.

Expected: Proxy should contain the textblock object, since
  source_clip means we ignore the edje object's clipper, and
  only care about the textblock's clipper (entire canvas).

Here's what was happening:
 - During a first pass, textblock is not visible, cur->cache.clip
   is calculated, marked as clip_use=1 with geom 0,0 0x0
 - In a second pass, the proxy is rendered, which needs to draw
   the textblock in a surface. But cache.clip was used and it was
   wrong.

Solution:
 - Ignore cache.clip when rendering inside a proxy. I'm pretty
   sure there are other instances where cache.clip will still
   be a problem.

Problem: textblock never called relayout since it was not
  visible.

Conclusion: cache.clip needs to die. It's a legacy optimization
  that now causes more issues than it fixes.

EDC Doc: Some fixes

We need to move this doc to the wiki and complete it :)

ecore_x: Remove XPrint usage

Summary:
Xprint has been deprecated since 2008.

It's recently (August 2015) been removed from debian.

Reviewers: zmike, devilhorns

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D3150

ecore-evas-gl-drm: Disable async rendering for EGL

Summary: Disable async rendering for EGL engine as EGL is sync only.
This fixes gl_drm engine to work (in my tests) using ecore_evas
example apps.

Signed-off-by: Chris Michael <cp.michael@samsung.com>

eina_quaternion: add since information for new APIs

eina_matrix: adjust since version for new APIs in 1.16

po: update po files

Evas image: Fix wrong usage of engine data macro

ENDT macro should be passed to engine functions instead of
some other function pointer!
@fix

evas/gl_x11: Fix indentation and invalid error check in native set/bind/unbind

evas sw x11 - xcb code has same changes as xlib with optimizations

evas engines - fix partial swap to not skip swaps if full mode used

@fix

evas sw generic - printf debug remove

eldbus - only do syntax checks if libdbus is new enough

ecore-wayland: Don't use custom animator when in server mode

Signed-off-by: Chris Michael <cp.michael@samsung.com>
Reviewed-by: @zmike
necessary to avoid breaking the compositor canvas animator

ecore-evas-wayland-egl: Add support for custom Ecore_Wl animator

Summary: This adds support for ecore_wl to handle ecore animators and
thus the egl engine can receive frame callbacks now.

Signed-off-by: Chris Michael <cp.michael@samsung.com>

release: Update NEWS and bump version for 1.16.0-alpha1 release

evas - region updates - go back to tiler as its faster

the overhead didnt show up in y tests. do show up with certain
expedite tests. hmmm. last time i messed with region code it was
actually same speed as tiler. bonus was it was fully accurate.

evas software x11 - set merge mode explicitly to smart or env var

this allows merge mode for software-x11 to be set too like for gl.

@feature

evas software x11 - increase shm buffer size for modern displays

have a bigger shm buffer for larger displays to recycle shm memory more
for sw rendering

@feature

evas rectangle - ensure centers of rects that just move/resize dont rend

calculating updates has been re-rendering centers fo solid rects if
the solid rects moves/resizes with a lot of overlap and objects
underneath changes - their changes show through. this fixes that

@fix

ecore docs - adjust docs to be explicit about ecore main loops nesting

it seems some people think ecore main loops can nest. they can't. be
explicit in docs.

ecore - detect nested ecore_main_loop_begins from silly code

@feature

evas - cutouts - use less cpu by skipping non-intersecting cutout objs

@feature

ector: fix leak of dash info in Cairo backend.

CID 1316270

Ecore con: Prevent possible use of uninitialised buffer on the stack.

I'm not actually sure if it's a false, because finding the possible
options is hard. Just to be safe, it's better to set buf to "" in the
else case. I'm doing this instead of initialising the variable so the
compiler/static analyser will be able to warn us if there are other code
paths that should probably set buf, but don't.

CID 1316016

@fix

Ecore Con: Fix possible timing attacks.

Gist of it: we check, and then there's a window between our check and
the mkdir. We don't really need it anyway, because we just want to mkdir
and if it exists, just go on and do nothing.

CID 1039559
CID 1039558

@fix

Ecore pipe: clean up handling of already read information.

Coverity was complaining about a possible integer overflow. This isn't
actually possible, but coverity has no way to know that because we were
in fact using a too big of a type. I fixed it to be the right type so
now everything should work.

CID 98384

@fix

evas-3d: fix primitives

Summary:
There were warnings after adding primitive in the same frame more then once, the check was added to avoid it

@fix

Reviewers: cedric, raster, Hermet

Subscribers: cedric, artem.popov

Differential Revision: https://phab.enlightenment.org/D3090

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

evas_box: fix children size and position calculation when padding is used for horizontal flow layout

Summary:
Fix children size and position calculation when padding is used

For each child size calculation padding is adjusted
but box height should include  padding.
Secondly, x and y position of children should not  include
the vertical and horizonatal padding as child size has already
included the given paddings.

@fix

Test Plan:
Please modify test_box.c file in
function test_box_vert2 as follows:

   bx = elm_box_add(win);
   evas_object_size_hint_weight_set(bx, EVAS_HINT_EXPAND, EVAS_HINT_EXPAND);
   elm_win_resize_object_add(win, bx);
   elm_box_padding_set(bx, 50, 50);
   elm_box_layout_set(bx, evas_object_box_layout_flow_horizontal, NULL, NULL);
   evas_object_show(bx);

Now,
1. open elementary_test
2. box
3. Box vert 2 (observe box is broken)
4. Try resizing the window (observe)

Reviewers: raster, cedric

Reviewed By: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D3049

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

eio: make poll backend pass eio_test_monitor.c test suite.

Summary: The current poll backend code uses only filename when reporting changes, while eio tests (and presumably other backends) use absolute path. This makes poll backends behavior more consistent with other backends.

Test Plan: Ran eio test suite with poll backend.

Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric

Projects: #efl

Differential Revision: https://phab.enlightenment.org/D3083

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

Edje test: remove unused variable.

Eo: Remove dead code.

This removes code that became dead in commit:
389c6d35f2908018e8b6ae3f8294c02c6b5073a0
The commit doesn't explain why we don't shrink or grow when using mmap,
but this is how it is. No reason to keep old code there.

CID 1240224

@fix

eolian: fix out-of-bounds indexing on tokens

Fixes CID 1324818 @fix

Revert "eina: change default log level at which we display backtrace."

This reverts commit 05c18876b3e2dba5dd7caea2517aaf24d92ab614.

i endabled backtraces by DEFAULT because it is otherwise pretty damned
hard to get them from things like enlightenment, or on tizen apps as
running them on the cmdline and having them work is close to
impossible. yes it's noise. FIX THE ERRORS then. this commit is just
like turning off gcc etc. warnings because the code is too noisy in
compilation with them on. it's a bug. fix it. the bt lets you do that
without gdb or re-executing AGAIN with an env var and HOPING to find
the bug the next time around.

not to mention efl programming docs cover this backtrace and say it
is on by default and how to get useful info out of this. this makes
the elf docs a lie by removing what is documented as default.

@fix

eldbus - make method calls less crasy since libdbus likes to abort

changes zmike made to e's systray make e abort due to libdbus seeing
the bus name as invalid. it actually is. it's
/org/ayatana/NotificationItem/steam for example - instead of
or.whatever.blah - thuis e aborts continually if you run steam at all.
eldbus should be more crash-proof thus this adds checks so dbus fun
doesnt cause an app to crash if it happens to call incorrect dbus names.

@fix

evas render - async sw - fix context duplication by using proper dup call

valgrind pointed this one out. we access freed memory when we dup a
context because the context CONTAINS ptrs to things like rects for
cutouts. we didnt dup these. use the proper context dup call (and
properly ref pixman color image too). this was a random bug/crash
waiting to happen and valgrind caught it. suprising it hasnt turned up
before :/

@fix

examples: fix eina_tmpstr example after revert of strftime

Was left broken after the eina_tmpstr_strftime revert.

Revert "eina_tmpstr: add eina_tmpstr_strftime"

As agreed on the ML, eina_tmpstr_strftime() should be removed.

This reverts commit abaf29cb768375957c9ee0b64d36034c21c618ea.

Evas GL: Fix render: force pixel_get if dirty

All examples and docs point to using only the dirty flag in
order to trigger a redraw of an Evas GL surface. The commit
21c43528234 broke this behaviour (for a good reason, but not
related to Evas GL).

This is a compatibility fix.

Evas GL: Add new API to get current Evas GL

While this seems to go against the rest of the API (because we
always pass in the Evas GL object), there is no way right now
fully restore a context if there are multiple Evas GL objects.

For instance, an app can use Evas GL from an Elm GLView, and also
use Cairo with another Evas GL at the same time. In that case Cairo
needs to restore the previous Evas GL but the library had no way
of getting the current Evas GL. This is the equivalent of
eglGetCurrentDisplay().

@feature

Evas image: Simplify updates if adding whole region at once

eldbus: fix _eldbus_message_iter_arguments_vget

Summary:
there is a problem with _eldbus_message_iter_arguments_vget.

Assume: We have a array of type y, there are 3 elements in the array.

Lets say we are iterating with

printf("%c", dbus_message_iter_get_arg_type(iter));

and

dbus_message_iter_next(iter);

throuw the iter.

You will see that this will output yyy. As we are having 3 times v.
While the signature of the message iterator is y.

If you now call eldbus_message_iter_arguments_get(message, "y", &cont)
it will return false, with the errormessage in line 766, reason for this
is that the type of the signature iterator differs from the message
iterator. Because of the upper example.
So all in all: The signature given to the method has not to be equal to
the signature of the message iterator, it has to be equal to the
iterated signature, which cannot be found easily with the eldbus api.

The solution is to only iterate in the message iterator that long until
the signature iterator is at the end, if the type differs in this
region, give a error and return false. Otherwise return true.

The sad thing about this is that it is a behaviour break, but the
behaviour is borked, so its a fix.

I tested this over a week now, without any problem in efl/elm/e.

Reviewers: stefan_schmidt, zmike

Reviewed By: stefan_schmidt

Subscribers: zmike, ceolin, simotek, DaveMDS, cedric

Projects: #efl

Differential Revision: https://phab.enlightenment.org/D3074

Edje tests: Test inheritance of flag no_render

edje_cc: Fix inheritance of "no_render" flag

@fix

edje_cc: Add "render" keyword for lazEDC

Since @zmike also thought this could be useful, let's add it.

ector: don't leak cairo context.

eio: improve documentation.

This set of documentation update basically make it clearer that Eio
use Eina and do cross linking of function used by Eio to Eina. It
also copy some of the warning coming from Eina documentation into Eio.

eina: fix misleading documentation regarding enum values.

evas-wayland-shm: Reduce size of resize pool during resize events

Summary: We really don't need to be allocating a shm pool this large
during resize, so reduce the size of the pool.

Signed-off-by: Chris Michael <cp.michael@samsung.com>

ecore_ipc: fix typo in ecore_ipc_server_send() API doc

ecore_imf: ibus - remove logically dead code

fixes CID: 1261460

edje_cc: Add "norender;" as lazEDC for "no_render: 1;"

edje_cc: Allow PROXY as clipper (clip_to and desc.clip_to)

This arbitrary limitation can now be lifted since masking
is well supported (ie. clip_to can point to an IMAGE rather than
a RECT).

@feature

evas-gl-drm: Add support for 'smart' merge mode

Summary: This just adds support for 'smart' merge mode in the evas
gl_drm engine to keep in line with gl_x11 engine.

Signed-off-by: Chris Michael <cp.michael@samsung.com>

evas-wayland-egl: Add support for 'smart' merge mode

Summary: This just adds support for 'smart' merge mode in the wayland
egl engine to keep in line with the gl x11 engine

Signed-off-by: Chris Michael <cp.michael@samsung.com>

edje_cc: Check default state as well

Strangely only the other states were checked for errors. Errors
include:
- no name for state, or "description with missing state"
- invalid clip_to

Also improve the error message a bit

Edje: Fix big oopsie in previous commit

Edje: Turn SNAPSHOT objects into filled images for now

Without that, the image has no fill information. Fill properties
may need to be added to SNAPHOT parts but the default behaviour
should make sense. Before this patch you just get a black rectangle.

Considering how image filters currently work, marking snapshots
as filled by default is not the best solution (need padding_set(0)
to render nicely).

Evas filters: Fix handling of FILL for image filters

Note: Image filters still need a LOT of work to be usable.

Edje: Fix invalid loading of snapshot parts

Edje: Add test case for SNAPSHOT

This is mostly an edje_cc compilation test.

Edje: Add filter support to SNAPSHOT parts

This makes SNAPSHOT a part type on it own, combining the
common and filter subtypes.

This means it is now possible to set an evas filter on
a SNAPSHOT object, just like for TEXT, IMAGE and PROXY.

@feature

Edje tests: Add PROXY part to the filters test case

Only compilation is tested here.
Use edje_player to check the resulting edj if you wanna see what
happens.

Edje: Fix IMAGE filters

Some invalid logic made filters work only in a rare situation.

@fix

Edje: Add support for filters to PROXY parts

Same syntax as for IMAGE and TEXT: description.filter

Edje: Fix filters for IMAGE parts inherited states