From 4db762ee6cdb48e6d05c3670a7a5cc4f82996f99 Mon Sep 17 00:00:00 2001 From: JunsuChoi Date: Thu, 7 Jun 2018 10:49:42 -0400 Subject: [PATCH] ecore_con : Fix that null pointer access for segmentation fault Summary: "cl->svr" was to be null on line 281. This problem occurred from 384f30c8ecc9. Replacing "cl->svr" with "svr" corrects the error while preserving the changes in 384f30c8ecc9. Test Plan: N/A Reviewers: #committers, cedric, Hermet, zmike Subscribers: bowonryu, zmike Tags: #efl Differential Revision: https://phab.enlightenment.org/D6255 --- src/lib/ecore_con/ecore_con_legacy.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/lib/ecore_con/ecore_con_legacy.c b/src/lib/ecore_con/ecore_con_legacy.c index 243a21356c..067cf712fb 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 != cl->svr->server)) + if (parent && (parent != svr->server)) efl_del(cl->socket); /* we own it */ else efl_unref(cl->socket);