mirror of
https://github.com/tiagovignatti/intel-gpu-tools.git
synced 2025-07-23 09:56:02 +00:00
Move free(cmd) to after last use of cmd in intel_gpu_top
Error: Use after free (CWE 416) Use after free of pointer 'cmd' in call to fprintf at line 496 of tools/intel_gpu_top.c in function 'main'. Previously freed at line 491 with free. [ This bug was found by the Parfait 0.4.2 bug checking tool. For more information see http://labs.oracle.com/projects/parfait/ ] Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
This commit is contained in:
parent
3d46ca8c4d
commit
e61b917134
@ -488,7 +488,6 @@ int main(int argc, char **argv)
|
||||
else if (child_pid == 0) {
|
||||
int res;
|
||||
res = system(cmd);
|
||||
free(cmd);
|
||||
if (res < 0)
|
||||
perror("running command");
|
||||
if (output) {
|
||||
@ -496,6 +495,7 @@ int main(int argc, char **argv)
|
||||
fprintf(output, "# %s exited with status %d\n", cmd, res);
|
||||
fflush(output);
|
||||
}
|
||||
free(cmd);
|
||||
exit(0);
|
||||
} else {
|
||||
free(cmd);
|
||||
|
Loading…
x
Reference in New Issue
Block a user