diff options
author | Felix Fietkau <nbd@openwrt.org> | 2011-01-23 21:55:41 +0300 |
---|---|---|
committer | Felix Fietkau <nbd@openwrt.org> | 2011-01-23 21:55:41 +0300 |
commit | 71f0be5e11b773ed4b09d1cc46099cdafacbd56a (patch) | |
tree | 8390327e91131228fbe02d1708032d9de502fa72 /blobmsg.c | |
parent | 63cea8dcb79be7bf2bd32b7c05ac289a473192a5 (diff) |
add blobmsg validation function
Diffstat (limited to 'blobmsg.c')
-rw-r--r-- | blobmsg.c | 28 |
1 files changed, 21 insertions, 7 deletions
@@ -15,6 +15,26 @@ #include "blobmsg.h" +bool blobmsg_check_attr(const struct blob_attr *attr, bool name) +{ + const struct blobmsg_hdr *hdr; + + if (blob_len(attr) < sizeof(struct blobmsg_hdr)) + return false; + + hdr = (void *) attr->data; + if (!hdr->namelen && name) + return false; + + if (hdr->namelen > blob_len(attr)) + return false; + + if (hdr->name[hdr->namelen] != 0) + return false; + + return true; +} + int blobmsg_parse(const struct blobmsg_policy *policy, int policy_len, struct blob_attr **tb, void *data, int len) { @@ -45,13 +65,7 @@ int blobmsg_parse(const struct blobmsg_policy *policy, int policy_len, if (hdr->namelen != pslen[i]) continue; - if (!hdr->namelen) - return -1; - - if (sizeof(*attr) + blobmsg_hdrlen(hdr->namelen) > blob_pad_len(attr)) - return -1; - - if (hdr->name[hdr->namelen] != 0) + if (!blobmsg_check_attr(attr, true)) return -1; if (tb[i]) |