Update the afc_get_device_info helper to return an afc_error_t

We should return any underlying error afc_get_device_info returns
so one is able to act properly. Also renamed it to "key" instead
of "field" to be more generic.

1 files changed, 17 insertions, 11 deletions

diff --git a/src/AFC.c b/src/AFC.c
index d2761cf..b27080a 100644
--- a/src/AFC.c
+++ b/src/AFC.c
@@ -468,26 +468,32 @@ afc_error_t afc_get_device_info(afc_client_t client, char ***infos)
         return ret;
 }
 
-/** Get a specific field of the device info for a client connection to phone.
- * Known values are: Model, FSTotalBytes, FSFreeBytes and FSBlockSize. This is
- * a helper function for afc_get_device_info().
+/** Get a specific key of the device info list for a client connection.
+ * Known key values are: Model, FSTotalBytes, FSFreeBytes and FSBlockSize.
+ * This is a helper function for afc_get_device_info().
  *
  * @param client The client to get device info for.
- * @param field The field to get the information for
+ * @param key The key to get the value of.
+ * @param value The value for the key if successful or NULL otherwise.
  *
- * @return A char * or NULL if there was an error.
+ * @return AFC_E_SUCCESS on success or an AFC_E_* error value.
  */
-char * afc_get_device_info_field(afc_client_t client, const char *field)
+afc_error_t afc_get_device_info_key(afc_client_t client, const char *key, char **value)
 {
-        char *ret = NULL;
+        afc_error_t ret = AFC_E_INTERNAL_ERROR;
         char **kvps, **ptr;
 
-        if (field == NULL || afc_get_device_info(client, &kvps) != AFC_E_SUCCESS)
-                return NULL;
+        *value = NULL;
+        if (key == NULL)
+                return AFC_E_INVALID_ARGUMENT;
+
+        ret = afc_get_device_info(client, &kvps);
+        if (ret != AFC_E_SUCCESS)
+                return ret;
 
         for (ptr = kvps; *ptr; ptr++) {
-                if (!strcmp(*ptr, field)) {
-                        ret = strdup(*(ptr+1));
+                if (!strcmp(*ptr, key)) {
+                        *value = strdup(*(ptr+1));
                         break;
                 }
         }