diff options
| author | Dan McGee <dan@archlinux.org> | 2011-08-25 17:14:19 -0500 |
|---|---|---|
| committer | Dan McGee <dan@archlinux.org> | 2011-08-25 17:14:19 -0500 |
| commit | c5982a3eb5287afdec6c9b859f81795c773da8fe (patch) | |
| tree | a7c560c2e6552a0344998766ca966ca740298915 /lib/libalpm | |
| parent | e1dce078b247142f7db80bf2d338e431df34691e (diff) | |
| download | pacman-c5982a3eb5287afdec6c9b859f81795c773da8fe.tar.xz | |
strtrim: don't move empty string
There were many cases where the string coming in was a blank line, e.g.
"\n\0", length 1. The trim routine starts by trimming leading spaces,
thus trimming everything. We would then proceed to do a memmove of the
NULL byte, which is completely worthless as we can just assign it
instead.
Signed-off-by: Dan McGee <dan@archlinux.org>
Diffstat (limited to 'lib/libalpm')
| -rw-r--r-- | lib/libalpm/util.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/lib/libalpm/util.c b/lib/libalpm/util.c index 7e3bc372..cf783893 100644 --- a/lib/libalpm/util.c +++ b/lib/libalpm/util.c @@ -192,7 +192,12 @@ char *_alpm_strtrim(char *str) pch++; } if(pch != str) { - memmove(str, pch, (strlen(pch) + 1)); + size_t len = strlen(pch); + if(len) { + memmove(str, pch, len + 1); + } else { + *str = '\0'; + } } /* check if there wasn't anything but whitespace in the string. */ |