make the blobmsg format endian agnostic (stick to big-endian)

[project/libubox.git] / blobmsg.h

diff --git a/blobmsg.h b/blobmsg.h
index e51c39d..c266855 100644 (file)
--- a/blobmsg.h
+++ b/blobmsg.h
@@ -31,7 +31,8 @@ enum blobmsg_type {
        BLOBMSG_TYPE_INT16,
        BLOBMSG_TYPE_INT8,
        __BLOBMSG_TYPE_LAST,
-       BLOBMSG_TYPE_LAST = __BLOBMSG_TYPE_LAST - 1
+       BLOBMSG_TYPE_LAST = __BLOBMSG_TYPE_LAST - 1,
+       BLOBMSG_TYPE_BOOL = BLOBMSG_TYPE_INT8,
 };
 
 struct blobmsg_hdr {
@@ -55,10 +56,15 @@ static inline const char *blobmsg_name(const struct blob_attr *attr)
        return (const char *) hdr->name;
 }
 
+static inline int blobmsg_type(const struct blob_attr *attr)
+{
+       return blob_id(attr);
+}
+
 static inline void *blobmsg_data(const struct blob_attr *attr)
 {
        struct blobmsg_hdr *hdr = blob_data(attr);
-       return (char *) hdr + blobmsg_hdrlen(hdr->namelen);
+       return (char *) hdr + blobmsg_hdrlen(be16_to_cpu(hdr->namelen));
 }
 
 static inline int blobmsg_data_len(const struct blob_attr *attr)
@@ -144,6 +150,11 @@ static inline uint8_t blobmsg_get_u8(struct blob_attr *attr)
        return *(uint8_t *) blobmsg_data(attr);
 }
 
+static inline bool blobmsg_get_bool(struct blob_attr *attr)
+{
+       return *(uint8_t *) blobmsg_data(attr);
+}
+
 static inline uint16_t blobmsg_get_u16(struct blob_attr *attr)
 {
        return *(uint16_t *) blobmsg_data(attr);
@@ -162,11 +173,11 @@ static inline uint64_t blobmsg_get_u64(struct blob_attr *attr)
 void *blobmsg_alloc_string_buffer(struct blob_buf *buf, const char *name, int maxlen);
 void blobmsg_add_string_buffer(struct blob_buf *buf);
 
-char *blobmsg_format_json(struct blob_attr *attr, bool list);
+/* blobmsg to json formatting */
 
 #define blobmsg_for_each_attr(pos, attr, rem) \
        for (rem = blobmsg_data_len(attr), pos = blobmsg_data(attr); \
-                (blob_pad_len(pos) <= rem) && \
+                rem > 0 && (blob_pad_len(pos) <= rem) && \
                 (blob_pad_len(pos) >= sizeof(struct blob_attr)); \
                 rem -= blob_pad_len(pos), pos = blob_next(pos))