On Tue 30 Jun 2020 12:19:42 PM CEST, Max Reitz wrote: >> @@ -537,8 +542,6 @@ int qcow2_get_cluster_offset(BlockDriverState *bs, >> uint64_t offset, >> bytes_needed = bytes_available; >> } >> >> - *cluster_offset = 0; >> - > > You drop this line without replacement now. That means that > *host_offset is no longer set to 0 if the L1 entry is out of bounds or > empty (which causes this function to return QCOW2_CLUSTER_UNALLOCATED > and no error). Was that intentional?
Hmm, no, it wasn't intentional. It does not have any side effect but I should be explicitly set it to 0. I'll fix it in the next version. Berto
