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;

-- 


Reply via email to