src/liboping.c: Let ping_setopt() return -1 if 'value' is NULL.

The manpage states that 'value' must not be NULL. Anyway, imho, it's still
better to avoid segfaults inside a library.

diff --git a/src/liboping.c b/src/liboping.c
index b21188b..36a3a3e 100644 (file)
--- a/src/liboping.c
+++ b/src/liboping.c
@@ -1054,6 +1054,9 @@ int ping_setopt (pingobj_t *obj, int option, void *value)
 {
        int ret = 0;
 
+       if (value == NULL)
+               return (-1);
+
        switch (option)
        {
                case PING_OPT_TIMEOUT: