linux-kernel - Re: [PATCH 2/2] acpi:ipmi: Make IPMI user handler const

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: <bfb820d6-11a5-d1ea-13be-bd46714c3c15@acm.org>
Date:   Thu, 9 Feb 2017 09:07:21 -0600
From:   Corey Minyard <minyard@....org>
To:     "Rafael J. Wysocki" <rjw@...ysocki.net>
Cc:     linux-kernel@...r.kernel.org,
        openipmi-developer@...ts.sourceforge.net,
        Corey Minyard <cminyard@...sta.com>,
        linux-acpi@...r.kernel.org, Zhao Yakui <yakui.zhao@...el.com>,
        Lv Zheng <lv.zheng@...el.com>
Subject: Re: [PATCH 2/2] acpi:ipmi: Make IPMI user handler const

On 02/09/2017 08:02 AM, Rafael J. Wysocki wrote:
> On Tuesday, January 10, 2017 07:40:13 AM minyard@....org wrote:
>> From: Corey Minyard <cminyard@...sta.com>
>>
>> ipmi_create_user() now takes the user handlers as const, make
>> it const in the ACPI IPMI code.
>>
>> Cc: linux-acpi@...r.kernel.org
>> Cc: Zhao Yakui <yakui.zhao@...el.com>
>> Cc: Lv Zheng <lv.zheng@...el.com>
>> Signed-off-by: Corey Minyard <cminyard@...sta.com>
> Please let me know if you want me to apply this.
>
> Otherwise, feel free to add my ACK to it and route it whatever way you think
> is appropriate.

Thanks, I'll add your ACK and apply it through the IPMI tree.

-corey

>> ---
>>   drivers/acpi/acpi_ipmi.c | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/acpi/acpi_ipmi.c b/drivers/acpi/acpi_ipmi.c
>> index f77956c..747c2ba 100644
>> --- a/drivers/acpi/acpi_ipmi.c
>> +++ b/drivers/acpi/acpi_ipmi.c
>> @@ -56,7 +56,7 @@ struct acpi_ipmi_device {
>>   struct ipmi_driver_data {
>>   	struct list_head ipmi_devices;
>>   	struct ipmi_smi_watcher bmc_events;
>> -	struct ipmi_user_hndl ipmi_hndlrs;
>> +	const struct ipmi_user_hndl ipmi_hndlrs;
>>   	struct mutex ipmi_lock;
>>   
>>   	/*
>>
> Thanks,
> Rafael
>