blobmsg: fix blobmsg_parse_array, drop name field requirement

[project/libubox.git] / blobmsg.c

diff --git a/blobmsg.c b/blobmsg.c
index 12f603c..7257eae 100644 (file)
--- a/blobmsg.c
+++ b/blobmsg.c
@@ -21,6 +21,7 @@ static const int blob_type[__BLOBMSG_TYPE_LAST] = {
        [BLOBMSG_TYPE_INT32] = BLOB_ATTR_INT32,
        [BLOBMSG_TYPE_INT64] = BLOB_ATTR_INT64,
        [BLOBMSG_TYPE_STRING] = BLOB_ATTR_STRING,
+       [BLOBMSG_TYPE_UNSPEC] = BLOB_ATTR_BINARY,
 };
 
 static uint16_t
@@ -52,7 +53,7 @@ bool blobmsg_check_attr(const struct blob_attr *attr, bool name)
        len = blobmsg_data_len(attr);
        data = blobmsg_data(attr);
 
-       if (!id || id > BLOBMSG_TYPE_LAST)
+       if (id > BLOBMSG_TYPE_LAST)
                return false;
 
        if (!blob_type[id])
@@ -61,11 +62,23 @@ bool blobmsg_check_attr(const struct blob_attr *attr, bool name)
        return blob_check_type(data, len, blob_type[id]);
 }
 
-bool blobmsg_check_attr_list(const struct blob_attr *attr, int type, bool name)
+bool blobmsg_check_attr_list(const struct blob_attr *attr, int type)
 {
        struct blob_attr *cur;
+       bool name;
        int rem;
 
+       switch (blobmsg_type(attr)) {
+       case BLOBMSG_TYPE_TABLE:
+               name = true;
+               break;
+       case BLOBMSG_TYPE_ARRAY:
+               name = false;
+               break;
+       default:
+               return false;
+       }
+
        blobmsg_for_each_attr(cur, attr, rem) {
                if (blobmsg_type(cur) != type)
                        return false;
@@ -77,6 +90,33 @@ bool blobmsg_check_attr_list(const struct blob_attr *attr, int type, bool name)
        return true;
 }
 
+int blobmsg_parse_array(const struct blobmsg_policy *policy, int policy_len,
+                       struct blob_attr **tb, void *data, int len)
+{
+       struct blob_attr *attr;
+       int i = 0;
+
+       memset(tb, 0, policy_len * sizeof(*tb));
+       __blob_for_each_attr(attr, data, len) {
+               if (policy[i].type != BLOBMSG_TYPE_UNSPEC &&
+                   blob_id(attr) != policy[i].type)
+                       continue;
+
+               if (!blobmsg_check_attr(attr, false))
+                       return -1;
+
+               if (tb[i])
+                       continue;
+
+               tb[i++] = attr;
+               if (i == policy_len)
+                       break;
+       }
+
+       return 0;
+}
+
+
 int blobmsg_parse(const struct blobmsg_policy *policy, int policy_len,
                   struct blob_attr **tb, void *data, int len)
 {