X-Git-Url: http://git.openwrt.org/?a=blobdiff_plain;f=blobmsg.c;h=1e93376a27b65cf816f94f87738b98e8bb497ef6;hb=4a9f74f7ae6c092c731969c93f90ab4d6a07b524;hp=f31715209013c31a2ec7d0f493fca4c9e6c8b284;hpb=d07b174de8faf76a20cbe003bfc143552f431f85;p=project%2Flibubox.git diff --git a/blobmsg.c b/blobmsg.c index f317152..1e93376 100644 --- a/blobmsg.c +++ b/blobmsg.c @@ -62,11 +62,12 @@ bool blobmsg_check_attr(const struct blob_attr *attr, bool name) return blob_check_type(data, len, blob_type[id]); } -bool blobmsg_check_attr_list(const struct blob_attr *attr, int type) +int blobmsg_check_array(const struct blob_attr *attr, int type) { struct blob_attr *cur; bool name; int rem; + int size = 0; switch (blobmsg_type(attr)) { case BLOBMSG_TYPE_TABLE: @@ -76,18 +77,25 @@ bool blobmsg_check_attr_list(const struct blob_attr *attr, int type) name = false; break; default: - return false; + return -1; } blobmsg_for_each_attr(cur, attr, rem) { - if (blobmsg_type(cur) != type) - return false; + if (type != BLOBMSG_TYPE_UNSPEC && blobmsg_type(cur) != type) + return -1; if (!blobmsg_check_attr(cur, name)) - return false; + return -1; + + size++; } - return true; + return size; +} + +bool blobmsg_check_attr_list(const struct blob_attr *attr, int type) +{ + return blobmsg_check_array(attr, type) >= 0; } int blobmsg_parse_array(const struct blobmsg_policy *policy, int policy_len, @@ -212,34 +220,45 @@ blobmsg_open_nested(struct blob_buf *buf, const char *name, bool array) name = ""; head = blobmsg_new(buf, type, name, 0, &data); + if (!head) + return NULL; blob_set_raw_len(buf->head, blob_pad_len(buf->head) - blobmsg_hdrlen(strlen(name))); buf->head = head; return (void *)offset; } -void +int blobmsg_vprintf(struct blob_buf *buf, const char *name, const char *format, va_list arg) { va_list arg2; char cbuf; - int len; + char *sbuf; + int len, ret; va_copy(arg2, arg); len = vsnprintf(&cbuf, sizeof(cbuf), format, arg2); va_end(arg2); - vsprintf(blobmsg_alloc_string_buffer(buf, name, len + 1), format, arg); + sbuf = blobmsg_alloc_string_buffer(buf, name, len + 1); + if (!sbuf) + return -1; + ret = vsprintf(sbuf, format, arg); blobmsg_add_string_buffer(buf); + + return ret; } -void +int blobmsg_printf(struct blob_buf *buf, const char *name, const char *format, ...) { va_list ap; + int ret; va_start(ap, format); - blobmsg_vprintf(buf, name, format, ap); + ret = blobmsg_vprintf(buf, name, format, ap); va_end(ap); + + return ret; } void * @@ -252,7 +271,6 @@ blobmsg_alloc_string_buffer(struct blob_buf *buf, const char *name, unsigned int if (!attr) return NULL; - data_dest = blobmsg_data(attr); blob_set_raw_len(buf->head, blob_pad_len(buf->head) - blob_pad_len(attr)); blob_set_raw_len(attr, blob_raw_len(attr) - maxlen); @@ -269,7 +287,8 @@ blobmsg_realloc_string_buffer(struct blob_buf *buf, unsigned int maxlen) if (required <= 0) goto out; - blob_buf_grow(buf, required); + if (!blob_buf_grow(buf, required)) + return NULL; attr = blob_next(buf->head); out: