Commit 8ea1bee4 authored by Dmitry Safonov's avatar Dmitry Safonov Committed by Pavel Emelyanov

cr-dump: don't forget to free mme.vmas

...
cr-dump.c:515:9: warning: Potential leak of memory pointed to by 'mme.vmas'
        return ret;
               ^~~
Signed-off-by: 's avatarDmitry Safonov <dsafonov@virtuozzo.com>
Signed-off-by: 's avatarPavel Emelyanov <xemul@virtuozzo.com>
parent f7e8eae4
...@@ -451,7 +451,7 @@ static int dump_task_mm(pid_t pid, const struct proc_pid_stat *stat, ...@@ -451,7 +451,7 @@ static int dump_task_mm(pid_t pid, const struct proc_pid_stat *stat,
mme.n_vmas = vma_area_list->nr; mme.n_vmas = vma_area_list->nr;
mme.vmas = xmalloc(mme.n_vmas * sizeof(VmaEntry *)); mme.vmas = xmalloc(mme.n_vmas * sizeof(VmaEntry *));
if (!mme.vmas) if (!mme.vmas)
goto err; return -1;
list_for_each_entry(vma_area, &vma_area_list->h, list) { list_for_each_entry(vma_area, &vma_area_list->h, list) {
VmaEntry *vma = vma_area->e; VmaEntry *vma = vma_area->e;
...@@ -512,6 +512,7 @@ static int dump_task_mm(pid_t pid, const struct proc_pid_stat *stat, ...@@ -512,6 +512,7 @@ static int dump_task_mm(pid_t pid, const struct proc_pid_stat *stat,
xfree(mme.mm_saved_auxv); xfree(mme.mm_saved_auxv);
free_aios(&mme); free_aios(&mme);
err: err:
xfree(mme.vmas);
return ret; return ret;
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment