On 09/19/2016 01:41 AM, Andrew Donnellan wrote:
> 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.donnel...@au1.ibm.com>
Reviewed-by: Nathan Fontenot <nf...@linux.vnet.ibm.com>
>
> ---
>
> Found by Coverity Scan. Compile tested only.
> ---
> arch/powerpc/platforms/pseries/dlpar.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/powerpc/platforms/pseries/dlpar.c
> b/arch/powerpc/platforms/pseries/dlpar.c
> index 4748124..6a99e72 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);
> }
> }
>
