Re: [PATCH] acpi: fix acpi_get_table() leak / acpi-sysfs denial of service

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Dan,

[auto build test ERROR on pm/linux-next]
[also build test ERROR on v4.11-rc8 next-20170426]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Dan-Williams/acpi-fix-acpi_get_table-leak-acpi-sysfs-denial-of-service/20170427-005809
base:   https://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm.git linux-next
config: x86_64-kexec (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

All errors (new ones prefixed by >>):

   drivers//acpi/sysfs.c: In function 'acpi_table_show':
>> drivers//acpi/sysfs.c:345:17: error: 'table' undeclared (first use in this function)
     acpi_put_table(table);
                    ^~~~~
   drivers//acpi/sysfs.c:345:17: note: each undeclared identifier is reported only once for each function it appears in

vim +/table +345 drivers//acpi/sysfs.c

   329				       struct bin_attribute *bin_attr, char *buf,
   330				       loff_t offset, size_t count)
   331	{
   332		struct acpi_table_attr *table_attr =
   333		    container_of(bin_attr, struct acpi_table_attr, attr);
   334		struct acpi_table_header *table_header = NULL;
   335		acpi_status status;
   336		ssize_t rc;
   337	
   338		status = acpi_get_table(table_attr->name, table_attr->instance,
   339					&table_header);
   340		if (ACPI_FAILURE(status))
   341			return -ENODEV;
   342	
   343		rc = memory_read_from_buffer(buf, count, &offset, table_header,
   344				table_header->length);
 > 345		acpi_put_table(table);
   346		return rc;
   347	}
   348	
   349	static int acpi_table_attr_init(struct kobject *tables_obj,
   350					struct acpi_table_attr *table_attr,
   351					struct acpi_table_header *table_header)
   352	{
   353		struct acpi_table_header *header = NULL;

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Attachment: .config.gz
Description: application/gzip


[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]