From: Aishwarya Pant Date: Sun, 12 Mar 2017 15:39:49 +0000 (+0530) Subject: staging: bcm2835-audio: deallocate work when queue_work(...) fails X-Git-Url: https://git.stricted.de/?a=commitdiff_plain;h=e3173e5783ffdf3aa7bcc8ac0a150b17d83c1792;p=GitHub%2Fmoto-9609%2Fandroid_kernel_motorola_exynos9610.git staging: bcm2835-audio: deallocate work when queue_work(...) fails This patch de-allocates work when queue_work(..) fails in the bcm2835-audio work functions Signed-off-by: Aishwarya Pant Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/staging/vc04_services/bcm2835-audio/bcm2835-vchiq.c b/drivers/staging/vc04_services/bcm2835-audio/bcm2835-vchiq.c index 6feb1a6b07d9..af16d0f3e2b3 100644 --- a/drivers/staging/vc04_services/bcm2835-audio/bcm2835-vchiq.c +++ b/drivers/staging/vc04_services/bcm2835-audio/bcm2835-vchiq.c @@ -145,6 +145,7 @@ int bcm2835_audio_start(struct bcm2835_alsa_stream *alsa_stream) work->alsa_stream = alsa_stream; work->cmd = BCM2835_AUDIO_START; if (!queue_work(alsa_stream->my_wq, &work->my_work)) { + kfree(work); return -EBUSY; } } @@ -168,6 +169,7 @@ int bcm2835_audio_stop(struct bcm2835_alsa_stream *alsa_stream) work->alsa_stream = alsa_stream; work->cmd = BCM2835_AUDIO_STOP; if (!queue_work(alsa_stream->my_wq, &work->my_work)) { + kfree(work); return -EBUSY; } } @@ -194,6 +196,7 @@ int bcm2835_audio_write(struct bcm2835_alsa_stream *alsa_stream, work->src = src; work->count = count; if (!queue_work(alsa_stream->my_wq, &work->my_work)) { + kfree(work); return -EBUSY; } }