generic-library/libbsd
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/libbsd/libbsd.git synced 2025-01-09 11:17:37 +01:00
Code Issues Projects Releases Wiki Activity

funopen: Fix memory leak in funopen_close() when closefn is NULL

Browse Source 
We need to free the cookiewrap even when the closefn method is NULL.

Warned-by: gcc ASAN
This commit is contained in:
Guillem Jover 2020-08-16 21:58:04 +02:00
parent 3d6b6ead64
commit cfeafeabad
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/funopen.c
View File

@ -87,10 +87,10 @@ funopen_close(void *cookie)
struct funopen_cookie *cookiewrap = cookie; struct funopen_cookie *cookiewrap = cookie;
int rc; int rc;
if (cookiewrap->closefn == NULL) if (cookiewrap->closefn)
return 0; rc = cookiewrap->closefn(cookiewrap->orig_cookie);
else
rc = cookiewrap->closefn(cookiewrap->orig_cookie); rc = 0;
free(cookiewrap); free(cookiewrap);