diff options
| author | Christopher Michael <cp.michael@samsung.com> | 2019-03-14 07:22:54 -0400 |
|---|---|---|
| committer | Christopher Michael <cp.michael@samsung.com> | 2019-03-14 07:22:54 -0400 |
| commit | e43b644168e3be77c0e4a29335c326f9237bb17d (patch) | |
| tree | 43999b79f7deeddbe5370242503e0f98d7b0bb1c | |
| parent | 27095e859a26b0c938d894463a180e6a1f6e0568 (diff) | |
| download | efl-e43b644168e3be77c0e4a29335c326f9237bb17d.tar.gz | |
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
| -rw-r--r-- | src/lib/ecore_con/ecore_con_legacy.c | 2 |
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) efl_event_callback_array_del(inner_socket, _ecore_con_client_socket_ssl_cbs(), cl); parent = efl_parent_get(cl->socket); - if (parent && (parent != svr->server)) + if (parent && svr && (parent != svr->server)) efl_del(cl->socket); /* we own it */ else efl_unref(cl->socket); |