evas color copy neon asm - declare tmp as output not input for warning--

less warnings - use output not input for tmp var. much better.

eo base - fix warnings for debug logs to get format string types happy

gcc is very unhappy with these log prints - specifically on arm 32bit.
this fixes it so we can focus on real warnings/issues.

evas generic loader ps - fix warning about printf format mismatch type

it's right. types are uint but format exects unsigned long. fix.

evas wayland egl engine - fix warning about struct wl_resource not used

this struct is not actually used other than in this func ptr decl and
it's warning-land for gcc, so .. make it void * and do things like
pretty much all other gl extn func ptrs... be very generic

evas gl drm engine - fix warning about struct wl_resource not used

this struct is not actually used other than in this func ptr decl and
it's warning-land for gcc, so .. make it void * and do things like
pretty much all other gl extn func ptrs... be very generic

evas smooth scaler - fix warning where indenting is misleading

indeed the indenting is misleading. fix.

evas blit main - put mmx funcs in ifdefs as they are not used on arm

fixes warning

evas draw func get - make pixels as unused as it is in neon asm cases

also a little shitespace cleanup and correct ()ing of ops to clean
this function up a bit.

evas convert color - neon asm - fix warning by casting - no side effects

it's a warning one way or another so reduce noise with a harmless case
as passing in a pit ro a 32bit type is more restrictive than the ptr
it accepts (an 8bit type)

efl base class - fix warning about comparing differing sizes

yes - we compare a difference between 2 ptrs and an index which is a
uint. the safe thing here is to promote the unit to the ptrdiff_t
type. reality is we cant have more than 2^32 cb's on an object
anyway... so this should be ok.

efl io - fix using wrong type for return values pointed to

this potentially could be a bug on 32bit systems. this fixes that and
addresses the warning that pointed it out

ecore pipe - fix warning of comparing integers of differing sizes

elm code - silence possibly uninitialied compiler warnings

every time i build efl i see all these warnigns spew by about elm_code
and possibly uninited vars being used. yes - they're filled in via ptr
ref bya  func but gcc definitely seems to think they MIGHT not be
filled in in some cases... and either way tyhe cost of setting to 0 is
minimal and less noise is better so we see real warnings more
easily... :)

evas wl shm - fix warning with unused variable

evas: use the right structure to not over allocate memory.

Eina_Trash is designed for storing cached pointer without any memory
consumption. Please be careful with EFL memory consumption.

ecore-wl2: roundtrip during client disconnect

ensure all pending requests are processed by the compositor and fixes
session recovery destroy

@fix

elm_code: Fix syntax crash on trailing newline in multiline macro

@fix

ecore: Don't call _ecore_fd_valid() in release builds

Not sure _ecore_fd_valid() is all that useful anymore, as the
commit that introduced it said it would be removed "before release"
a long time ago - it's a debug assist that probably doesn't need
to be in release builds.

(I'm counting syscalls on rpi3 - still, calling this an optimization
seems like a bit of a stretch.)

Sysinfo gadget themes: Account for vertical theme.

efl_object: document when callbacks are called

ecore_wl2: Update compositor bind now that damage_buffer is released

We had to do some weird special case stuff when binding wl_compositor
because we implemented damage buffer before depending on a wayland
release that provided it.

Now our wayland dep is newer than damage buffer, so the test can be
more conventional.

Should have no functional change.

eolian: silence static analyzer null check (CID 1369018)

eolian: fix memory leak in doc tokens (CID 1367505)

vpath usage - simplify to bare minimum to make gustavo happy

since these are only local path resolves, the do and wait are
technically not needed. also remove any other tmp strings
and use the vpath string resolving feature to avoid
printfs/strjoins/cats etc. etc. as well.

also remember to remove old name string from evas shm code - it worked
for me. i guess i was lucky and it happened to be NULL thus free was
fine.

build: add mapping between ecore_file and ecore-file for pkgconfig files

elua.pc was depending on a ecore_file pc file it is named ecore-file. We have
mapping for similar cases. This one was just forgotten it seems.

Fixes T5075

src/utils: remove no longer used gitigonre and directories

When the XCB support was removed end of last year the .gitignore file left
behind. It was the only file still sitting in the src/utils tree so git removes
the whoile subtree now.

net_proxy_helper: fix typo in dependencies

Fixes:
error: cannot find the library 'lib/eina/libeina.la' or unhandled argument 'lib/eina/libeina.la'

Signed-off-by: Romain Naour <romain.naour@gmail.com>

eina_modinfo: add missing dependency on libeina.la

Fixes:
libtool:   error: cannot find the library 'lib/eina/libeina.la' or unhandled
argument 'lib/eina/libeina.la'

Signed-off-by: Romain Naour <romain.naour@gmail.com>

elementary-cxx: Make C++ test compile a not used archive so linking can be avoided

Fixes linking errors for some older combos of C++ compiler and linker.

T2838

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

Revert "vpath usage - simplify to bare minimum to make gustavo happy"

This reverts commit 2037474dc0fd2b360452f2a15abcbe533b57ca37.

This causes the wayland_shm engine to seg fault immediately at
startup when attempting to create shm buffers.

Please make sure when committing changes to the wayland_shm engine to
test on intel, exynos, and at least one platform without dmabuf
capabilities - or using the EVAS_WAYLAND_SHM_DISABLE_DMABUF env var
to disable dmabuf on intel or exynos.

Anyone without the time or hardware to fully test changes to wayland_shm
can submit a patch to phabricator and assign it to me so I can fully
test it before landing.

docs: avoid possible duplicates when figuring out all impls of a func

docs: list all existing implements of a method/property

docs: reverse inheritance hierarchy api in doctree

elm: examples: add new layout_model_connect binary to gitignore

docs: eina: add doc for new eina value type

docs: elm: document new efl UI image factory class

docs: efl mvc: document all new model view controller related eo files

docs: ecore: document newly added efl model composite classes

docs: efl filters: document all new data structures for efl filters

vpath usage - simplify to bare minimum to make gustavo happy

since these are only local path resolves, the do and wait are
technically not needed. also remove any other tmp strings and use the
vpath string resolving feature to avoid printfs/strjoins/cats etc.
etc. as well.

efl vpath - for runtime dir dont fall back to tmp but instead abort

if we can't create a runtimedir maybe its best to abort. also ensure
it is created as the effective user id.

efreetd - remove unused var

eina utils and vpath - use euid not uid if setuid for correct user

was using uid not euid for "if setuid use safe way to get location" -
ie the user that effectively owns the process, not nominally.

@fix

elm config - handle setuid apps correctly and dont create junk dirs in .

this fixes a bug where elm_cofnig would firstly use
~/.config/elementary which isnt what we have used before and so would
break older config. also it'd create junk if it was setuid. fix this.

@fix

evas fb dev env var - allow in setuid processes with sanitizing

this allows only /dev/fb[0-0] or /dev/fb/something where somthing does
not begin with a . - thus no way to break out of the fb subdir... so
it should be ok... this keeps setuid safety and allows this env var to
work now as intended in this situation.

efreetd cache - use eina home env get to centralize homedir fetching

this cleans up and improves a getenv that should really use eina

efreetd - dont use XDG_RUNTIME_DIR or EFREET_LOG env vars for setuid

if a setuid app needs efreet - it will not be able to contact the
users' efreetd and thus may spawn it's own... and thus we cant have
this now spawned efreetd using env vars inherited from the
unpriveleged etc. user, so ignore them.

efreet - use vpath to get XDG_RUNTIME_DIR so we have a single impl

having too many places that get XDG_RUNTIME_DIR makes it harder to
keep secure etc, so make it a single location for efreet too.

evas wayland shm engine - use vpath for getting xdg runtime dir

this should make the wayland shm engine safe to use from setuid
processes. it shouldnt depend on getenv of specific file paths if
setuid. vpatrh takes care of this in one central place

ecore buffer module - use vpath for xdg runtime for setuid safety

this should make ecore buffer module safe to use in setuid processes

evas cserve - remove unused if 0'd out code that is misleading my greps

looking for XDG_RUNTIME_DIR usage to lock down for setuid apps... this
msled me... remove it as it's unused.

elm widget item: add guide about when to use item_signal_emit

genlist/gengrid now uses cache for view objects of items and
re-associate edje objects to different obj items whenever needed.
problem here is if app user changes the state of view object
(by sending edje signal, elm_object_item_signal_emit()) and
forget to reset it in unrealized, the old edje object state will link to other
object item, so its application user responsibility to reset
the edje state to normal in "unrealized" callback.

Signed-off-by: Amitesh Singh <amitesh.sh@samsung.com>

evas: Fix invalid current state (invalid bool value)

newframe() resets some of the gl context properties, so match them
to our shadow copy in the gc state.

target_surface_set() also resets some of those properties but only
in our shadow copy, not in the actual GL context. As a consequence
we can't rely on setting a bool to 0 or 1 unless we also call the
equivalent GL function. Expand bitfields to 2 bits allows us to
set a bool to the invalid value of -1 (yes, that's not a real bool).

Also there is no need to reset the target surface to NULL during
newframe. It will be reset during target_surface_set.

This fixes some issues I encounter while working on GL filters.

elput - use vpath to get xdg runtime to also be setuid safe

this time around... use a / at the end of the (:run:) virtual dir
path to make it work.

doxygen: fix typos, cacologique statements in API reference doxygen of Ecore_Common

Summary: There are some typos and calogique statements in doxygen of Ecore_Common so I had fixed typos, cacologique statements.

Test Plan: doxygen revision

Reviewers: stefan, cedric, raster, Jaehyun_Cho

Subscribers: conr2d, jpeg

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

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

ecore-wl2: Support sending focus for wl_shell

wl_shell does not use a surface state for activation, so we still need
to send the focus in/out for that based on keyboard enter/leave.

@fix

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

ecore-wl2: Don't send focus in/out events based on keyboard enter/leave

We should not be sending focus events based on keyboard behaviour, but
rather send them according to xdg shell activate status. This makes
our focus behaviour more "standards" compliant.

@fix

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

Revert "elput - use vpath to get xdg runtime to also be setuid safe"

This reverts commit 24e34e19a1db84cdcb6241207cb99d14ca83c41b.

This broke keyboard input for the enlightenment wayland compositor,
please test elput changes on at least one of the drm backends,
preferably with enlightenment.

The wayland compositor is hard enough to keep stable due to breakage
from core changes only tested on X - but elput's main user is our
wayland compositor, was this tested anywhere?

Revert "elput: Minor formatting fix"

This reverts commit 95d93cb7208857cc89d79f3ffbbd7ee71ba1936f.

Formatting change on top of a totally untested coverity fix that
breaks input for enlightenment's wayland compositor.

 #TeamworkIsDeprecated

Give Batman it's own edc.

when resetting tex_target, set it to -1 so we force a reset

evas gl commonnon - context - expand fields to 2 bits for invalid vals

we use invalid values (-1 or 0xff or 0x3 etc. invalid vals to know to reset
some state). expand fields out in size a little bit to allow that to
work again.

elput: Minor formatting fix

NB: No functional changes, just some minor formatting adjustments

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

evas yuv convert - fix unused increments

this removes useless yp1 += and yop2 += as they are SET int he loop.
this fixes 1367510 and 1367511

evas mask filter - remove unused increment of msk as its set in-loop

msk is set inside the loop body, so msk += is pointless.

fix CID 1367512

eina debug monitor - dont use XDG_RUNTIME_DIR if setuid

avoids possible issues with debug monitor + setuid

ecore_con - only use XDG_RUNTIME_DIR if not setuid

this should address possible misuse of this env var in setuid apps.
keep using home and tmp to maintain socket "abi" (the filenames that
are used).

evas cserver client - remove dead if 0'd code

elput - use vpath to get xdg runtime to also be setuid safe

this fixes CID 1357168 in the case of setuid binaries as vpath handles
setuid cases.

evas gl generic - remove dead code else case

plane limits are checked at top of function so no need to check in the
middle again. it's dead code. fixes CID 1362729

evas gl generic - comment switch fallthrough as intended

this is intended. document as such.

efl vpath - be paranoid about setuid execution and dont use env vars

if setuid, do not trust env vars for any xdg stuff at all.

evas callbacks - add comment to note fallthrough is intended

swtich fallthrough oftenis not intended but in this case it is, so
documente it as such.

elm config - forbid bakcslash in profile name too

/ already is forbidden but \ might be valid on windows, so forbid it
too.

elm config - handle profile name of ".." as its obviously invalid

this forbids .., as a profile name in addition to the existing "stip /
from profile name".

eina utils - home and tmp environ - store statitcally and handle setuid

if setuod we dont want to trust HOME environ at all and get it from
passwd file... also we dont want to keep re-getting too... so store
statically as well as tmp.

this also kind of helps CID 1366469

edje_cc - fix potential divide by 0 in info output if file size is 0

this will realistically never happen in a sane case... but its
theoretically possible. fixes CID 1366923

eet data - fix checking return value of eina_value_pset like elsewhere

eina_value_pset() return is checked pretty much everywhere except
here. this addresses that inconsistency. this fixes CID 1367487

evas cache image - remove double unlock of image task locks.

this is indeed a bug and fixes coverity CID 1367492

eina share - fix spinlock release if magic check fails on share data

this fixes a potential double spinlock unrelease if magic check fails
on share data. this fixes CID 1367493

evas image obj - remove logically dead code

coverity is right - it's logically dead code. fix CID 1367774

evas image load - remove unreachable code

it's just printing a warning anyway and coverity CID 1368210 is right
as it says it's unreachable. fix.

evas image load - handle null module handle in case

this fixes CID 1368338

build: do not use a absolute system path for service files during distcheck

Our systemd service files are installed into an absolute system path by default
which simply does not work when doing a distcheck. Set the path differently for
the distcheck options.

elm code: check return value of elm_layout_theme_set

fixes CID 1366922

ecore_con proxy helper - fix null return handling for thread q wait

this fixes CID 1368376

Revert "evas_async_events: integrated Ecore_Pipe in evas_async_events"

This reverts commit ea0492e0c6912af704b14e51ea80715e2486ef2d.

this drops software engine rendering to like 5-10fps! or at least
visually about that... not tested enough at all!

ecore: add test to make sure that the timer reset during callback work as expected.

ecore: reset during timer callback execution doesn't make sense.

@fix
T3222

evas destructor - take lock arount post render list while freeing

this should fix CID 1369019 though really shouldnt be an issue...

eina freeq - fix possible seg with NULL freeq and pattern fill logic

this fixes handling of a NULL freeq handle - found by coverity.

fixes CID 1369021

evas cserve client - fix theoretical size less than 0 issue

theoretically size might be zero but it won't be... but make coverity
happy so it doesn't come back to triage

fix CID 1361430

evas textblock filter data set - avoid crash on freeing data binding

if balue is NULL the data binding is removed and freed, but we don't
return at that point but instead fall back to using/replacing the
databinding.

this fixes CID 1369022

ecore thread - up default 3 of thread workers to 4x cpu cures

this ups it from number of cpu cores to 4 times that (so if threads to
blocking i/o the spare cycles can be used up).

edje: fix cached edje modification issue

Summary:
_edje_part_***_set/get (for mouse_events, repeat_events, ignore_flags, mask_flags)
overwrite cached edje value. These behaviors affect all edje object added after
these changes, and result in not intended.

@fix

Reviewers: jpeg, cedric

Subscribers: akanad, woohyun

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

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

eet: read sould be failed if cipher_key exist but file not ciphered

Summary:
Currentely, if eet_read_cipher()'s cipher_key param is exist but file
not ciphered, function always read and return successful result.
But, this behavior can not check data integrity.

e.g.
1. App create config file using eet with encryption key.
2. Attacker replace config to malicious config (not encryted).
3. App can not notice if eet_read_cipher() read that successfully.

@fix

Test Plan:
$eet -e test.cfg my_config test.src 1
$eet -d test.cfg my_config decode_res.txt my_encrytion_key

(Currentely decode success, but should be failed)

Reviewers: woohyun, raster

Subscribers: id213sin, akanad, cedric, jpeg

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

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

elementary: set mirrored mode in a group_add()

Summary:
When a scroller just added and theme not changed,
scrollable interface's 'mirrored' property is always default.

elm_config_mirrored_set(EINA_TRUE) -> elm_scroller_add(),
 : No mirrored mode
elm_scroller_add() -> elm_config_mirrored_set(EINA_TRUE),
 : Mirrored mode

@fix

Test Plan:
1-1. elementary_test scroller
1-2. Turn on UI-Mirroring

2-1. elementary_test
2-2. Turn on UI-Mirroring
2-3. Click 'scroller' button

(Check scroller test's 'prev page' and 'next page' buttons)

Reviewers: cedric, woohyun, akanad

Subscribers: jpeg

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

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

elm_scroller: keep contents layout rel even if rtl is set

Summary:
Once rtl mode is set, elm_layout reverses its x-axis.
however, scroller uses elm_layout to contain both original content and
proxy image of it when loop mode is set.

In this situation, elm_layout should not reverse its x-axis when It is
used for container of proxy to support loop mode of scroller.

This commit will not apply rtl mode to contents layout (mirrored set)

Test Plan:
* Check that there is a proxy image over right edge of original content.

1. Run elementary_test
2. Open scroller
3. Switch UI_mirrored mode on
4. Test (Control with keyboard / mouse wheel control looks bad on rtl mod)
   a. go to the right edge with keyboard
   b. get over the edge

Reviewers: woohyun, z-wony, jpeg, cedric

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

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

elm_genlist: do not check scroll if show_item is NULL

Summary:
If show_item is NULL then does not need to check scroll.
But, sometimes show_item is NULL but check_scroll is true.
It was wrong and can makes a crash.

Signed-off-by: Minkyu Kang <mk7.kang@samsung.com>
Reviewers: cedric, Hermet, SanghyeonLee, raster

Subscribers: jpeg

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

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