diff options
Diffstat (limited to 'utils/iscsi-gen-initiatorname.sh.template')
-rw-r--r-- | utils/iscsi-gen-initiatorname.sh.template | 16 |
1 files changed, 10 insertions, 6 deletions
diff --git a/utils/iscsi-gen-initiatorname.sh.template b/utils/iscsi-gen-initiatorname.sh.template index a06d6d0..5c2bfdf 100644 --- a/utils/iscsi-gen-initiatorname.sh.template +++ b/utils/iscsi-gen-initiatorname.sh.template @@ -77,7 +77,7 @@ usage_and_exit() echo "Where OPTIONS are from:" echo " -h print usage and exit" echo " -f overwrite existing InitiatorName, if any" - echo " -p IQN-PRE set the prefix for the IQN generated (default ${DEFAULT_IQN_PREFIX})" + echo " -p IQN-PRE set prefix for generated IQN (default ${DEFAULT_IQN_PREFIX})" exit $xit_val } @@ -112,7 +112,12 @@ done shift $(($OPTIND-1)) if [ "$#" -gt 0 ] ; then - echo "Invalid argument(s): $*" + echo "Error: Invalid argument(s): $*" 1>&2 + usage_and_exit 1 +fi + +if [ "$EUID" -ne 0 ] ; then + echo "Error: You must be root to run this command" 1>&2 usage_and_exit 1 fi @@ -131,21 +136,20 @@ KERNEL_INAME="$(kernel_supplied_initiatorname)" # if we have a local initiator name and "force" is not set end it now if [ "$InitiatorName" -a -z "$FORCE" ] ; then echo "Error: you cannot overwrite the current InitiatorName unless 'force' is set." 1>&2 - echo "Please call '$NAME -f' to update iSCSI InitiatorName, if needed." 1>&2 - exit 1 + usage_and_exit 1 fi # ensure we can write the initiator name file if [ -r "$INAME_FILE" ] ; then if [ ! -w "$INAME_FILE" ] ; then echo "Error: cannot update InitiatorName, write protected: $INAME_FILE" 1>&1 - echo "Please ensure you are root and filesystem is read/write." 1>&2 + echo "Please ensure the filesystem is read/write." 1>&2 exit 1 fi # the file exists but we can write over it elif [ ! -w "$INAME_DIR" ] ; then echo "Error: no write permission in directory: $INAME_DIR" 1>&2 - echo "Please ensure you are root and filesystem is read/write." 1>&2 + echo "Please ensure the filesystem is read/write." 1>&2 exit 1 fi |