On 12/7/23 16:08, Richard W.M. Jones wrote:
Use a named variable to make the boolean condition explicit. Pure refactoring, except applying De Morgan's law to avoid the confusing double negative condition. --- options/keys.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/options/keys.c b/options/keys.c index 52b2736..3a49273 100644 --- a/options/keys.c +++ b/options/keys.c @@ -20,6 +20,7 @@
#include <stdio.h> #include <stdlib.h> +#include <stdbool.h> #include <unistd.h> #include <termios.h> #include <string.h> @@ -152,9 +153,12 @@ get_keys (struct key_store *ks, const char *device, const char *uuid, if (ks) { for (i = 0; i < ks->nr_keys; ++i) { struct key_store_key *key = &ks->keys[i]; + bool key_id_matches_this_device;
- if (STRNEQ (key->id, device) && (!uuid || STRNEQ (key->id, uuid))) - continue; + key_id_matches_this_device = + STREQ (key->id, device) || + (uuid && STREQ (key->id, uuid)); + if (!key_id_matches_this_device) continue;
switch (key->type) { case key_string:
Reviewed-by: Laszlo Ersek <lersek@redhat.com>