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

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <201704270330.7qTYDGxV%fengguang.wu@intel.com>
Date:   Thu, 27 Apr 2017 03:19:14 +0800
From:   kbuild test robot <lkp@...el.com>
To:     Dan Williams <dan.j.williams@...el.com>
Cc:     kbuild-all@...org, rafael.j.wysocki@...el.com,
        Anush Seetharaman <anush.seetharaman@...el.com>,
        Tiffany Kasanicky <tiffany.j.kasanicky@...el.com>,
        Ryon Jensen <ryon.jensen@...el.com>,
        linux-kernel@...r.kernel.org, stable@...r.kernel.org,
        linux-acpi@...r.kernel.org,
        Kristin Jacque <kristin.jacque@...el.com>,
        Zhang Rui <rui.zhang@...el.com>
Subject: Re: [PATCH] acpi: fix acpi_get_table() leak / acpi-sysfs denial of
 service

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

Download attachment ".config.gz" of type "application/gzip" (25155 bytes)