2 files changed, 6 insertions, 5 deletions
diff --git a/util-linux/getopt.c b/util-linux/getopt.c
index 83cc1efea..f6ecc3dde 100644
--- a/util-linux/getopt.c
+++ b/util-linux/getopt.c
@@ -347,9 +347,9 @@ static struct option *add_long_options(struct option *long_options, char *option
 static void set_shell(const char *new_shell)
 {
-        if (!strcmp(new_shell, "bash") || !strcmp(new_shell, "sh"))
+        if (strcmp(new_shell, "bash") == 0 || strcmp(new_shell, "sh") == 0)
                return;
-        if (!strcmp(new_shell, "tcsh") || !strcmp(new_shell, "csh"))
+        if (strcmp(new_shell, "tcsh") == 0 || strcmp(new_shell, "csh") == 0)
                option_mask32 |= SHELL_IS_TCSH;
        else
                bb_error_msg("unknown shell '%s', assuming bash", new_shell);
diff --git a/util-linux/readprofile.c b/util-linux/readprofile.c
index d5230387c..31abb6bb2 100644
--- a/util-linux/readprofile.c
+++ b/util-linux/readprofile.c
@@ -174,7 +174,7 @@ int readprofile_main(int argc UNUSED_PARAM, char **argv)
                        bb_error_msg_and_die("%s(%i): wrong map line",
                                        mapFile, maplineno);
-                if (!strcmp(fn_name, "_stext")) /* only elf works like this */ {
+                if (strcmp(fn_name, "_stext") == 0) /* only elf works like this */ {
                        add0 = fn_add;
                        break;
                }
@@ -224,8 +224,9 @@ int readprofile_main(int argc UNUSED_PARAM, char **argv)
                if (optBins) {
                        if (optVerbose || this > 0)
                                printf("  total\t\t\t\t%u\n", this);
-                } else if ((this || optAll)
+                } else
-                        && (fn_len = next_add-fn_add) != 0
+                if ((this || optAll)
+                 && (fn_len = next_add-fn_add) != 0
                ) {
                        if (optVerbose)
                                printf("%016llx %-40s %6u %8.4f\n", fn_add,