lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Sat, 26 Aug 2006 21:43:36 -0400 From: Len Brown <len.brown@...el.com> To: "Om Narasimhan" <om.turyx@...il.com> Cc: linux-kernel@...r.kernel.org, kernel-janitors@...ts.osdl.org Subject: Re: memory leak fix in acpi_memhotplug.c, kmalloc to kzalloc conversion. On Saturday 26 August 2006 20:21, Om Narasimhan wrote: > Hi, > This patch fixes one memory leak in drivers/acpi/acpi_memhotplug.c > Replaces all kmalloc() calls succeeded by memset() to kzalloc() calls. > Applies cleanly to 2.6.18-rc4. Compile tested. Compile testing is more effective is you use a .config that builds each of the source files changed. > Patch starts here. > > > drivers/acpi/ac.c | 4 +--- > drivers/acpi/acpi_memhotplug.c | 14 +++++--------- > drivers/acpi/asus_acpi.c | 3 +-- > drivers/acpi/battery.c | 13 +++---------- > drivers/acpi/bus.c | 3 +-- > drivers/acpi/button.c | 4 +--- > drivers/acpi/container.c | 4 +--- > drivers/acpi/ec.c | 20 +++++--------------- > drivers/acpi/fan.c | 3 +-- > drivers/acpi/i2c_ec.c | 8 ++------ > drivers/acpi/osl.c | 2 +- > drivers/acpi/pci_bind.c | 21 +++++---------------- > drivers/acpi/pci_irq.c | 16 +++------------- > drivers/acpi/pci_link.c | 6 ++---- > drivers/acpi/pci_root.c | 3 +-- > drivers/acpi/power.c | 4 +--- > drivers/acpi/processor_core.c | 4 +--- > drivers/acpi/sbs.c | 4 +--- > drivers/acpi/thermal.c | 13 +++---------- > drivers/acpi/utils.c | 4 +--- > 20 files changed, 40 insertions(+), 113 deletions(-) Please direct patches touching drivers/acpi to me, cc linux-acpi@...r.kernel.org > diff --git a/drivers/acpi/ac.c b/drivers/acpi/ac.c > index 96309b9..72738a5 100644 > --- a/drivers/acpi/ac.c > +++ b/drivers/acpi/ac.c > @@ -221,11 +221,9 @@ static int acpi_ac_add(struct acpi_devic > if (!device) > return -EINVAL; > > - ac = kmalloc(sizeof(struct acpi_ac), GFP_KERNEL); > + ac = kzalloc(sizeof(struct acpi_ac), GFP_KERNEL); > if (!ac) > return -ENOMEM; > - memset(ac, 0, sizeof(struct acpi_ac)); > - > ac->device = device; > strcpy(acpi_device_name(device), ACPI_AC_DEVICE_NAME); > strcpy(acpi_device_class(device), ACPI_AC_CLASS); > diff --git a/drivers/acpi/acpi_memhotplug.c b/drivers/acpi/acpi_memhotplug.c > index b0d4b14..eb8a5da 100644 > --- a/drivers/acpi/acpi_memhotplug.c > +++ b/drivers/acpi/acpi_memhotplug.c > @@ -297,6 +297,7 @@ static int acpi_memory_disable_device(st > * Ask the VM to offline this memory range. > * Note: Assume that this function returns zero on success > */ > + struct acpi_memory_info *info, *n; What tree is this patch against? This line is already present (above the comment) in 2.6.18-rc4. thanks, -Len - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists