blobmsg: make blobmsg_len and blobmsg_data_len return unsigned value
authorPetr Štetiar <ynezz@true.cz>
Tue, 10 Dec 2019 10:53:23 +0000 (11:53 +0100)
committerPetr Štetiar <ynezz@true.cz>
Wed, 25 Dec 2019 09:31:58 +0000 (10:31 +0100)
One usually doesn't guard against negative length values in the code.

Signed-off-by: Petr Štetiar <ynezz@true.cz>
blobmsg.c
blobmsg.h

index 97e0c20575ff17e5bc549113964fd7bef1df3d0b..1a8b783e9ba92b35aff2b1a3ad6912a903b704e1 100644 (file)
--- a/blobmsg.c
+++ b/blobmsg.c
@@ -35,7 +35,8 @@ bool blobmsg_check_attr(const struct blob_attr *attr, bool name)
 {
        const struct blobmsg_hdr *hdr;
        const char *data;
-       int id, len;
+       size_t len;
+       int id;
 
        if (blob_len(attr) < sizeof(struct blobmsg_hdr))
                return false;
index b06ef597d8c6d836d9eecb9227e6825576ada13e..0af0878900009d0d481828ff59f0fbab59454ec6 100644 (file)
--- a/blobmsg.h
+++ b/blobmsg.h
@@ -86,7 +86,7 @@ static inline void *blobmsg_data(const struct blob_attr *attr)
        return data;
 }
 
-static inline int blobmsg_data_len(const struct blob_attr *attr)
+static inline size_t blobmsg_data_len(const struct blob_attr *attr)
 {
        uint8_t *start, *end;
 
@@ -99,7 +99,7 @@ static inline int blobmsg_data_len(const struct blob_attr *attr)
        return blob_len(attr) - (end - start);
 }
 
-static inline int blobmsg_len(const struct blob_attr *attr)
+static inline size_t blobmsg_len(const struct blob_attr *attr)
 {
        return blobmsg_data_len(attr);
 }