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  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:   Sun, 10 Sep 2017 09:27:58 -0700
From:   Linus Torvalds <>
To:     Masahiro Yamada <>
Cc:     Sam Ravnborg <>, Michal Marek <>,
        Linux Kbuild mailing list <>,
        "" <>,
        Rob Herring <>,
        Jonathan Corbet <>,
        Richard Purdie <>,
        Greg Kroah-Hartman <>,
        Andrew Morton <>,
        Mauro Carvalho Chehab <>,
        Nicholas Piggin <>,
        "open list:DOCUMENTATION" <>,
        Markus Heiser <>,
        Linux Kernel Mailing List <>,
        Frank Rowand <>,
        Rob Herring <>,
        SeongJae Park <>,
        "Yann E. MORIN" <>
Subject: Re: [RFC PATCH 0/3] kbuild: generate intermediate C files instead of
 copying _shipped files

On Sun, Sep 10, 2017 at 6:58 AM, Masahiro Yamada
<> wrote:
> "is_reserved_word()" sounds like a boolean function
> that returns 1 or 0.
> Maybe, the choice of the function name was not nice.

Yeah, not great name. That's the old name, though - I didn't change
that part, I just changed how it used to return the token structure
pointer, which would be NULL when it wasn't a keyword.

I actually *should* have made it just return 0 for the "not a keyword"
case rather than -1, and that would have ended up being semantically
closer to the old use (because you could treat the return value as a
boolean, like you could with the token pointer). But it's been
literally decades since I used bison/flex, and I didn't remember the
rules for 'enum yytokentype', so I just thought "negative numbers for
error" was safer. Zero would have been fine, no token can have that
number anyway (it just means EOF).

And negative wasn't safer, it caused that bug due to the bare boolean
use I hadn't noticed.

Oh well.


Powered by blists - more mailing lists