eldbus: check message serial before using

Summary:
Whatever the dbus_connection_send_with_reply returns, the serial value
should be checked, because if the seral value is invalid a process could be aborted.

There is backtrace as below.

The dbus_connection_send_with_reply could return TRUE
even though it has a problem. Please refer to following comment:

   /* Refuse to send fds on a connection that cannot handle
      them. Unfortunately we cannot return a proper error here, so
      the best we can do is return TRUE but leave *pending_return
      as NULL. */

Test Plan:
There is not a exact reproduce step. If the Tizen login manager is relaunched
repeatedly, then the dbus and other service processes are relaunched.
If a service process tries to use dbus when the dbus has problem as above,
then it could be possilbe to get above backtrace.

Reviewers: raster, zehortigoza

Subscribers: cedric, jpeg

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

@fix

efl_ui_win: add EFL_UI_WIN_SOCKET_IMAGE type check

    elm_win_socket_listen api only work for EFL_UI_WIN_SOCKET_IMAGE type window.
    So it is better to add type check code.

evas - handle devices that on deletion delete other devices on shutdown

this fixes a whole bunch of warnings that just recently turned up...

elocation - fix return of status, address, position to actually ret data

the data was never returned! fix fix fix.

found by PVS studio

@fix

ecore x - convert to using unsed instead of dummy state,ents after ret

found by PVS studio

elm code - handle alloc failure for line split

this culd optentially use large offsets before accessing the ptr thus
it may not segv inthe first page but actually work if positionoffsets
are large enouh, so handle this case explicitly.

@fix

found by PVS studio

elm code - handle alloc failure for line split

this culd optentially use large offsets before accessing the ptr thus
it may not segv inthe first page but actually work if positionoffsets
are large enouh, so handle this case explicitly.

@fix

found by PVS studio

evas mem - remove redundenty unused evas mem absraction

was never used as was not practical to do so so it effectively does
nothing so remove it to reduce analyser warnings

found by PVS studio

efreet icon cache creation binary - remove redundent check

found by PVS studio

elm theme - remove redundent check

found by PVS studio

elm config tool - fix redundent check

found by PVS studio

evas textblock - remove redundant check

found by PVS studio

epp - cpp - fix buffer end/null checks given static buffer paths

this is an ncient bug that's never been triggered... but it's there.
fix and compare to null buffer.

found by PVS studio

@fix

epp - remove redundent check for char

makes static analyisers happy... not a bug

found by PVS studio

evas cache2 - remove excess file and key checks - only check file

no need for key as no key is handled and no file is handled too. cleaner
code.

found by PVS studio

elm gesture layer - use fabs not abs when actually comparing floats

2 abs uses should have been fabs as they compe a src float with
anot her float.

found by PVS studio

@fix

eina debug - set length after freeing/iterating over the keys

found by PVS studio

elm calendar - fix gmtime call to use an actual value

e85c92662bfec6a88bb7d2d091682daa429acba0 added this bug. before it was
always set to 259200 (first sunday after epoch).

found by PVS studio

ecore_drm2 - fix startup if edid blob is null.. assume all 0's

this causes a crash when no edid is available. survive and dont crash.
this causes e wayland to not work at all on my baytrail laptop.

@fix

eldbus lower error print levesl from ER to WRN in get all props

this is generally unnecessary noise as the majority of the time i see
this it's because the message was canceled by the caller... thats not
an error though then... it's intended. less debug noise to sift through.

ecore_evas_drm: Only check for gpu time offset if env var set

Since most systems shouldn't need this, and it's possible
that it's causing trouble for some systems, make it off by
default.

Fix T5806

tests - make ecore con test suite reliable and work

basically ecore_con's test suite was broken. ipv6 checks were broken.
you now need to set an env var to get ipv6 tested. using eocre_con to
detetc it didnt init eina or ecore_con and if you do init these then
the other tests that check return values from these fail... so make it
an env var if you want to test ipv6. now it's reliable and works!

ector software - fix min/max int range for fixedpoint math

min value bitshifts are negative and thus not portable. just tax max
(as its 1 more than min it if we do -max) as the limit as its within
range. this should fix it

@fix

found by PVS studio

evas render - remove obj null check in proxy clip as obj not null

obj is never null. so this check is pointless. not a bug though.

found by PVS studio

evas object textblock - fix append if cur node is null

then it will appeand totally as it should be. small buglet found. fixed

found by PVS studio

@fix

evas gl gnenric engine - dont check w and h ptrs - will never be null

analyser unhappiness.

found by PVS studio

embryo_cc - remove pointless test for null byte and space

space wins. the nul byte test is pointless. not a bug but pointless code

found by PVS studio

ecore-x - remove useless extra assignments of value to itself

usless code that does nothing - not a bug though

found by PVS studio

edje_cc - remove pointless free of null ptr

not a bug but not useful code either freeing what is already null

found by PVS studio

efl ui text - remove useless else + free

string is null so no need to free. not a bug but cleaner and analysers
dont like it.

found by PVS studio

edje entry - remove extra else and free as sdtring is null there

string is null so no need to free. not a bug but cleaner and analysers
dont like it.

found by PVS studio

eina simple xml parser - remove pointless memcmp of 0 bytes

no point - all we are doing is having a final fallback of any tag that
starts with ! that isnt a special one like !DOCTYPE, !-- comment and
![CDATA stuff... analysers dont like these pointless calls.

found by PVS studio

els box - remove rudundant double check

not a bug but checking something twice in a row isn't too useful

found by PVS studio

elm code - remove unneded extra check from copy and pasta it seems

extra check already done in if above - no need. looks like copy &
paste of the checks from above thus why it's there, but not needed.
not a bug, byt analysers dont like it so remove

found by PVS studio

emotion gst1 module - remove extra unneded if's

not a bug, but don't need these

found by PVS studio

eeze_disk - remove redundant if check already done

removes redundant fi check. not a bug but analysers dont like it and
it is redundant

found by PVS studio

efl net - fix ipv6 getsockname to use the proper struct to do it

      struct sockaddr_storage *addr;

AS the actual storage ( a ptr to store the whole data). kind of...
wrong. it should be:

      struct sockaddr_storage addr;

see examples of this kind of usage of getsockname:

http://www.masterraghu.com/subjects/np/introduction/unix_network_programming_v1.3/ch04lev1sec10.html

found by PVS studio

@fix

eeze disk - remove uneeded extra check for test var

test for non null already done above, so test must obviously be null
here so no need to check. analysers dont like this but it's not a bug.
make them happy thought

found by PVS studio

evas psd loader - remove unneeded extra check in if

one if condition is always true by virtual of previous if statements
and drop-through so can remove. not actually any bug but analysers
don't like it

found by PVS studio

eina debug - if calloc for session struct failed - return NULL

makes analysers happy but really doesnt change anything - moves the
abort/crash/exit somewhere else.

found by PVS studio

edje_cc - make list free simpler to make analysers happy

analysers like PVS studio dont know the eina_list_free returning NULL
design pattern for consistency and so dislike the var being set twice
in a row. this will make them happy without any skin off our backs.
not a bug at all.

found by PVS studio

evas git loader - move findo reset inside else as only that changes it

only the else changes finfo so reset inside there. not really any bug
at all byt style-wise a bit better and analysers don't like it

found by PVS studio

elm list - remove harmless extra null set to keep analysers happy

this was totally harmless and not an actual bug but it keeps analysers
happy.

found by PVS studio

elm dayselector - remove double set of content var

this looks like a leftover from legacy to eo porting. it was harmless
but analysis hates it

found by PVS studio

elm code - remove double set of textlen

analysis doesnt like it, but not a bug

found by PVS studio

edje edit - remove pointless setting attr_mount to 0

static analysers dont like it bug not a bug at all

found by PVS studio

evas pmap loader - remove pointless extra assign of end of string to 0

not an actual bug - just analysis impacting

found by PVS studio

eldbus parser - remove duplicate tag handler thats already handled

confusing and dead code as it's already handled. not an actual bug but
analysis impacting

found by PVS studio

evas events - remove set of list to null then back to value - confusing

eina_list_free always returns NULL. it's meant as a design pattern to
follow like the append/prepend that ensures a freed list is NULL, but
it can be confusing to analysers so remove the assing. not a bug
actually.

found by PVS studio

eina evlog debugging - when freeing debug buf with mmap 0 size after

munmap would not munmap the memory with  a size of 0... so zero the
size after the munmap.

found by PVS studio

@fix

evas obj smart - remove pointless if

s is already checked for null... so no need to check again

found by PVS studio

@fix

edje edit - when adding size class set max width and height to -1

since we're not using size class yet.. we hvent had to debug this...
but it is a bug.

fond by PVS studio

@fix

efl net ssl constructor - remove double set of same var

all struct vars are set... one 2x. not a bug but pointless code in the
src

found by PVS studio

evas bmp loader - fix warnings about same type handled twice

so a type we handle earlir inan if we re-handle as invalid later. this
wouldnt lead to a crash or bugs as the if's would ned to be evaluated
in order normally, but it's good to get it right.

found by PVS studio

etc to rgba conversion - dont read stack garbage into the est imgs

surprising this wasn't caught when testing fallbacks. etc2 is used
hyper-rarely but still... basically it'd skip pixels and read stack
junk (which really would not crash but we'd have junk in image data).
or should.

found by PVS studio

@fix

ephysics - fix stacking comparsion correctly if 1 smaller than 1

this fixes a pretty trivial comparison issue..

found by PVS studio

@fix

evas legacy event mask - cast to unit64_t before bitshift

this fixes a bit wraparound in the shift as the 1 is an int (32bit)
type that then gets shifted .. then after that cast to 64bit.

found by PVS studio

@fix

eina inarray accessor - use right type in parameter

this actually wasn't a bug that would cause a crash. cloning an array
access would fail as the magic check would find its an accessor not an
array. indeed a bug... but we never used this anywhere i can find.

this was cast to the correct func ptr callabck in the accessor struct
as the clone method though.. thus everyhting was happy with it
seemingly.

found by PVS studio

@fix

elput: Fix synthetic relative motion events based on abs events

libinput won't actually give us this info, and logs an error instead.
We can't synthesize based on pointer position due to pointer
warping.

So we must track abs reports and use previous abs reports to
synthesize rel events.

ecore_evas_drm: Calculate gpu blank time offset once at startup

So vmware's graphics driver reports the MONOTONIC drm cap, yet uses
CLOCK_REALTIME instead.  This leaves us with a gigantic offset between the
gpu timestamp and the times ecore_time_get() gets from CLOCK_MONOTONIC.

Since ticking screws directly with loop time this results in some long
distance clock jumping.

This commit fixes drm/gl_drm operation under vmware.

ecore_drm2: Add a fallback method for vblank waiting

We can't depend on vblank waits being implemented by the driver, but we
can count on page flips functioning, so add a fallback that does a page
flip and waits for it.

ecore-wl2: implement support for aux hints

this is a direct copy of a feature from tizen git with no modifications other
than what was required for compilation and functionality

https://review.tizen.org/git/?p=platform/upstream/efl.git;a=commitdiff_plain;h=01e72b7e3484ece4b589f95315990ba2c366c231
https://review.tizen.org/git/?p=platform/upstream/efl.git;a=commitdiff;h=670d84b579f248ae0e3df48e9953fe8128da9468

fix T5780

@feature

Revert "evas: Fix use of invalid EO object"

This reverts commit 9b1e785f8ebba618016dd217aa18835b4fb675b7.

this breaks devices with vt switching under drm

elput: fix seat cleanup

ensure seat deletion does not conflict with elput shutdown

@fix

elput: unref devices in post event cb

@fix

elm_table: do not propagate max size

evas table does not set max size, propagating this triggers unnecessary
callbacks

@fix

elm_box: use correct aspect ratio for HORIZONTAL layout

ratio should be flipped in this case

@fix

elm_box: use homogeneous layout if only one child exists

this should be a small speedup for a corner case of box use

elm_win: do not implement aspect_get()

this breaks aspect hints.

@fix

efl-wl: escape command args from test

efl-wl: hide popup surface before unsetting attrs during destructor

ensure focus remains with the popup parent

efl-wl: simplify refocus on popup hide

efl-wl: use correct window when requesting x11 clipboard selection

Ecore_Con: Fix wrong comparison.

Otherwise it will always evaluate to false.

Ecore_Conn: Set EOS only if the connection is not closed.#algo.

This check prevents that EOS is set on a closed connection.

Fixes T5180

Ecore_Con: Fix CLOEXEC documentation.

The socket attributes are inherited by the child.

ecore_drm2: Allow picking a sequence number for blanktime_get

This lets us do a blocking wait for a vsync.  Something we should try to
do as infrequently as possible, but in some cases we need it one time at
startup to catch graphics driver bugs.

Elm entry: fix call to set text via entry_set

Accidentally called the super here, so did not get routed to
'_elm_entry_text_set'.

Fixes T5803.

elm test: img.zoomable: allow top rect to pass events

eo-cxx: Add overload for Eina_Bool inout handling interoperability

Add convert_inout_impl overload to handle bool/Eina_Bool conversion in inout direction.

Ecore_Conn: Enable CLOEXEC by default.

This flag should be enabled by default in order to avoid socket leaks.

Ecore_IPC: Preserve Ecore_Con legacy behaviour.

This patch sets some Ecore_Con flags that were missing after
the EO migration. These flags must be set in order
maintain the Ecore_IPC behaviour before Ecore_Con EO was implemented.

Fixes T5722

efl.ui.image.zoomable: Add missing edje.group_size_min/max_get

This supresses the warnings when photocam is used as
an external edje object.

evas: Always call show/hide intercept

Ref T5370

elm_widget: change to @inout

according to q66 this is correcter than ptr(..).

Anyhow this still leads to a bug in cxx that crashes compilation in cxx
examples, people are notified!

elm_widget: fix warning

ecore_events: inarray should be flushed before return

@fix

ecore_evas_wayland: fix bug ecore_evas cannot update the evas when rotation

    usecase:
    show -> rotation -> hide -> show
    ecore_evas_wayland didn't check the rotation.
    when ee is landscapemode, it cannot update the right area of evas.

elm_code: Fix missing middle button selection paste

Resolves task T5520
@fix

elput: Close fds when asked to

Our close callback tells logind we're done with a device, but it should
also actually close the fd it's passed, or we end up leaking piles of
fds on VC switch.

see weston commit 8f5acc2f3a29c3831af4ddd6bed57f703c98dc77
and subsequent regression in commit 72dea06d7952e3ce8dd8057f7106186da4fa2678
and pending fix in https://patchwork.freedesktop.org/patch/168992/

edje_cc: fix wrong comment about lazEDC norequired.

Summary: - norequired; stands for required: 0

Reviewers: conr2d, jpeg

Reviewed By: jpeg

Subscribers: cedric

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

elm: Move calls to efl_destructor(efl_super()) to the end

When destroying any object, its parent class destructor should
be called after the subclass destructor has been called. Only
some extremely limited work may be done after the super call.

This commit makes sure that all efl_destructor() overrides in
elementary are doing operations in the right order.

Also, remove a return void.

elm_widget: only emit events when there is really a changed manager

A manager change to NULL is to be used as "this object is not registered
anymore" call, which will happen quite regular during shutdown.

elm_widget: call the events after eval outself with new manager/parent

otherwise we might end up with a node_get error, since the widget where
the event is called on is still registered in the wrong manager

elm_widget: only set focus.manager if we are really registering

otherwise we might set focus.manager even if we are not registered

elm_widget: give feedback if registration was successfull or not

with this we can whipe out the focus.manager field in elm.widget so for
the case that something goes wrong we only get the error message where
actually something went wrong, and not the whole bunch of follow ups
where the code assumes its registered but it isnt.

elput: Fix multiple open/close of drm devices

When I added the code to probe drm devices to ensure they're
modeset capable (ref 414d406b3b442216543cdaef112787696ae09898)
I didn't realize elput didn't allow us to open and close more than
one drm device at startup without blowing up libinput.

This is a somewhat dirty hack to rough that in.

The problem is that open/close the device during startup will
result in an async "gone" callback from logind, which then kicks
off an input shutdown.  We need to try harder to only do that
shutdown when it makes sense.

ecore-wl2: handle null offer drags more effectively

this is valid and refers to an offer with no types; a leave event
with no enter is a protocol error, however

fix T5770

@fix

Revert "ecore wl2 - dnd - handle NULL drags... shouldnt happen but does"

This reverts commit 85e585846600742a7f395751816f348b4152269f.

please do not randomly add null checks for the purpose of quickly closing tickets.

ecore wl2 - dnd - handle NULL drags... shouldnt happen but does

this should fix T5770

@fix