]> git.proxmox.com Git - mirror_qemu.git/commit
crypto: Remove use of GCRYPT_VERSION macro.
authorRichard W.M. Jones <rjones@redhat.com>
Wed, 27 May 2020 09:34:09 +0000 (10:34 +0100)
committerDaniel P. Berrangé <berrange@redhat.com>
Mon, 15 Jun 2020 10:33:51 +0000 (11:33 +0100)
commitd6cca8e111696fbbd7c233dc53f9c80b6a43359d
tree859d35430ae648b8714a130ddc704cc72ced0c44
parent92500362210268faf4ae81644a66a7616d6bde1d
crypto: Remove use of GCRYPT_VERSION macro.

According to the gcrypt documentation it's intended that
gcry_check_version() is called with the minimum version of gcrypt
needed by the program, not the version from the <gcrypt.h> header file
that happened to be installed when qemu was compiled.  Indeed the
gcrypt.h header says that you shouldn't use the GCRYPT_VERSION macro.

This causes the following failure:

  qemu-img: Unable to initialize gcrypt

if a slightly older version of libgcrypt is installed with a newer
qemu, even though the slightly older version works fine.  This can
happen with RPM packaging which uses symbol versioning to determine
automatically which libgcrypt is required by qemu, which caused the
following bug in RHEL 8:

  https://bugzilla.redhat.com/show_bug.cgi?id=1840485

qemu actually requires libgcrypt >= 1.5.0, so we might put the string
"1.5.0" here.  However since 1.5.0 was released in 2011, it hardly
seems we need to check that.  So I replaced GCRYPT_VERSION with NULL.
Perhaps in future if we move to requiring a newer version of gcrypt we
could put a literal string here.

Signed-off-by: Richard W.M. Jones <rjones@redhat.com>
Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
crypto/init.c