The following commit has been merged in the openafs-stable-1_6_x branch:
commit 9a324f14e7c98a302d7276d7c151ccef9242c9f9
Author: Simon Wilkinson <[email protected]>
Date: Fri Mar 30 19:18:47 2012 +0100
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.
Reviewed-on: http://gerrit.openafs.org/7077
Reviewed-by: Derrick Brashear <[email protected]>
Tested-by: Derrick Brashear <[email protected]>
(cherry picked from commit 1e30c00e7d9b45d65e819d39414939f2d5f7631b)
Change-Id: I886bef77fdedb63d1c83e657c25d112e0a635db2
Reviewed-on: http://gerrit.openafs.org/11844
Tested-by: BuildBot <[email protected]>
Reviewed-by: Benjamin Kaduk <[email protected]>
Reviewed-by: Stephan Wiesand <[email protected]>
src/libafscp/afscp_server.c | 4 +---
1 files changed, 1 insertions(+), 3 deletions(-)
--
OpenAFS Master Repository
_______________________________________________
OpenAFS-cvs mailing list
[email protected]
https://lists.openafs.org/mailman/listinfo/openafs-cvs
