summary |
shortlog |
log |
commit | commitdiff |
tree
raw |
patch |
inline | side by side (from parent 1:
354ea7a)
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.
(cherry picked from commit
924c7057076a23e322d40693912633dc546867e9)
/*----- 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)