isa/sudo
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

copy_vector: plug memory leak in error path

Browse Source 
Only the array was being freed, not the contents.
GitHub issue #202.
This commit is contained in:
Todd C. Miller
2022-11-18 07:19:47 -07:00
parent 981cc2f8e3
commit 9c694e9a65
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/sudo_intercept.c
View File

@@ -96,7 +96,7 @@ copy_vector(char * const *src)
for (i = 0; i < len; i++) { for (i = 0; i < len; i++) {
copy[i] = sudo_mmap_strdup(src[i]); copy[i] = sudo_mmap_strdup(src[i]);
if (copy[i] == NULL) { if (copy[i] == NULL) {
sudo_mmap_free(copy); free_vector(copy);
debug_return_ptr(NULL); debug_return_ptr(NULL);
} }
} }