diff options
author | Dan McGee <dan@archlinux.org> | 2011-07-05 14:16:17 -0500 |
---|---|---|
committer | Dan McGee <dan@archlinux.org> | 2011-07-05 21:29:02 -0500 |
commit | ae7139adcfa65991c71616e8de7910ff722d4166 (patch) | |
tree | 2126517e48edf71ae49e51fa264cc8ea4b946f81 /lib/libalpm | |
parent | dfc532668d4a4182ce196a895fdd5b017b505c6f (diff) | |
download | pacman-ae7139adcfa65991c71616e8de7910ff722d4166.tar.xz |
Remove most usages of strncmp()
The supposed safety blanket of this function is better handled by
explicit length checking and usages of strlen() on known NULL-terminated
strings rather than hoping things fit in a buffer. We also have no need
to fully fill a PATH_MAX length variable with NULLs every time as long
as a single terminating byte is there. Remove usages of it by using
strcpy() or memcpy() as appropriate, after doing length checks via
strlen().
Signed-off-by: Dan McGee <dan@archlinux.org>
Diffstat (limited to 'lib/libalpm')
-rw-r--r-- | lib/libalpm/handle.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/libalpm/handle.c b/lib/libalpm/handle.c index ddd76a25..9bffd4fd 100644 --- a/lib/libalpm/handle.c +++ b/lib/libalpm/handle.c @@ -299,7 +299,7 @@ static char *canonicalize_path(const char *path) { len += 1; } CALLOC(new_path, len + 1, sizeof(char), return NULL); - strncpy(new_path, path, len); + strcpy(new_path, path); new_path[len - 1] = '/'; return new_path; } |