diff options
| author | Daniel P. Berrangé | 2020-10-20 18:08:27 +0200 |
|---|---|---|
| committer | Daniel P. Berrangé | 2021-01-29 18:07:53 +0100 |
| commit | 166310299a1e7824bbff17e1f016659d18b4a559 (patch) | |
| tree | d4832fe55bca945eb17a72832922cfda7ab981cd /os-posix.c | |
| parent | crypto: Fix memory leaks in set_loaded for tls-* (diff) | |
| download | qemu-166310299a1e7824bbff17e1f016659d18b4a559.tar.gz qemu-166310299a1e7824bbff17e1f016659d18b4a559.tar.xz qemu-166310299a1e7824bbff17e1f016659d18b4a559.zip | |
os: deprecate the -enable-fips option and QEMU's FIPS enforcement
The -enable-fips option was added a long time ago to prevent the use of
single DES when VNC when FIPS mode is enabled. It should never have been
added, because apps are supposed to unconditionally honour FIPS mode
based on the '/proc/sys/crypto/fips_enabled' file contents.
In addition there is more to achieving FIPS compliance than merely
blocking use of certain algorithms. Those algorithms which are used
need to perform self-tests at runtime.
QEMU's built-in cryptography provider has no support for self-tests,
and neither does the nettle library.
If QEMU is required to be used in a FIPS enabled host, then it must be
built with the libgcrypt library enabled, which will unconditionally
enforce FIPS compliance in any algorithm usage.
Thus there is no need to keep either the -enable-fips option in QEMU, or
QEMU's internal FIPS checking methods.
Reviewed-by: John Snow <jsnow@redhat.com>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
Diffstat (limited to 'os-posix.c')
| -rw-r--r-- | os-posix.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/os-posix.c b/os-posix.c index 1de2839554..a6846f51c1 100644 --- a/os-posix.c +++ b/os-posix.c @@ -153,6 +153,9 @@ int os_parse_cmd_args(int index, const char *optarg) break; #if defined(CONFIG_LINUX) case QEMU_OPTION_enablefips: + warn_report("-enable-fips is deprecated, please build QEMU with " + "the `libgcrypt` library as the cryptography provider " + "to enable FIPS compliance"); fips_set_state(true); break; #endif |