Fix kadmin script mode command-not-found error

In ss_wrapper.c, if ss_execute_command() returns an error, we should
call ss_perror() with *args as the third argument and not request
(which is NULL).  Expand out the conditional into three commented
branches for greater clarity, since the error-handling is no longer
identical for the ss_execute_command() and ss_execute_line() cases.

ticket: 7991

diff --git a/src/kadmin/cli/ss_wrapper.c b/src/kadmin/cli/ss_wrapper.c
index d9a1d8f77e66cea19f646ae1d5e721a7fc29a4cc..776c383d7128bfb136b7e286cdde6e5a918b6039 100644 (file)
--- a/src/kadmin/cli/ss_wrapper.c
+++ b/src/kadmin/cli/ss_wrapper.c
@@ -49,17 +49,25 @@ main(int argc, char *argv[])
         ss_perror(sci_idx, retval, _("creating invocation"));
         exit(1);
     }
-    if (request == NULL && *args == NULL) {
-        (void)ss_listen(sci_idx);
-    } else {
-        if (request != NULL)
-            code = ss_execute_line(sci_idx, request);
-        else
-            code = ss_execute_command(sci_idx, args);
+
+    if (*args != NULL) {
+        /* Execute post-option arguments as a single script-mode command. */
+        code = ss_execute_command(sci_idx, args);
+        if (code) {
+            ss_perror(sci_idx, code, *args);
+            exit_status = 1;
+        }
+    } else if (request != NULL) {
+        /* Execute the -q option as a single interactive command. */
+        code = ss_execute_line(sci_idx, request);
         if (code != 0) {
             ss_perror(sci_idx, code, request);
             exit_status = 1;
         }
+    } else {
+        /* Prompt for commands. */
+        (void)ss_listen(sci_idx);
     }
+
     return quit() ? 1 : exit_status;
 }