On Fri, Sep 21, 2018 at 02:42:38PM -0400, Mimi Zohar wrote: > On Fri, 2018-09-07 at 13:25 -0700, Eric Biggers wrote: > > From: Eric Biggers <ebiggers@xxxxxxxxxx> > > > > The 'init_keyring' variable actually just gave the value of > > CONFIG_INTEGRITY_TRUSTED_KEYRING. We should check the config option > > directly instead. No change in behavior; this just simplifies the code. > > We try to minimize as much as possible "ifdefs" in C code. This > change is moving in the wrong direction. > > Mimi So your preferred approach is to store the values of Kconfig options in variables? That defeats much of the point of having Kconfig options... - Eric
