13.06.2019 1:09, Max Reitz wrote: > Places that use patterns like > > if (bs->drv->is_filter && bs->file) { > ... something about bs->file->bs ... > } > > should be > > BlockDriverState *filtered = bdrv_filtered_rw_bs(bs); > if (filtered) { > ... something about @filtered ... > } > > instead.
Hmm, in other words, support filters with backing child in all places, where only file-based filters are supported, as we don't want make any semantic difference between these two types of filters. > > Signed-off-by: Max Reitz <mre...@redhat.com> Reviewed-by: Vladimir Sementsov-Ogievskiy <vsement...@virtuozzo.com> -- Best regards, Vladimir