OpenAFS Master Repository branch, master, updated. openafs-devel-1_5_76-2329-g9a007a9
Gerrit Code Review
gerrit@openafs.org
Sun, 8 Apr 2012 18:19:11 -0700 (PDT)
The following commit has been merged in the master branch:
commit 9a007a9df43645b63a8b642029b4931928f9268b
Author: Simon Wilkinson <sxw@your-file-system.com>
Date: Sat Mar 31 06:58:01 2012 -0400
cmd: Use strl* rather than strn* to avoid overrun
The NName function was using strncat(a, b, sizeof(a)), which doesn't
work as you would expect if 'a' already contains data. To avoid the
potential buffer overflow, switch to just using strlcat.
Caught by clang-analyzer
Change-Id: Idd2c630c07a93b27e8d629339589aa6686290eae
Reviewed-on: http://gerrit.openafs.org/7092
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Derrick Brashear <shadow@dementix.org>
src/cmd/cmd.c | 5 ++---
1 files changed, 2 insertions(+), 3 deletions(-)
--
OpenAFS Master Repository