Re: [Qemu-devel] [PATCH 2/2] qapi: Fix memleak in string visitors on int lists

2016-05-31 Thread Markus Armbruster
[I accidentally sent this just to Eric, resending to list...] Eric Blake writes: > Commit 7f8f9ef1 introduced the ability to store a list of > integers as a sorted list of ranges, but when merging ranges, > it leaks one or more ranges. It was also using range_get_last() > incorrectly within ran

[Qemu-devel] [PATCH 2/2] qapi: Fix memleak in string visitors on int lists

2016-05-16 Thread Eric Blake
Commit 7f8f9ef1 introduced the ability to store a list of integers as a sorted list of ranges, but when merging ranges, it leaks one or more ranges. It was also using range_get_last() incorrectly within range_compare() (a range is a start/end pair, but range_get_last() is for start/len pairs), and