initrd.scripts: improve findnfsmount(), simplify code and branches

defaults/initrd.scripts

@@ -404,18 +404,18 @@ mount_sysfs() {
 findnfsmount() {
     if [ -z "${IP}" ]; then
         # IP is not set, return straight away
-        return 0
+        return 1
     fi
 
     if ! busybox udhcpc -n -T 15 -q; then
         bad_msg "udhcpc returned error, skipping nfs setup..."
-        return 0
+        return 1
     fi
 
     local options=
 
     [ -e /rootpath ] && NFSROOT=$(cat /rootpath)
-    if [ "${NFSROOT}" = "" ]; then
+    if [ -z "${NFSROOT}" ]; then
         # Obtain NFSIP
         # TODO: this is bogus, because dmesg is a circular buffer...
         # TODO: provide dmesg as symlink (see gen_initramfs.sh)
@@ -443,40 +443,42 @@ findnfsmount() {
         done
 
         # Setup NFSROOT
-        if [ "${NFSIP}" != "" ] && [ "$NFSPATH" != "" ]; then
+        if [ -n "${NFSIP}" ] && [ -n "$NFSPATH" ]; then
             NFSROOT="${NFSIP}:${NFSPATH}"
         else
             bad_msg "The DHCP Server did not send a valid root-path."
             bad_msg "Please check your DHCP setup, or set nfsroot=<...>"
+            return 1
         fi
     fi
 
-    if [ "${NFSROOT}" != "" ]; then
+    # expecting a valid NFSROOT here, or the code should have returned
-        NFSOPTIONS=${NFSROOT#*,}
+    NFSOPTIONS=${NFSROOT#*,}
-        NFSROOT=${NFSROOT%%,*}
+    NFSROOT=${NFSROOT%%,*}
-        if [ "${NFSOPTIONS}" = "${NFSROOT}" ]; then
+    if [ "${NFSOPTIONS}" = "${NFSROOT}" ]; then
-            NFSOPTIONS="${DEFAULT_NFSOPTIONS}"
+        NFSOPTIONS="${DEFAULT_NFSOPTIONS}"
-        else
+    else
-            NFSOPTIONS="${DEFAULT_NFSOPTIONS},${NFSOPTIONS}"
+        NFSOPTIONS="${DEFAULT_NFSOPTIONS},${NFSOPTIONS}"
-        fi
+    fi
 
-        local path=
+    local path=
         # override path if on livecd
-        if is_livecd; then
+    if is_livecd; then
-            path="${CDROOT_PATH}"
+        path="${CDROOT_PATH}"
-            good_msg "Attempting to mount NFS CD image on ${NFSROOT}."
+        good_msg "Attempting to mount NFS CD image on ${NFSROOT}."
-        else
+    else
-            path="${NEW_ROOT}"
+        path="${NEW_ROOT}"
-            good_msg "Attempting to mount NFS root on ${NFSROOT}."
+        good_msg "Attempting to mount NFS root on ${NFSROOT}."
-        fi
+    fi
 
-        good_msg "NFS options: ${NFSOPTIONS}"
+    good_msg "NFS options: ${NFSOPTIONS}"
-        mount -t nfs -o ${NFSOPTIONS} "${NFSROOT}" "${path}"
+    mount -t nfs -o ${NFSOPTIONS} "${NFSROOT}" "${path}"
-        if [ "${?}" = "0" ]; then
+    if [ "${?}" = "0" ]; then
-            REAL_ROOT="/dev/nfs"
+        REAL_ROOT="/dev/nfs"
-        else
+        return 0
-            bad_msg "NFS Mounting failed. Is the path corrent ?"
+    else
-        fi
+        bad_msg "NFS Mounting failed. Is the path corrent ?"
+        return 1
     fi
 }