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 b21188b21abc9fc0654396f26a619e124bafdb9c..36a3a3ef623d00c68f68d6fe1bdc241c89d5890e 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: