From 3181f5556f228a697dabec04eff661b179f7c2d5 Mon Sep 17 00:00:00 2001
From: "Ralf S. Engelschall" <rse@FreeBSD.org>
Date: Sun, 4 Feb 2007 20:52:57 +0000
Subject: [PATCH] cleanup code: remove superfluous comma at end of enumeration
 declaration, remove useless "break" after exit(3) call, and add a missing
 va_end(3) call.

---
 usr.bin/getent/getent.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/usr.bin/getent/getent.c b/usr.bin/getent/getent.c
index 0694fdc751f7..d7558135b6f2 100644
--- a/usr.bin/getent/getent.c
+++ b/usr.bin/getent/getent.c
@@ -77,7 +77,7 @@ enum {
 	RV_OK		= 0,
 	RV_USAGE	= 1,
 	RV_NOTFOUND	= 2,
-	RV_NOENUM	= 3,
+	RV_NOENUM	= 3
 };
 
 static struct getentdb {
@@ -109,7 +109,6 @@ main(int argc, char *argv[])
 	for (curdb = databases; curdb->name != NULL; curdb++) {
 		if (strcmp(curdb->name, argv[1]) == 0) {
 			exit(curdb->callback(argc, argv));
-			break;
 		}
 	}
 	fprintf(stderr, "Unknown database: %s\n", argv[1]);
@@ -178,6 +177,7 @@ printfmtstrings(char *strings[], const char *prefix, const char *sep,
 		curpref = sep;
 	}
 	printf("\n");
+	va_end(ap);
 }
 
 /*