bplist: Prevent OOB read when parsing data/string/array/dict size nodes
As reported in #91, the code that will read the big endian integer value
of variable size did not check if the actual number of bytes is still
withing the range of the actual plist data.
This commit fixes the issue with proper bounds checking.
diff --git a/src/bplist.c b/src/bplist.c
index 1a40556..0cfe5fe 100644
--- a/src/bplist.c
+++ b/src/bplist.c
@@ -530,6 +530,8 @@
return NULL;
(*object)++;
next_size = 1 << next_size;
+ if (*object + next_size >= bplist->data + bplist->size)
+ return NULL;
size = UINT_TO_HOST(*object, next_size);
(*object) += next_size;
break;
