netdev - Re: [RFC] airo.c: Description and function is not the same

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]

Date:	Tue, 23 Jan 2007 10:11:34 -0500
From:	Dan Williams <dcbw@...hat.com>
To:	Richard Knutsson <ricknu-0@...dent.ltu.se>
Cc:	linville@...driver.com, netdev@...r.kernel.org
Subject: Re: [RFC] airo.c: Description and function is not the same

On Sun, 2007-01-21 at 13:01 +0100, Richard Knutsson wrote:
> Hello
> 
> In the tour of converting local definitions of boolean-type/values, I 
> ran into airo.c's description of ex decapsulate():
> 
>  *      Returns: BOOLEAN - TRUE if packet should be dropped otherwise FALSE
> 
> but returns SUCCESS (defined as 0) and ERROR (defined as -1).
> 
> Also, shouldn't those functions be converted to return 'bool' when the 
> description say so (happy to do it).

Does anyone use the Cisco MIC code anymore? I guess we can't just rip it
out...  It was a proprietary Cisco extension back before WPA.

In any case, the comment should be changed to reflect the current return
values of the function, and SUCCESS and ERROR in decapsulate() should be
changed to 0 and -1 respectively; defining stuff like success/error just
makes things ugly, and 0 and -1 are well-enough-known that there should
be no question what they mean.

Essentially, the code is using 0 and -1 as booleans anyway.  I say get
rid of SUCCESS & ERROR and just use 0 and -1, then change the comment.

dan

> Richard Knutsson
> 
> -
> To unsubscribe from this list: send the line "unsubscribe netdev" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html