Re: [Qemu-devel] [PATCH] qcow2: Fix offset in qcow2_read_extensions

2012-02-28 Thread Stefan Hajnoczi
On Mon, Feb 27, 2012 at 4:27 PM, Kevin Wolf wrote: > The spec says that the length of extensions is padded to 8 bytes, not > the offset. Currently this is the same because the header size is a > multiple of 8, so this is only about compatibility with future changes > to the header size. > > While

[Qemu-devel] [PATCH] qcow2: Fix offset in qcow2_read_extensions

2012-02-27 Thread Kevin Wolf
The spec says that the length of extensions is padded to 8 bytes, not the offset. Currently this is the same because the header size is a multiple of 8, so this is only about compatibility with future changes to the header size. While touching it, move the calculation to a common place instead of