subcommand: Avoid referencing possibly undefined $ret.

If a subcommand does not set $ret (like the minimal "hello" example
does), and sets the nounset bash option (set -u), gsu aborts at
the attempt to reference $ret. Fix this flaw by setting $ret to
$GSU_SUCCESS prior to calling the subcommand.

Reported by Congmao Wang.

diff --git a/subcommand b/subcommand
index 3ff31f0a6492e6d3420d533de96d9e02e358825a..261e35f5e39b6f377529c24a136427106d6301cb 100644 (file)
--- a/subcommand
+++ b/subcommand
@@ -549,6 +549,7 @@ gsu()
        # external commands
        for i in $gsu_cmds; do
                if test "$arg" = "$i"; then
+                       ret=$GSU_SUCCESS
                        com_$arg "$@"
                        if (("$ret" < 0)); then
                                gsu_err_msg