Do not check whether a pointer is NULL, since free(3) already takes care of

this case.

Reviewed by:	ume

1 files changed, 3 insertions, 6 deletions

diff --git a/lib/libc/net/getservent.c b/lib/libc/net/getservent.c
index 24c73d7..2224add 100644
--- a/lib/libc/net/getservent.c
+++ b/lib/libc/net/getservent.c
@@ -67,10 +67,8 @@ servent_data_clear(struct servent_data *sed)
 		sed->fp = NULL;
 	}
 #ifdef YP
-	if (sed->key) {
-		free(sed->key);
-		sed->key = NULL;
-	}
+	free(sed->key);
+	sed->key = NULL;
 #endif
 }
 
@@ -201,8 +199,7 @@ _getservent_yp(struct servent_data *sed)
 	}
 
 	if (!sed->stepping_yp) {
-		if (sed->key)
-			free(sed->key);
+		free(sed->key);
 		rv = yp_first(sed->yp_domain, "services.byname", &sed->key,
 		    &sed->keylen, &result, &resultlen);
 		if (rv) {