diff options
author | Robert Krakora <rob.krakora@messagenetsystems.com> | 2009-05-28 10:16:19 -0400 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2009-06-16 17:21:16 -0400 |
commit | 75c74d1c78ec3b713a986e0efc645ed558384cdf (patch) | |
tree | c37df54bc361a2a9122f0ba853a9bc92197fca47 | |
parent | f41b6961566b2cd038ad91568882f8fc1feac160 (diff) |
V4L/DVB (11896): em28xx: Fix for Slow Memory Leak
Test Code: (Provided by Douglas)
v4l-dvb/v4l2-apps/test/stress-buffer.c
The audio DMA area was never being freed and would slowly leak over
time as the v4l device was opened and closed by an application.
Thanks again to Douglas for generating the test code to help locate
memory leaks!!!
Signed-off-by: Robert Krakora <rob.krakora@messagenetsystems.com>
Signed-off-by: Douglas Schilling Landgraf <dougsland@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
-rw-r--r-- | drivers/media/video/em28xx/em28xx-audio.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/drivers/media/video/em28xx/em28xx-audio.c b/drivers/media/video/em28xx/em28xx-audio.c index 0131322475bf..7bd8a70f0a0b 100644 --- a/drivers/media/video/em28xx/em28xx-audio.c +++ b/drivers/media/video/em28xx/em28xx-audio.c | |||
@@ -339,6 +339,11 @@ static int snd_em28xx_pcm_close(struct snd_pcm_substream *substream) | |||
339 | mutex_lock(&dev->lock); | 339 | mutex_lock(&dev->lock); |
340 | dev->adev.users--; | 340 | dev->adev.users--; |
341 | em28xx_audio_analog_set(dev); | 341 | em28xx_audio_analog_set(dev); |
342 | if (substream->runtime->dma_area) { | ||
343 | dprintk("freeing\n"); | ||
344 | vfree(substream->runtime->dma_area); | ||
345 | substream->runtime->dma_area = NULL; | ||
346 | } | ||
342 | mutex_unlock(&dev->lock); | 347 | mutex_unlock(&dev->lock); |
343 | 348 | ||
344 | return 0; | 349 | return 0; |