timer23
/
efl
forked from enlightenment/efl
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Fix memleaks reported by Coverity: Setting 'buf' to NULL (without 

freeing first) leaks the storage that buf Did point to (which could
have been from a realloc above).

NB: Fixes Coverity CID1039277, CID1039278

Signed-off-by: Chris Michael <cp.michael@samsung.com>
This commit is contained in:
Chris Michael 2013-07-08 12:48:35 +01:00
parent ca318c05fd
commit aedff2dc31
1 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/lib/ecore_ipc/ecore_ipc.c
View File

@ -1467,6 +1467,7 @@ _ecore_ipc_event_server_data(void *data EINA_UNUSED, int ev_type EINA_UNUSED, vo
Ecore_Ipc_Event_Server_Data *e2;
int max;
if (buf) free(buf);
buf = NULL;
max = svr->max_buf_size;
if ((max < 0) || (msg.size <= max))
@ -1504,6 +1505,7 @@ _ecore_ipc_event_server_data(void *data EINA_UNUSED, int ev_type EINA_UNUSED, vo
free(svr->buf);
svr->buf = NULL;
svr->buf_size = 0;
if (buf) free(buf);
return ECORE_CALLBACK_CANCEL;
}
goto redo;
@ -1513,6 +1515,7 @@ _ecore_ipc_event_server_data(void *data EINA_UNUSED, int ev_type EINA_UNUSED, vo
else
{
scroll:
if (buf) free(buf);
buf = malloc(svr->buf_size - offset);
if (!buf)
{