On Wed, Jan 25, 2017 at 07:24:48PM +0100, Max Reitz wrote: > On 25.01.2017 18:22, Jeff Cody wrote: > > In bdrv_find_backing_image(), if we are searching an image for a backing > > file that contains a protocol, we currently only compare unmodified > > paths. > > > > However, some management software will change the backing filename to be > > a relative filename in a path. QEMU is able to handle this fine, > > because internally it will use path_combine to put together the full > > protocol URI. > > > > However, this can lead to an inability to match an image during a QAPI > > command that needs to use bdrv_find_backing_image() to find the image, > > when it is searched by the full URI. > > > > When searching for a protocol filename, if the straight comparison > > fails, this patch will also compare against the full backing filename to > > see if that is a match. > > > > Signed-off-by: Jeff Cody <jc...@redhat.com> > > --- > > block.c | 12 ++++++++++++ > > 1 file changed, 12 insertions(+) > > Thanks, applied to my block tree: > > https://github.com/XanClic/qemu/commits/block >
Thanks! > > How much would you mind writing an iotest? I don't mind, I can do that. -Jeff
