From: Arran Cudbard-Bell <a.cudbardb@freeradius.org>
Date: Thu, 29 Apr 2021 13:58:31 +0000 (-0500)
Subject: Make things simple for the stupid compiler
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=a721c8fe705962092fc4c536b351f7dda7111d73;p=thirdparty%2Ffreeradius-server.git

Make things simple for the stupid compiler
---

diff --git a/src/bin/radlock.c b/src/bin/radlock.c
index 65905e00f17..d5470936e5b 100644
--- a/src/bin/radlock.c
+++ b/src/bin/radlock.c
@@ -196,10 +196,10 @@ int main(int argc, char *argv[])
 			EXIT_WITH_SUCCESS;
 
 		default:
-			fr_perror("radlock");
-			EXIT_WITH_FAILURE;
+			break;
 		}
-		break;
+		fr_perror("radlock");
+		EXIT_WITH_FAILURE;
 
 	case RADLOCK_UNLOCK:
 		sem_id = fr_sem_get(file, 0, uid, gid, false, true);
@@ -218,10 +218,10 @@ int main(int argc, char *argv[])
 			goto again;
 
 		default:
-			fr_perror("radlock");
-			EXIT_WITH_FAILURE;
+			break;
 		}
-		break;
+		fr_perror("radlock");
+		EXIT_WITH_FAILURE;
 
 	case RADLOCK_REMOVE:
 		sem_id = fr_sem_get(file, 0, uid, gid, false, true);
@@ -231,7 +231,7 @@ int main(int argc, char *argv[])
 			fr_perror("radlock");
 			EXIT_WITH_FAILURE;
 		}
-		break;
+		EXIT_WITH_SUCCESS;
 
 	case RADLOCK_INFO:
 	{
@@ -293,9 +293,8 @@ int main(int argc, char *argv[])
 		INFO("\tGroup         : %s (%u)", cgid_str, info.sem_perm.cgid);
 		INFO("\tTime          : %s",
 		     fr_asprintf(autofree, "%pV", fr_box_date(fr_time_from_sec(info.sem_ctime))));
-		EXIT_WITH_SUCCESS;
 	}
-		break;
+		EXIT_WITH_SUCCESS;
 
 	case RADLOCK_PERM:
 	{
@@ -323,11 +322,10 @@ int main(int argc, char *argv[])
 			EXIT_WITH_FAILURE;
 		}
 	}
-		break;
+		EXIT_WITH_SUCCESS;
 
 	case RADLOCK_INVALID:
 		usage(64);
-		break;
 	}
 
 	return 0;