[libvirt] [PATCH 0/3] Fix bitmap parsing code and add tests

Peter Krempa pkrempa at redhat.com
Fri Aug 16 12:41:27 UTC 2013


On 08/16/13 12:39, Daniel P. Berrange wrote:
> On Fri, Aug 16, 2013 at 12:32:06PM +0200, Peter Krempa wrote:
>> The bitmap parsing code might cause a crash of the application using it.
>> Fix it and add tests so that it doesn't happen again.
>>
>> Peter Krempa (3):
>>   virbitmap: Refactor virBitmapParse to avoid access beyond bounds of
>>     array
>>   virbitmaptest: Fix function header formatting
>>   virbitmaptest: Add test for out of bounds condition
>>
>>  src/util/virbitmap.c  | 38 +++++++++++++-------------------
>>  tests/virbitmaptest.c | 60 ++++++++++++++++++++++++++++++++++++++++++++-------
>>  2 files changed, 67 insertions(+), 31 deletions(-)
> 
> ACK to all 3. I was just going to suggest adding tests to Alex,
> when I saw your followup.

Thanks, I've fixed the commit message of patch 3/3 according to Eric's
suggestion and added info about the commit introducing the problem to
1/3 and pushed.

Peter


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 901 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20130816/8080303d/attachment-0001.sig>


More information about the libvir-list mailing list