[PATCH 4/4] util: qcow2GetExtensions: Remove support for 'data file' extension
Peter Krempa
pkrempa at redhat.com
Fri Apr 24 15:18:45 UTC 2020
On Fri, Apr 24, 2020 at 09:52:00 -0500, Eric Blake wrote:
> On 4/24/20 4:24 AM, Peter Krempa wrote:
> > The implementation was never finished in libvirt. Remove it.
> >
> > Signed-off-by: Peter Krempa <pkrempa at redhat.com>
> > ---
> > src/util/virstoragefile.c | 19 ++-----------------
> > 1 file changed, 2 insertions(+), 17 deletions(-)
>
> Shouldn't we at least recognize if there is a qcow2 file with the extension
> header set, at which point we can error out and tell the user their image is
> unsupported, rather than trying to use it without the data file?
I don't think it's scalable to do so unless qemu exposes a whitelist of
supported extensions we can declare we support. Otherwise we are
band-aiding stuff we lear that exists and ignore any other extensions.
More information about the libvir-list
mailing list