Re: [Qemu-devel] [PATCH v2 09/12] qcow2/overlaps: Protect snapshot table

[Max Reitz <mreitz@redhat.com>]

On 2015-02-05 at 10:29, Eric Blake wrote:
> On 11/24/2014 08:56 AM, Max Reitz wrote:
>> Keep track of the snapshot table in the metadata list to protect it
>> against accidental modifications.
>>
>> Signed-off-by: Max Reitz <mreitz@redhat.com>
>> ---
>>   block/qcow2-snapshot.c | 10 ++++++++++
>>   block/qcow2.c          |  6 ++++++
>>   2 files changed, 16 insertions(+)
>>
>> +    if (header.nb_snapshots) {
>> +        qcow2_metadata_list_enter(bs, header.snapshots_offset,
>> +                                  size_to_clusters(s, header.nb_snapshots *
>> +                                                   sizeof(QCowSnapshotHeader)),
> In other patches in this series, you had been aligning the sizeof() with
> the parameter expression it continues, rather than the first parameter.
>
> blah(s, count *
>          sizeof(struct))
>
> vs.
>
> blah(s, count *
>       sizeof(struct))
>
>
> I don't care either way, just wanted to point it out, in case you care.

I do care. :-)

In this case I didn't align it because that would have broken the 80 
characters limit.

Mayb