From 7b0294558cc3add381a58a1018ee6d97d031ec71 Mon Sep 17 00:00:00 2001
From: Konstantin Pereiaslov <perk11@perk11.info>
Date: Mon, 1 Jan 2024 17:50:54 -0600
Subject: [PATCH] Fix passing invalid value to pause-method argument not
 causing an error

---
 arguments_parsing.c | 13 +++++++------
 pause_methods.h     |  1 +
 2 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/arguments_parsing.c b/arguments_parsing.c
index fee9db3..fc25e32 100644
--- a/arguments_parsing.c
+++ b/arguments_parsing.c
@@ -108,22 +108,23 @@ char *parse_command_line_arguments(int argc, char *argv[]) {
                 break;
             case 'm':
                 char *method = strdup(optarg);
-                for (int i = 0; i<sizeof(method); i++) {
+                for (int i = 0; i < sizeof(method); i++) {
                     method[i] = toupper(method[i]);
                 }
-                for (int i = 1;  pause_method_string[i] != NULL; i++) {
+                pause_method = PAUSE_METHOD_UNKNOWN;
+                for (int i = 1; pause_method_string[i] != NULL; i++) {
                     if (strcmp(pause_method_string[i], method) == 0) {
                         pause_method = i;
                         break;
                     }
                 }
-                if (!pause_method) {
+                if (pause_method == PAUSE_METHOD_UNKNOWN) {
                     fprintf_error("Invalid value for --pause-method|m argument: \"%s\". Supported values: ", optarg);
-                    for (int i=1; pause_method_string[i] != NULL; i++) {
+                    for (int i = 1; pause_method_string[i] != NULL; i++) {
                         fprintf_error(pause_method_string[i]);
-                        if (pause_method_string[i+1] != NULL) {
+                        if (pause_method_string[i + 1] != NULL) {
                             fprintf_error(", ");
-                         }
+                        }
                     }
                     fprintf_error("\n");
                     exit(1);
diff --git a/pause_methods.h b/pause_methods.h
index 7c61ca8..377d9d5 100644
--- a/pause_methods.h
+++ b/pause_methods.h
@@ -6,6 +6,7 @@
 #define RUNWHENIDLE_PAUSE_METHODS_H
 
 enum pause_method {
+    PAUSE_METHOD_UNKNOWN = 0,
     //order must match order in pause_method_string
     PAUSE_METHOD_SIGTSTP = 1,
     PAUSE_METHOD_SIGSTOP = 2,