stardis

commit f4409b68c6bf71bc35c3bc221e47733e2f2e61b3
parent be54031cc1b255cb3ef755188a1dc75b392c5cd4
Author: Christophe Coustet <christophe.coustet@meso-star.com>
Date:   Thu, 24 Sep 2020 12:34:36 +0200

BugFix: undetected option incompatibility

Diffstat:
M
src/stardis-parsing.c
 | 
19
+++++++++++++++++++

1 file changed, 19 insertions(+), 0 deletions(-)
diff --git a/src/stardis-parsing.c b/src/stardis-parsing.c
@@ -652,10 +652,29 @@ parse_args
     /*case 'F': see 's' */
 
     case 'g':
+      if(args->mode & MODE_BIN_GREEN) {
+        res = RES_BAD_ARG;
+        logger_print(args->logger, LOG_ERROR,
+          "Options -%c and -%c are exclusive.\n",
+          (char)opt, mode_option(MODE_BIN_GREEN));
+        goto error;
+      }
       args->mode |= MODE_GREEN;
       break;
 
     case 'G':
+      if(args->mode & (MODE_BIN_GREEN | MODE_GREEN)) {
+        res = RES_BAD_ARG;
+        if(args->mode & MODE_BIN_GREEN)
+          logger_print(args->logger, LOG_ERROR,
+            "Option -%c cannot be used twice.\n",
+            (char)opt);
+        else
+          logger_print(args->logger, LOG_ERROR,
+            "Options -%c and -%c are exclusive.\n",
+            (char)opt, mode_option(MODE_GREEN));
+        goto error;
+      }
       args->mode |= MODE_BIN_GREEN;
       args->bin_green_filename = optarg;
       break;