blobmsg: simplify and fix name length checks in blobmsg_check_name
authorFelix Fietkau <nbd@nbd.name>
Mon, 25 May 2020 12:49:35 +0000 (14:49 +0200)
committerFelix Fietkau <nbd@nbd.name>
Tue, 26 May 2020 08:06:53 +0000 (10:06 +0200)
blobmsg_hdr_valid_namelen was omitted when name==false
The blob_len vs blobmsg_namelen changes were not taking into account
potential padding between name and data

Signed-off-by: Felix Fietkau <nbd@nbd.name>
blobmsg.c

index daaa9fc8444b7320acb8a48d5bdfb0ece6d734cc..308bef7bc6b052bf708308b6789e96b514c88bbb 100644 (file)
--- a/blobmsg.c
+++ b/blobmsg.c
@@ -48,8 +48,8 @@ static bool blobmsg_hdr_valid_namelen(const struct blobmsg_hdr *hdr, size_t len)
 
 static bool blobmsg_check_name(const struct blob_attr *attr, size_t len, bool name)
 {
-       char *limit = (char *) attr + len;
        const struct blobmsg_hdr *hdr;
+       uint16_t namelen;
 
        hdr = blobmsg_hdr_from_blob(attr, len);
        if (!hdr)
@@ -58,16 +58,11 @@ static bool blobmsg_check_name(const struct blob_attr *attr, size_t len, bool na
        if (name && !hdr->namelen)
                return false;
 
-       if (name && !blobmsg_hdr_valid_namelen(hdr, len))
-               return false;
-
-       if ((char *) hdr->name + blobmsg_namelen(hdr) + 1 > limit)
-               return false;
-
-       if (blobmsg_namelen(hdr) > (blob_len(attr) - sizeof(struct blobmsg_hdr)))
+       namelen = blobmsg_namelen(hdr);
+       if (blob_len(attr) < (size_t)blobmsg_hdrlen(namelen))
                return false;
 
-       if (hdr->name[blobmsg_namelen(hdr)] != 0)
+       if (hdr->name[namelen] != 0)
                return false;
 
        return true;