Re: [PATCH][ACPI] Let's not gamble that a possible double free will never happen in asus_hotk_get_info()

2007-08-04 Thread Jesper Juhl
On 03/08/07, Len Brown <[EMAIL PROTECTED]> wrote: > On Saturday 28 July 2007 18:45, Jesper Juhl wrote: > > Signed-off-by: Jesper Juhl <[EMAIL PROTECTED]> > > --- > > > > drivers/acpi/asus_acpi.c | 1 + > > 1 files changed, 1 insertions(+), 0 deletions(-) > > > > diff --git

Re: [PATCH][ACPI] Let's not gamble that a possible double free will never happen in asus_hotk_get_info()

2007-08-04 Thread Jesper Juhl
On 03/08/07, Len Brown [EMAIL PROTECTED] wrote: On Saturday 28 July 2007 18:45, Jesper Juhl wrote: Signed-off-by: Jesper Juhl [EMAIL PROTECTED] --- drivers/acpi/asus_acpi.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/drivers/acpi/asus_acpi.c

[PATCH][ACPI] Let's not gamble that a possible double free will never happen in asus_hotk_get_info()

2007-07-28 Thread Jesper Juhl
Hi, The Coverity checker points out (CID: 1500) that we can in some cases end up doing a double free of 'model' in asus_hotk_get_info(). I'm not 100% sure it is right, but better safe than sorry, especially since this is so simple to turn into a non-issue - simply set 'model' to NULL after the

[PATCH][ACPI] Let's not gamble that a possible double free will never happen in asus_hotk_get_info()

2007-07-28 Thread Jesper Juhl
Hi, The Coverity checker points out (CID: 1500) that we can in some cases end up doing a double free of 'model' in asus_hotk_get_info(). I'm not 100% sure it is right, but better safe than sorry, especially since this is so simple to turn into a non-issue - simply set 'model' to NULL after the