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 <address@hidden>
---
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.