Am 27.09.2022 um 09:54 hat Markus Armbruster geschrieben: > Potential issue before this patch: I can't see bdrv_open_child() being > undone. Shouldn't we close bs->file? And what about > bdrv_open_child()'s side effect on @options?
bs->file is handled by the caller, bdrv_open_driver(). options is going to be freed anyway, so I don't think we care about its content. And anyway, doesn't bdrv_open_child() only remove those entries from it that it processed, like in the success case? Kevin