blobmsg: add missing endian conversions
authorFelix Fietkau <nbd@openwrt.org>
Wed, 16 Nov 2011 23:47:49 +0000 (00:47 +0100)
committerFelix Fietkau <nbd@openwrt.org>
Wed, 16 Nov 2011 23:47:54 +0000 (00:47 +0100)
blobmsg.h
blobmsg_json.c

index b4e6758a4951a8c6a6849844675c039ab4d15453..28135763795c7e4c7a8aaf8f511ddc9a5b6ee2c1 100644 (file)
--- a/blobmsg.h
+++ b/blobmsg.h
@@ -93,18 +93,21 @@ blobmsg_add_u8(struct blob_buf *buf, const char *name, uint8_t val)
 static inline int
 blobmsg_add_u16(struct blob_buf *buf, const char *name, uint16_t val)
 {
+       val = cpu_to_be16(val);
        return blobmsg_add_field(buf, BLOBMSG_TYPE_INT16, name, &val, 2);
 }
 
 static inline int
 blobmsg_add_u32(struct blob_buf *buf, const char *name, uint32_t val)
 {
+       val = cpu_to_be32(val);
        return blobmsg_add_field(buf, BLOBMSG_TYPE_INT32, name, &val, 4);
 }
 
 static inline int
 blobmsg_add_u64(struct blob_buf *buf, const char *name, uint64_t val)
 {
+       val = cpu_to_be64(val);
        return blobmsg_add_field(buf, BLOBMSG_TYPE_INT64, name, &val, 8);
 }
 
@@ -157,17 +160,17 @@ static inline bool blobmsg_get_bool(struct blob_attr *attr)
 
 static inline uint16_t blobmsg_get_u16(struct blob_attr *attr)
 {
-       return *(uint16_t *) blobmsg_data(attr);
+       return be16_to_cpu(*(uint16_t *) blobmsg_data(attr));
 }
 
 static inline uint32_t blobmsg_get_u32(struct blob_attr *attr)
 {
-       return *(uint32_t *) blobmsg_data(attr);
+       return be32_to_cpu(*(uint32_t *) blobmsg_data(attr));
 }
 
 static inline uint64_t blobmsg_get_u64(struct blob_attr *attr)
 {
-       return *(uint64_t *) blobmsg_data(attr);
+       return be64_to_cpu(*(uint64_t *) blobmsg_data(attr));
 }
 
 void *blobmsg_alloc_string_buffer(struct blob_buf *buf, const char *name, int maxlen);
index dd03736a150d3eadf94624f574f8a0b037660cd6..96bfe28936d5f686d9712aae9d050fba9ae12785 100644 (file)
@@ -232,13 +232,13 @@ static void blobmsg_format_element(struct strbuf *s, struct blob_attr *attr, boo
                sprintf(buf, "%s", *(uint8_t *)data ? "true" : "false");
                break;
        case BLOBMSG_TYPE_INT16:
-               sprintf(buf, "%d", *(uint16_t *)data);
+               sprintf(buf, "%d", be16_to_cpu(*(uint16_t *)data));
                break;
        case BLOBMSG_TYPE_INT32:
-               sprintf(buf, "%d", *(uint32_t *)data);
+               sprintf(buf, "%d", be32_to_cpu(*(uint32_t *)data));
                break;
        case BLOBMSG_TYPE_INT64:
-               sprintf(buf, "%lld", (long long int) *(uint64_t *)data);
+               sprintf(buf, "%lld", (long long int) be64_to_cpu(*(uint64_t *)data));
                break;
        case BLOBMSG_TYPE_STRING:
                blobmsg_format_string(s, data);