Since strv_* functions handle null arguments, this warning is actually
valid.
src/strv.c: In function ‘strv_copy’:
src/strv.c:68:21: warning: ‘k’ may be used uninitialized in this function
[-Wuninitialized]
---
src/strv.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git src/strv.c src/strv.c
index 71b77c9..f15aa87 100644
--- src/strv.c
+++ src/strv.c
@@ -67,11 +67,11 @@ void strv_free(char **l) {
char **strv_copy(char **l) {
char **r, **k;
- if (!(r = new(char*, strv_length(l)+1)))
+ if (!(k = r = new(char*, strv_length(l)+1)))
return NULL;
if (l)
- for (k = r; *l; k++, l++)
+ for (; *l; k++, l++)
if (!(*k = strdup(*l)))
goto fail;
--
1.7.4.1
_______________________________________________
systemd-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/systemd-devel
