linux-kernel - Re: [PATCH 1/1] drivers: staging: rtl8192u: ieee80211: ieee80211_softmac.c: auth parse error code byte order fix

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <5563406A.3010202@windriver.com>
Date:	Mon, 25 May 2015 11:31:54 -0400
From:	Paul Gortmaker <paul.gortmaker@...driver.com>
To:	tolga ceylan <tolga.ceylan@...il.com>,
	Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
	Cristina Opriceana <cristina.opriceana@...il.com>,
	Haneen Mohammed <hamohammed.sa@...il.com>,
	Greg Donald <gdonald@...il.com>,
	Aya Mahfouz <mahfouz.saif.elyazal@...il.com>,
	Benoit Taine <benoit.taine@...6.fr>,
	<devel@...verdev.osuosl.org>, <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 1/1] drivers: staging: rtl8192u: ieee80211: ieee80211_softmac.c:
 auth parse error code byte order fix

On 15-05-23 10:53 PM, tolga ceylan wrote:
> On 05/12/2015 08:55 PM, Tolga Ceylan wrote:
>> Error code returned from auth_parse() should in cpu byte order.
>>
>> Signed-off-by: Tolga Ceylan <tolga.ceylan@...il.com>
>> ---
>>   drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c b/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c
>> index d2e8b12..0cf0e89 100644
>> --- a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c
>> +++ b/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c
>> @@ -1558,7 +1558,7 @@ static inline u16 auth_parse(struct sk_buff *skb, u8 **challenge, int *chlen)
>>   		}
>>   	}
>>
>> -	return cpu_to_le16(a->status);
>> +	return le16_to_cpu(a->status);
>>
>>   }
>>
>>
> 
> I haven't gotten a response for this patch. Just checking back.

In the future, you can help speed up the review process by helping
the reviewer/maintainer.  How?  In your commit log.  Describe the 
end user symptoms and how the problem was found, why it happens
and what is the best fix for the problem (assuming it isn't trivially
obvious).  So, instead of just translating the C change into words
for a commit log like you had above, you could have instead had...

	When trying to use an RL8192 with auth mode foo and module
	parameter blah blah, the driver fails to connect, only showing
	blah blah in the logs.

	This happens because auth_parse blah blah and returns blah
	which is incorrectly interpreted by foo_bar() since it is in
	the wrong endian as blah is in native byte order and ...

	We fix it by noting the error value should be ... since ...


Also, minor nit ; if you are going to put a path fragment in the subject,
then don't replace all the / with ": " --- i.e. use
	
 [PATCH] staging: rtl8192u/ieee80211/ieee80211_softmac.c: auth parse error code byte order fix

or 

 [PATCH] staging/rtl8192u: ieee80211_softmac.c - auth parse error code byte order fix

Paul.
--

> 
> Regards,
> Tolga Ceylan
> 
--
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/