make sure we're explicitly capturing return codes in places where they are tested...

[monkeysphere.git] / src / share / ma / update_users

diff --git a/src/share/ma/update_users b/src/share/ma/update_users
index 092d10870c6e8356bbec956ec3ed4f844cf734a5..bfefc31cdb83b294175131d502136b17520e7ce5 100644 (file)
--- a/src/share/ma/update_users
+++ b/src/share/ma/update_users
@@ -93,8 +93,8 @@ for uname in $unames ; do
 
            # process authorized_user_ids file, as monkeysphere user
            su_monkeysphere_user \
-               ". ${SYSSHAREDIR}/common; process_authorized_user_ids $TMP_AUTHORIZED_USER_IDS"
-           RETURN="$?"
+               ". ${SYSSHAREDIR}/common; process_authorized_user_ids $TMP_AUTHORIZED_USER_IDS" \
+               || RETURN="$?"
        else
            log debug "not processing authorized_user_ids."
        fi
@@ -154,4 +154,5 @@ for uname in $unames ; do
     rm -rf "$TMPLOC"
 done
 
+return $RETURN
 }