On Thu 02 Jul 2020 02:46:27 PM CEST, Max Reitz wrote:
>> - /* must be allocated */
>> - assert(first_cluster_type == QCOW2_CLUSTER_NORMAL ||
>> - first_cluster_type == QCOW2_CLUSTER_ZERO_ALLOC);
>> + assert(*l2_index + nb_clusters <= s->l2_size);
>
> Not l2_slice_size?
Oh, indeed!
>> + } else if (check_offset) {
>
> My gcc (v10.1.1) appears to be a bit daft, and so doesn’t recognize
> that check_offset must always be initialized before this line is hit.
Yeah I noticed that patchew complained, I'll fix that.
Berto
