On Thu, Jan 12, 2012 at 12:17 PM, Kevin Wolf <kw...@redhat.com> wrote: > Am 06.01.2012 15:01, schrieb Stefan Hajnoczi: >> +BlockDriverState *bdrv_find_backing_image(BlockDriverState *bs, const char >> *id) >> +{ >> + if (!bs->drv) { >> + return NULL; >> + } >> + >> + if (bs->backing_hd) { >> + if (strcmp(bs->backing_file, id) == 0) { >> + return bs->backing_hd; > > So it's not really just some id, but the backing file name? I would find > it clearer to reflect that in the parameter name and the QMP error in > the next patch.
Okay, thanks.