eldbus: Fix dereference after null check
authorChristopher Michael <cp.michael@samsung.com>
Fri, 15 Mar 2019 16:14:18 +0000 (12:14 -0400)
committerYeongjong Lee <yj34.lee@samsung.com>
Tue, 2 Apr 2019 03:45:17 +0000 (12:45 +0900)
commitb9e770db5679fc97f36197c594b7a03e96b154ae
treeca4381eee581f3d7a4d527e9fe3f77eac23eaf0e
parenteb563a93c2b593d77624e66a0b9c83083b729057
eldbus: Fix dereference after null check

Summary:
Coverity reports we are passing NULL variable 'properties' to
eina_array_pop here which dereferences it. Wrap the 'end' block in an
if which checks for valid 'properties' variable

Fixes Coverity CID1399422

@fix

Depends on D8350

Reviewers: raster, cedric, zmike, bu5hm4n, stefan_schmidt

Reviewed By: cedric

Subscribers: #reviewers, #committers

Tags: #efl

Differential Revision: https://phab.enlightenment.org/D8351
src/lib/eldbus/eldbus_model_proxy.c