From: Alejandro Colomar <alx@kernel.org>
Date: Mon, 15 Apr 2024 09:52:43 +0000 (+0200)
Subject: lib/getrange.c: getrange(): Return early to reduce indentation
X-Git-Tag: 4.15.2~45
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=bbb2735cc042e3ef5e4c9febc4965cdecd991f55;p=thirdparty%2Fshadow.git

lib/getrange.c: getrange(): Return early to reduce indentation

Signed-off-by: Alejandro Colomar <alx@kernel.org>
---

diff --git a/lib/getrange.c b/lib/getrange.c
index 389f90426..0410cb81f 100644
--- a/lib/getrange.c
+++ b/lib/getrange.c
@@ -48,38 +48,38 @@ getrange(const char *range,
 			return -1;
 		*has_max = true;
 
-		/* -<long> */
-	} else {
-		errno = 0;
-		*min = strtoul_noneg(range, &endptr, 10);
-		if (endptr == range || 0 != errno)
+		return 0;  /* -<long> */
+	}
+
+	errno = 0;
+	*min = strtoul_noneg(range, &endptr, 10);
+	if (endptr == range || 0 != errno)
+		return -1;
+	*has_min = true;
+
+	switch (*endptr) {
+	case '\0':
+		/* <long> */
+		*has_max = true;
+		*max = *min;
+		break;
+	case '-':
+		endptr++;
+		if ('\0' == *endptr)
+			return 0;  /* <long>- */
+		if (!isdigit(*endptr))
 			return -1;
-		*has_min = true;
-
-		switch (*endptr) {
-		case '\0':
-			/* <long> */
-			*has_max = true;
-			*max = *min;
-			break;
-		case '-':
-			endptr++;
-			if ('\0' == *endptr)
-				return 0;  /* <long>- */
-			if (!isdigit(*endptr))
-				return -1;
-
-			errno = 0;
-			*max = strtoul_noneg(endptr, &endptr, 10);
-			if ('\0' != *endptr || 0 != errno)
-				return -1;
-			*has_max = true;
-
-			/* <long>-<long> */
-			break;
-		default:
+
+		errno = 0;
+		*max = strtoul_noneg(endptr, &endptr, 10);
+		if ('\0' != *endptr || 0 != errno)
 			return -1;
-		}
+		*has_max = true;
+
+		/* <long>-<long> */
+		break;
+	default:
+		return -1;
 	}
 
 	return 0;