[PATCH 1/9] util: storagefile: Drop image format probing by file suffix
Eric Blake
eblake at redhat.com
Wed Feb 19 15:18:40 UTC 2020
On 2/17/20 11:13 AM, Peter Krempa wrote:
> Probing by file suffix was meant to be a last resort if probing by
> contents fails or is not supported. For most formats we never specified
> any suffix. There's a few formats implementing both magic bytes and
> suffix and finally DMG which had only suffix probing. Since suffix
> probing is nowhere reliable and only one format depends on in whic has a
which
> comment that qemu doesn't do the probing either drop the whole
> infrastructure.
>
> Signed-off-by: Peter Krempa <pkrempa at redhat.com>
> ---
> src/util/virstoragefile.c | 58 ++++++++++++---------------------------
> 1 file changed, 17 insertions(+), 41 deletions(-)
>
> @@ -324,50 +322,50 @@ static struct FileTypeInfo const fileTypeInfo[] = {
> /* XXX QEMU says there's no magic for dmg,
> * /usr/share/misc/magic lists double magic (both offsets
> * would have to match) but then disables that check. */
> - 0, NULL, ".dmg",
> + 0, NULL,
> 0, -1, 0, {0},
> -1, 0, 0, NULL, NULL, NULL
> },
Is it even worth keeping the dmg entry around? But deleting it (if
appropriate) should be separate from this patch.
Reviewed-by: Eric Blake <eblake at redhat.com>
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3226
Virtualization: qemu.org | libvirt.org
More information about the libvir-list
mailing list