]> git.ipfire.org Git - thirdparty/krb5.git/commitdiff
Allow null inputs to response item functions
authorNathaniel McCallum <npmccallum@redhat.com>
Fri, 12 Oct 2012 14:30:09 +0000 (10:30 -0400)
committerGreg Hudson <ghudson@mit.edu>
Fri, 12 Oct 2012 15:44:14 +0000 (11:44 -0400)
src/lib/krb5/krb/response_items.c

index 243df48d0d73c35f8fc5e3dd79db1be0cae5a282..606d4f34c38fa3050bef5836f9b94c485c59a6b9 100644 (file)
@@ -53,6 +53,9 @@ k5_response_items_reset(k5_response_items *ri)
 {
     size_t i;
 
+    if (ri == NULL)
+        return;
+
     for (i = 0; i < ri->count; i++)
         free(ri->questions[i]);
     free(ri->questions);
@@ -74,12 +77,14 @@ k5_response_items_reset(k5_response_items *ri)
 krb5_boolean
 k5_response_items_empty(const k5_response_items *ri)
 {
-    return ri->count == 0;
+    return ri == NULL ? TRUE : ri->count == 0;
 }
 
 const char * const *
 k5_response_items_list_questions(const k5_response_items *ri)
 {
+    if (ri == NULL)
+        return NULL;
     return (const char * const *)ri->questions;
 }
 
@@ -88,6 +93,9 @@ find_question(const k5_response_items *ri, const char *question)
 {
     size_t i;
 
+    if (ri == NULL)
+        return -1;
+
     for (i = 0; i < ri->count; i++) {
         if (strcmp(ri->questions[i], question) == 0)
             return i;
@@ -101,8 +109,12 @@ push_question(k5_response_items *ri, const char *question,
               const char *challenge)
 {
     char **tmp;
-    const size_t size = sizeof(char*) * (ri->count + 2);
+    size_t size;
+
+    if (ri == NULL)
+        return EINVAL;
 
+    size = sizeof(char *) * (ri->count + 2);
     tmp = realloc(ri->questions, size);
     if (tmp == NULL)
         return ENOMEM;