diff options
author | Andrew Donnellan <andrew.donnellan@au1.ibm.com> | 2016-09-19 16:41:32 +1000 |
---|---|---|
committer | Michael Ellerman <mpe@ellerman.id.au> | 2016-09-20 16:17:54 +1000 |
commit | 90ce35145cb622b3cd0007d50e1f6a5a97321235 (patch) | |
tree | 26550142d067142c54038f6f126ad7735174c8eb /arch/powerpc/platforms | |
parent | 11b7e154b132232535befe51c55db048069c8461 (diff) | |
download | op-kernel-dev-90ce35145cb622b3cd0007d50e1f6a5a97321235.zip op-kernel-dev-90ce35145cb622b3cd0007d50e1f6a5a97321235.tar.gz |
powerpc/pseries: fix memory leak in queue_hotplug_event() error path
If we fail to allocate work, we don't end up using hp_errlog_copy. Free it
in the error path.
Signed-off-by: Andrew Donnellan <andrew.donnellan@au1.ibm.com>
Reviewed-by: Nathan Fontenot <nfont@linux.vnet.ibm.com>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Diffstat (limited to 'arch/powerpc/platforms')
-rw-r--r-- | arch/powerpc/platforms/pseries/dlpar.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/arch/powerpc/platforms/pseries/dlpar.c b/arch/powerpc/platforms/pseries/dlpar.c index 2511ccf..423e450 100644 --- a/arch/powerpc/platforms/pseries/dlpar.c +++ b/arch/powerpc/platforms/pseries/dlpar.c @@ -413,6 +413,7 @@ void queue_hotplug_event(struct pseries_hp_errorlog *hp_errlog, queue_work(pseries_hp_wq, (struct work_struct *)work); } else { *rc = -ENOMEM; + kfree(hp_errlog_copy); complete(hotplug_done); } } |