array: Make array_get() return an error if an element wasn't set

The array_get() function returns success when an entry is retrieved that
was never set. This makes it hard to distinguish uninitialized entries.
This commit changes array_get() to return an error if the caller is
attempting to retrieve an uninitialized entry.

diff --git a/array.c b/array.c
index 3a143ded3f429210f612a4fc620313988fd83d69..325e68b14a2eac2e2eb4289acd5f206dd2dd374f 100644 (file)
--- a/array.c
+++ b/array.c
@@ -208,8 +208,12 @@ int array_get(struct array *array, int idx, void **data)
        }
 
        real_idx = idx % array->size;
-       *data = array->data[real_idx];
 
+       if(!array->flags[real_idx]) {
+               return(-ENOENT);
+       }
+
+       *data = array->data[real_idx];
        return(0);
 }