add missing msgpack_unpacker_free(), and a "clean" assertion avoiding side effects

This commit is contained in:
NiteHawk 2015-08-18 12:24:16 +02:00
parent 1788d6ce01
commit 720c18bcf8

View File

@ -46,8 +46,8 @@ size_t receiver_to_unpacker(receiver* r, size_t request_size,
{ {
// make sure there's enough room, or expand the unpacker accordingly // make sure there's enough room, or expand the unpacker accordingly
if (msgpack_unpacker_buffer_capacity(unpacker) < request_size) { if (msgpack_unpacker_buffer_capacity(unpacker) < request_size) {
bool expanded = msgpack_unpacker_reserve_buffer(unpacker, request_size); msgpack_unpacker_reserve_buffer(unpacker, request_size);
assert(expanded); assert(msgpack_unpacker_buffer_capacity(unpacker) >= request_size);
} }
size_t recv_len = receiver_recv(r, msgpack_unpacker_buffer(unpacker), size_t recv_len = receiver_recv(r, msgpack_unpacker_buffer(unpacker),
request_size); request_size);
@ -92,6 +92,7 @@ void unpack(receiver* r) {
} }
} }
msgpack_unpacked_destroy(&result); msgpack_unpacked_destroy(&result);
msgpack_unpacker_free(unp);
} }
int main(void) { int main(void) {