[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] iotests: Fix CID for VMDK afl image
From: |
Fam Zheng |
Subject: |
Re: [Qemu-devel] [PATCH] iotests: Fix CID for VMDK afl image |
Date: |
Tue, 30 Jan 2018 14:26:24 +0800 |
On Mon, Jan 29, 2018 at 11:49 PM, Kevin Wolf <address@hidden> wrote:
> Am 24.01.2018 um 05:42 hat Fam Zheng geschrieben:
>> The descriptor block in the image, which includes the CID to verify, has been
>> invalid since the reference image was added. Since commit 9877860e7bd we
>> report
>> this error earlier than the "file too large", so 059.out mismatches.
>>
>> Instead of fixing the output, we fix the image descriptor, because the code
>> path we want to exercise is actually the invalid image size.
>>
>> The binary change is generated along the operations of:
>>
>> $ bunzip2 afl9.vmdk.bz2
>> $ qemu-img create -f qcow2 fix.vmdk 1G
>
> Sure about -f qcow2? :-)
Typo when writing up the commit message, will fix.
>
>> $ dd if=afl9.vmdk.bz2 of=fix.vmdk bs=512 count=1 conv=notrunc
>> $ mv fix.vmdk afl9.vmdk
>> $ bzip2 afl9.vmdk
>>
>> Signed-off-by: Fam Zheng <address@hidden>
>> ---
>> tests/qemu-iotests/sample_images/afl9.vmdk.bz2 | Bin 178 -> 618 bytes
>> 1 file changed, 0 insertions(+), 0 deletions(-)
>
> I think you also want to revert 76bf133c in this patch, otherwise 059
> fails after applying it.
Yeah, sending another version with that.
Fam