summaryrefslogtreecommitdiff
path: root/src/lib/ecore_con
diff options
context:
space:
mode:
authorChristopher Michael <cp.michael@samsung.com>2019-03-14 07:22:54 -0400
committerChristopher Michael <cp.michael@samsung.com>2019-03-14 07:22:54 -0400
commite43b644168e3be77c0e4a29335c326f9237bb17d (patch)
tree43999b79f7deeddbe5370242503e0f98d7b0bb1c /src/lib/ecore_con
parent27095e859a26b0c938d894463a180e6a1f6e0568 (diff)
ecore_con: Fix dereferencing of null pointer
Summary: Coverity reports that 'svr' might be NULL here, so we should check it is valid before trying to use it. Fixes Coverity CID1396990 @fix Depends on D8320 Reviewers: raster, cedric, q66, zmike, bu5hm4n, stefan Reviewed By: cedric Subscribers: #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D8321
Diffstat (limited to 'src/lib/ecore_con')
-rw-r--r--src/lib/ecore_con/ecore_con_legacy.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/lib/ecore_con/ecore_con_legacy.c b/src/lib/ecore_con/ecore_con_legacy.c
index 5cbb39bf65..1956176604 100644
--- a/src/lib/ecore_con/ecore_con_legacy.c
+++ b/src/lib/ecore_con/ecore_con_legacy.c
@@ -293,7 +293,7 @@ _ecore_con_client_free(Ecore_Con_Client *cl)
293 efl_event_callback_array_del(inner_socket, _ecore_con_client_socket_ssl_cbs(), cl); 293 efl_event_callback_array_del(inner_socket, _ecore_con_client_socket_ssl_cbs(), cl);
294 294
295 parent = efl_parent_get(cl->socket); 295 parent = efl_parent_get(cl->socket);
296 if (parent && (parent != svr->server)) 296 if (parent && svr && (parent != svr->server))
297 efl_del(cl->socket); /* we own it */ 297 efl_del(cl->socket); /* we own it */
298 else 298 else
299 efl_unref(cl->socket); 299 efl_unref(cl->socket);