aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJaroslav Kysela <perex@perex.cz>2008-06-17 16:20:13 +0200
committerJaroslav Kysela <perex@perex.cz>2008-06-17 16:20:13 +0200
commit9f515b6898d0f2c1b9eb34ae9986794045f34e2b (patch)
tree85bb527513fbbcc62d8d3fa5d6762f906df3787f
parent6719292af5513da52aa876d0e7ac4367a0851845 (diff)
ALSA: emu10k1 - fix possible memory leak in memory allocation routines
The leak was introduced in "[ALSA] emu10k1 - simplify page allocation for synth" commit. Signed-off-by: Jaroslav Kysela <perex@perex.cz>
-rw-r--r--sound/pci/emu10k1/memory.c10
1 files changed, 8 insertions, 2 deletions
diff --git a/sound/pci/emu10k1/memory.c b/sound/pci/emu10k1/memory.c
index 42943b4fcb7..759e29f8947 100644
--- a/sound/pci/emu10k1/memory.c
+++ b/sound/pci/emu10k1/memory.c
@@ -464,11 +464,17 @@ static int synth_alloc_pages(struct snd_emu10k1 *emu, struct snd_emu10k1_memblk
/* first try to allocate from <4GB zone */
struct page *p = alloc_page(GFP_KERNEL | GFP_DMA32 |
__GFP_NOWARN);
- if (!p || (page_to_pfn(p) & ~(emu->dma_mask >> PAGE_SHIFT)))
+ if (!p || (page_to_pfn(p) & ~(emu->dma_mask >> PAGE_SHIFT))) {
/* try to allocate from <16MB zone */
- p = alloc_page(GFP_ATOMIC | GFP_DMA |
+ struct page *p1 =
+ alloc_page(GFP_ATOMIC | GFP_DMA |
__GFP_NORETRY | /* no OOM-killer */
__GFP_NOWARN);
+ /* free page outside dma_mask range */
+ if (p)
+ free_page((unsigned long)page_address(p));
+ p = p1;
+ }
if (!p) {
__synth_free_pages(emu, first_page, page - 1);
return -ENOMEM;