eldbus - fix varargs missing end in iter get and next

there was an error return handler that would return without doing
va_end(). coverity picked this up. this fixes CID 1039883

evas cache - fix logically dead code

fix CID 1039928

evas cache - fix logically dead code

fixes CID 1039929

evas cache - fix logically dead code

fixes CID 1039930

evas cache - fix logically dead code

fixes CID 1039931

evas cache - fix logically dead code

this fixes CID 1039932

evas cache - remove logically dead code

fix CID 1039934

evas cache - fix logically dead code

this fixes CID 1039935

eina_inlist_sorted_state_insert - fix logically dead code

this fixes CID 1039938

eina list insert sorted - fix logically dead code

this fixes CID 1039941

README: Update statement about C++ usage.

Its no longer interfacing to bullet only but also our C++ bindings that
need the C++ compiler.

fix noop self-assignment

this fixes CID 1040028

evas font - fix null access of font format possibility

this fixes NULL access in CID 1099705, 1099703, 1099704

evas scalecache - fix thread deadlock posssibility

there is a re-ordering of how locks are taken and this should cover
that deadlock possibility. fixes CID 1106338

evas preload + locking - fix deadlock

there is a possible deadlock condition where locks are not taken in
the same order (img lock then cancel lock). re-order so this doesn't
happen. this fixes CID 1106339

evas gl - fix possible null tls resource access in bindframebuffer

this fixes CID 1181887

epp - fix bit underflow in bitshift for multibyte

this fixes CID 1193200

xpm - colors cant be 96 bytes long.

since colors are hex, 96 bytes of hex would be 32bytes per r, g and b,
and that would be 128 bits per  r g and b and that just is never seen
or used. also coverity is right in CID 1193201 that we just can't
shift that many, so go down to max 16 bits per rgb which is sensible
and able to be done. (thats 12 bytes max for hex of color).

fix possible focket fd of -1 in cserve

this fixes CID 1193207 where a server fd may be -1, connect fail and
still be -1 due to failed connect.

evas cserve - handle possilbe null ref and fentry pointers

this fixes CID 1193209 with possible NULL refs.

evas ico loader - ensure count is within sane ranges

this hopefully addresses CID 1205000

evas 3d - remove logically dead code

coverity is right. this code is misleading - fixes CID 1210806

fix edje file fetch

this fixes CID 1210811

fix null check of edje file data

this fixes CID 1210812

evas/proxy: When src object has no proxies, redraw flag is set for EINA_FALSE.

[Problem] When obj which has src object(proxy concept) are excluded from render_object,
src object's 'proxy.redraw' isn't change although src obj have not proxies.
because 'proxy.redraw' flag is changed to EINA_FALSE only in obj's subrender() called.

[Resolution] When the count of source obj's proxies is 0, src obj's 'proxy.redraw' is set for EINA_FALSE.

Signed-Off-By: Min Kyoung Kim <mer.kim@samsung.com>

Merge branch 'master' of ssh://git.enlightenment.org/core/efl

Grid: fix a mirroring bug

Summary:
When the grid is mirrored, originally code is just changed (x1,y1) to (x2,y2).
I fix calculate logic for the x-axis.
@fix

Test Plan:
1. elementary_test -> grid
2. click UI-MIRRORING toggle

Reviewers: Hermet, woohyun, raster, seoz

Reviewed By: seoz

Subscribers: cedric

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

edje_pick: fix a copied aliases

Summary:

@fix

Reviewers: raster, Hermet, seoz, cedric

Subscribers: cedric, seoz, Sachiel

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

edje_pick: fix a segfault on 'set' copy

Summary:
wrong structure, a Edje_Image_Directory_Set_Entry, used for copy
a set's data to new file.

@fix

Reviewers: raster, seoz, cedric, Hermet

Reviewed By: Hermet

Subscribers: Sachiel, cedric, seoz

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

evas 3d - fix divide by zero issue from coverity

this fixes CID 1216432

edje edit - clarify src a bit better (change no logic)

edje edit - make source set api clearer as to what it does for coverity

this addresses CID 1222452 - this fallthough is intended, so
document it and make the code clearer

Revert "Added support for xdg-shell protocol in ecore-wayland"

This reverts commit 2f4db577b524e0813ee0777642a2690d252977fd.

Revert "ecore-wayland: Fix formatting"

This reverts commit ff299727038070a34538c208b2031c6d16c18e1f.

Revert "ecore-wayland: Calloc Can fail, so check that it works (or return if"

This reverts commit a49835a8d165fa5844a902eec035c63036369632.

Revert "ecore-wayland: Fix formatting"

This reverts commit e14f40a46d5009ff3a6771a20d1b6fac66470cd5.

Revert "ecore-wayland: Add missing EINA_UNUSED for unused function paramater"

This reverts commit 67766929ca56da08dfe28ff0f69930244d53d10c.

Revert all the xdg_shell changes. Should not have been pushed due to
new API function.

Revert "Added support for xdg-shell protocol in ecore-wayland."

This reverts commit e591dff56dd36c6f45d28dbbaa05b290d2996113.

Revert this. Should not have been pushed yet. Sorry Stefan !! :)

evas-software-x11: Remove EINA_UNUSED for function paramater that is
used

Render_Mode is used here so we don't need EINA_UNUSED for it in the
function.

@fix

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

Added support for xdg-shell protocol in ecore-wayland.

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

Reviewers: devilhorns

Reviewed By: devilhorns

Subscribers: cedric

Projects: #efl

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

ecore-wayland: Add missing EINA_UNUSED for unused function paramater

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

ecore-wayland: Fix formatting

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

ecore-wayland: Calloc Can fail, so check that it works (or return if
not).

@fix

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

ecore-wayland: Fix formatting

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

Added support for xdg-shell protocol in ecore-wayland

Summary: Signed-off-by: vivek <vivek.ellur@samsung.com>

Reviewers: devilhorns

Subscribers: cedric

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

evas-gl-common: Fix invalid use of Texture

We cannot use the texture to find a valid format Before the texture is
actually created. This is invalid, leads to "warning: tex is used
uninitialized' message from the compiler, and just plain wrong.
Instead, use the alpha property of the Evas_GL_Image to find the
proper texture format, Then create the texture.

@fix

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

autotools: Renamed --disable-c++11 to --disable-cxx-bindings.

The name was misleading.

Fix T1438

eolian-cxx: Renamed event_*_callback_{add,call} to callback_*_{add,call}.

ecore_drm_dbus: Do not use the same name for global and local variables

We use conn here for a global Dbus connection variable as well as sometimes
in a local scope. Better rename the global one to avoid confusion and
potential failures when the code would fallback to the global one even if
the local scope should be used and the var was simply forgotten.

evas 3d - fix error handling for ftell from coverity

coverity was right again - CID 1222453 - fix.

evas 3d - fix null access for fopen on 3d mesh save

this fixes CID 1222455 yes fopen for write may fail!

evas key bitshifts for 64bit type - fix.

coverity is right. this is another set of thse bitshift problems with
unsigned long long. fix 1224346, 1224345, 1224344 1224343 and 1224342

edje - remove logically dead code

fixes CID 1224348 - coverity is right

edje - fix coverity complaint about invalid mem access

fixes real issue for invalid object - CID 1224349

fix coveirty range complaint on modifer bit mask

this fixes CID 1224356, 1224355, 1224354, 1224353 and 1224352

evas software generic engine - fix tb configuring to be consistent

this addresses CID 1224761 - consistent checks for tb in render engine

eina thread queue - check write return

this addresses CID 1226191. this should never have an error as the fd
should always be valid as long as needed.

evas gl x11 engine do extra check to make coverity happen

this fixes CID 1226192

ecore x vsync - be more conservative about DISPLAY string src

i hope this addresses CID 1229131 - don't trust the DISPLAY var
content much at all - limit it to [a-z][A-Z][0-9][-] only. hopefully
coverity is happier.

fix eina unicode realloc fix of coverity complaints - return right buf.

eina thread queue - fix possible deadlock

coverity pointed out a deadlock - CID 1226193. i hope this fixes it
(need to wait days for a coverity re-run). tests pass and real life
usage works, so i assume its ok. in theory i see no problem either.

embyro-cc - fix coverity overruun issue

overrun in fatal msg access - fix CID 1039315

embryo_cc - coverity mem access issue - fix

this fixes CID 1039316 in case error num is beyond errmsg bounds

embryo_cc - fix coverity buffer overrun

fixes CID 1039317

fix coverity leak detection in ecore_evas_x hints...

this fixes CID 1188012

evas - cserve - dummy slave - shut coverity up and unmap mmaped mem

while right, coverity doest know that this is just a dummy example,
so... make it happy by unmapping. fixes CID 1193218

fix possible resource leak in edje entry - coverity spotted

this should fix CID 1193228

eina - and follow-on calloc fixup - 0 terminate first calloced string

this follows on from 6dc52db881cbe29dfe8b9a2a0a6745a8b3fa0673

eina unicode - fix calloc stupidity

in looking at CID 1230994 i noticed that eina_unicode_unicode_to_utf8
and eina_unicode_utf8_to_unicode are really dumb. they calloc an array
of bytes then proceed to fill them all in anyway. why? also the
realloc handing in eina_unicode_unicode_to_utf8 wasn't strictly corect
and could leak memory. so this just fixes silly slow code and a leak.

address non nul terminated string due to strncpy

this addresses CID 1230994. as such  eina_unicode_unicode_to_utf8()
always returns a nul terminated string. so it's guaranteed. but yes -
if string is 7 bytes or longer it will not put a nul byte on the
destination. as such for a single unicode char this can never happen
as in utf8 it's 6 bytes. but since eina_unicode_unicode_to_utf8()
safely returns a nul terminated string at all times - we can just use
strcpy safely. no need for strncpy. also handle null return from
eina_unicode_unicode_to_utf8()

fix resource leak on format lookup fail

so in fixing one CID for a format lookup failing and thus invalid mem
access, a possible leak was made in this case. fix that by doing
lookup before any allocation and if lookup fails, return there

this fixes CID 1230999 1230998 1230997

ecore-con - fix possible leak due to other fix

this fixes potential leak due to
ff652084e028b3c8e8f69cf16baecbad32bb40e7 fixing a valgrind access
issue. make sure the server is in the list before handling error
conditions in case a callback sets delete_me (should never though).

fix non 0 terminated buffer in ethmubd slave

this fixes a potential non 0 terminated string buffer in ethumb slave
process -  harmless for apps, but might happen. this fixes CID 1193232

autotools: try to simplify our Makefile.am to reduce warnings.

autotools: use non deprecated macro.

I still don't know why there is still a warning.

autotools: do not build upower module on Windows system.

Evas textblock: strcpy->strncpy (although the buffer is always big enough).

Many tools will complain about the use of strcpy, and it's not that hard to use
strncpy anyway.

CID1039581.

Evas bidi: Make code more explicit (silence coverity).

CID1039922.

Evas textblock: remove checks for impossible null derefs.

This improves code clarity based on input from coverity.

CID1190401
CID1190402

Edje dbg_info: Mark root as not null (will never be).

evas gl common - fix possible negative array accesses if search format fails

this should fix CID 1223506, 1223505, 1223504

evas gl common - fix potential invalid access during texture free

this should address CID 1223507

evas-wayland-egl: Add missing free

Summary:
Add free to eng_gl_context_free
@fix

Test Plan: N/A

Reviewers: devilhorns

Reviewed By: devilhorns

Subscribers: cedric

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

evas-wayland-egl: Fix typos about egl context attribute list

Summary:
Fix typos 3 to 2.
@fix

Test Plan: N/A

Reviewers: devilhorns

Reviewed By: devilhorns

Subscribers: cedric

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

wayland-egl: Fix Outbuf structure not having evas & engine info

gl_generic functions expect the Outbuf to contain the canvas, so we
fixed the eng_window_new function to accept that, And the engine info.
This simplifies the calls to eng_window_new.

@fix

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

wayland-egl: Fix calls to eng_window_new

Since we simplified the eng_window_new function to accept the canvas
and engine info, we need to update calls to that function

@fix

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

wayland-egl: Fix eng_window_new function prototype

We need to pass in the canvas and engine info structure as the
gl_generic engine needs the outbuf->canvas to be set.

@fix

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

evas-wayland-egl: Fix evas wayland_egl engine crash on mesa driver

Summary:
Evas wayland_egl engine crashes on startup when it is running with mesa.
Evas is trying to get EGL / GLES extension function addresses where evas_gl_symbols.
It is called before eglGetDisplay. Then mesa considers that current egl platform is
X11 if it has no EGL_PLATFORM env var. This behavior of mesa is bad. But we should
provide workaround until it is fixed. Thus EGL_PLATFORM env var should be defined to
wayland before calling eglGetProcAddress.
@fix

Test Plan: run elementary_test under wayland with wayland_egl engine.

Reviewers: raster, stefan_schmidt, devilhorns

Reviewed By: devilhorns

Subscribers: cedric

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

eet: fix possible ABI break.

eina: rename to eina_module_symbol_global_set

As discussed on the ML, eina_module_global_set should become eina_module_symbol_global_set.

Evas text: fix text object size with empty texts.

@fix.

fix valgrind report on ecore con server connect failure

this should fix T1508 - my testing seems to indicate so. test suite
succeeds, some manual tests of mine work. so is ay this is a good fix
that could fix critical segvs in apps using ecore-con - eg like
weather module in e

release: Update NEWS and bump version for 1.11.0-beta2 release

autotools: proper fix for generated Makevars and distcheck

Reverts 21da4a54545cd6871d1d888566ce9fe488afef67.

It is needed to generate Makevars in-tree even when building out-of-tree because of
how Autotools work. However, distcheck doesn't properly remove the Makevars file in
the generated distdir and makes po/ read only, preventing the build system from
generating an up-to-date version of Makevars. This commit adds the required hooks
needed to fix this behavior.

more versions of vbox mods panic - blacklist all vboxvideo users from vsync

imf - fix x11 vs wl fix to leave x11 alone for those without scim

scim has a nasty habit of sitting and hanging if tryng to set up and
no scim daemon is there. since in x11 you may have multiple input
methods, leave it to be manually set up via env var, and if not then
dont work at all.

ecore-imf - fix broken state on non-x11 environments

this fixes ecore-imf to auto-pick scim or wayland vased in DISPLAY and
WALAND_DISPLAY env vars. it's not a perfect/good/correct fix. it
should be picking based on canvas engine type, but that is a far more
invasive effort, so for release, do this simple one that brings back
wayland input panel and keeps x11 input methods going as long as app
is either x11 (DISPLAY) or wayland (WAYLAND_DISPLAY). if both are set,
it'll get it wrong. as i said - that's a more invasive fix.

eldbus: do not ignore numeric "0" in property_set

"0" value was considered to be error even if signature was numeric

@fix

evas-wayland-egl: Fix evas wayland_egl engine not loading due to missing symbol

Seems the newer wayland-egl code was missing the log domain variable
and thus would not load at time of dlsym due to missing symbol. This
fixes that problem by defining the variable.

@fix

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

ethumb client - fix access to pending struct that has been freed

th pending cancel may call callbacks... which can mess with the list
and thus the pending_add we have, sso do our work before calling this
(like set pending_call to NULL).