The old, bogus behaviour was that it would report `KERR_READONLY' if the
keyring was neither open for writing, /nor/ modified. I think this is
relatively benign, but still well deserving of fixing.
Spotted by Clang.
/*----- Useful macros -----------------------------------------------------*/
#define KEY_WRITE(f) do { \
/*----- Useful macros -----------------------------------------------------*/
#define KEY_WRITE(f) do { \
- if (!(f)->f & KF_WRITE) \
+ if (!((f)->f & KF_WRITE)) \
return (KERR_READONLY); \
} while (0)
return (KERR_READONLY); \
} while (0)