Hi,
On 15/03/2015 22:00, Luka Perkov wrote:
> wbuf = blobmsg_alloc_string_buffer(&buf, "md5", 33);
> +
> + for (i = 0; i < 16; i++)
> + sprintf((wbuf + (i * 2)), "%02x", (uint8_t) md5[i]);
there is a set of brackets too many here
> +
> + *(wbuf + 33) = 0;
is this not off by 1 ? if we want to be pedantic it should also be '\0'
i think wbuf[32] = '\0'; is a saner syntax
John
_______________________________________________
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel
