]> git.ipfire.org Git - thirdparty/libvirt.git/commit
qemuProcessCreatePretendCmd: Rework FIPS handling
authorMichal Privoznik <mprivozn@redhat.com>
Mon, 25 Apr 2016 05:47:59 +0000 (07:47 +0200)
committerMichal Privoznik <mprivozn@redhat.com>
Mon, 25 Apr 2016 16:47:31 +0000 (18:47 +0200)
commit927d047ee7ff4ebc27a95fd47f8abc6031d870bd
tree5749f1bb74b27484c27496ed4bb044318a730d5d
parentff2126225df019566e4e580d92e69e9df3ca96c0
qemuProcessCreatePretendCmd: Rework FIPS handling

This function - in contrast with qemuBuildCommandLine - merely
constructs our internal command representation of a domain. This
is then later compared against expected output. Or, this function
is used also in virConnectDomainXMLToNative(). But due to a copy
paste error this function, just like its image - has @forceFips
argument that if enabled forces FIPS, otherwise mimics FIPS state
in the host. If FIPS is enabled or forced the generated command
line is different to state in which FIPS is disabled. Problem is,
while this could be desired in the virConnectDomainXMLToNative()
case, this is undesirable in the test suite as it will produce
unpredicted results.
Solution to this is to rename argument to @enableFips to
specifically tell whether we expect command line to be build in
either of fashions and make virConnectDomainXMLToNative()
implementation fetch FIPS state and pass it to
qemuProcessCreatePretendCmd().

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
src/qemu/qemu_driver.c
src/qemu/qemu_process.c
src/qemu/qemu_process.h