From: Simon Wilkinson <sxw@your-file-system.com>
Date: Fri, 30 Mar 2012 18:18:47 +0000 (+0100)
Subject: libafscp: Use strdup, rather than rolling our own
X-Git-Tag: openafs-stable-1_8_0pre1~2623
X-Git-Url: https://git.openafs.org/?p=openafs.git;a=commitdiff_plain;h=1e30c00e7d9b45d65e819d39414939f2d5f7631b;hp=34afb3e6b903fc1659b1c869fa2fb4226038e076

libafscp: Use strdup, rather than rolling our own

   A = malloc(strlen(B)+ 1);
   memset(A, 0, strlen(B) + 1);
   strlcpy(A, B, strlen(B) + 1);
can be more simply written as
   A = strdup(B);

Doing so also avoids a warning from clang that strlcpy isn't checking
for A overflowing.

Change-Id: I0e21f86eda7bdc0ce869e651c5ceb23d7494e1e2
Reviewed-on: http://gerrit.openafs.org/7077
Reviewed-by: Derrick Brashear <shadow@dementix.org>
Tested-by: Derrick Brashear <shadow@dementix.org>
---

diff --git a/src/libafscp/afscp_server.c b/src/libafscp/afscp_server.c
index 92d29b0..abb82e8 100644
--- a/src/libafscp/afscp_server.c
+++ b/src/libafscp/afscp_server.c
@@ -122,9 +122,7 @@ afscp_CellByName(const char *cellname, const char *realmname)
     memset(thecell, 0, sizeof(struct afscp_cell));
     strlcpy(thecell->name, cellname, sizeof(thecell->name));
     if (realmname != NULL) {
-	thecell->realm = malloc(strlen(realmname) + 1);
-	memset(thecell->realm, 0, strlen(realmname) + 1);
-	strlcpy(thecell->realm, realmname, strlen(realmname) + 1);
+	thecell->realm = strdup(realmname);
     } else {
 	thecell->realm = NULL;
     }