cedric pushed a commit to branch master. http://git.enlightenment.org/core/efl.git/commit/?id=88b2bfe4354fdbd975d11096108d6e486918f0c6
commit 88b2bfe4354fdbd975d11096108d6e486918f0c6 Author: Thiep Ha <thie...@gmail.com> Date: Tue Jul 28 01:08:11 2015 +0200 ecore_ipc: remove redundant code Summary: buf is always NULL (already freed and set to NULL). We don't need to add NULL checking and free it. Reviewers: raster, cedric, Hermet Subscribers: seoz, cedric Differential Revision: https://phab.enlightenment.org/D2783 Signed-off-by: Cedric BAIL <ced...@osg.samsung.com> --- src/lib/ecore_ipc/ecore_ipc.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/lib/ecore_ipc/ecore_ipc.c b/src/lib/ecore_ipc/ecore_ipc.c index 78f725a..4588199 100644 --- a/src/lib/ecore_ipc/ecore_ipc.c +++ b/src/lib/ecore_ipc/ecore_ipc.c @@ -1506,12 +1506,11 @@ _ecore_ipc_event_server_data(void *data EINA_UNUSED, int ev_type EINA_UNUSED, vo } svr->prev.i = msg; offset += (s + msg.size); - if ((svr->buf_size == offset) && ((svr->buf) || (buf))) + if ((svr->buf_size == offset) && (svr->buf)) { if (svr->buf) free(svr->buf); svr->buf = NULL; svr->buf_size = 0; - if (buf) free(buf); return ECORE_CALLBACK_CANCEL; } goto redo; --