At Thu, 22 Aug 2013 15:06:38 +0200, Knut Petersen wrote:

[1 <text/plain; ISO-8859-1 (7bit)>] On 22.08.2013 10:53, Takashi Iwai wrote:

...

Actually this triggers the double free. It's freed in snd_rme96_free() via snd_card_free().

Yes. Now I see that.

...

I applied the patch with removal of this double vfree call.

Thanks.

Unfortunately there still is a problem with the code: vmalloc() and vfree() were used without inclusion of linux/vmalloc.h.

Indeed. Applied now. Thanks.

Takashi

cu, Knut

[2 0001-alsa-rme96-Add-missing-inclusion-of-linux-vmalloc.h.patch <text/x-patch (7bit)>]

...

From 2a3f98ff69308f3c2ee508ac0887e1f8bf493d64 Mon Sep 17 00:00:00 2001

From: Knut Petersen Knut_Petersen@t-online.de Date: Thu, 22 Aug 2013 14:36:16 +0200 Subject: [PATCH] alsa/rme96: Add missing inclusion of linux/vmalloc.h

Signed-off-by: Knut Petersen Knut_Petersen@t-online.de

sound/pci/rme96.c | 1 + 1 file changed, 1 insertion(+)

diff --git a/sound/pci/rme96.c b/sound/pci/rme96.c index 2acf565..cc6deca 100644 --- a/sound/pci/rme96.c +++ b/sound/pci/rme96.c @@ -28,6 +28,7 @@ #include <linux/interrupt.h> #include <linux/pci.h> #include <linux/module.h> +#include <linux/vmalloc.h>

#include <sound/core.h>

#include <sound/info.h>

1.8.1.4