Fix: snapshot del-output with name on musl

Some implementations of strtol(), like the one in musl, will
return EINVAL in errno when no valid number was found in
the string.

Signed-off-by: Michael Jeanson <mjeanson@efficios.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

diff --git a/src/bin/lttng/commands/snapshot.c b/src/bin/lttng/commands/snapshot.c
index d948226ae9770591c32dc3599151b8f6e1b6b81f..00aa5b790448e57d3a26d1b8f838bd51d6acf564 100644 (file)
--- a/src/bin/lttng/commands/snapshot.c
+++ b/src/bin/lttng/commands/snapshot.c
@@ -350,7 +350,7 @@ static int cmd_del_output(int argc, const char **argv)
 
        errno = 0;
        id = strtol(argv[1], &name, 10);
 
        errno = 0;
        id = strtol(argv[1], &name, 10);

-       if (id == 0 && errno == 0) {
+       if (id == 0 && (errno == 0 || errno == EINVAL)) {

                ret = del_output(UINT32_MAX, name);
        } else if (errno == 0 && *name == '\0') {
                ret = del_output(id, NULL);
                ret = del_output(UINT32_MAX, name);
        } else if (errno == 0 && *name == '\0') {
                ret = del_output(id, NULL);