[libvirt] [PATCH v2 0/4] Fix a SIGSEGV in libvirtd when querying AMD SEV info

Erik Skultety posted 4 patches 5 years, 10 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1534414409.git.eskultet@redhat.com
Test syntax-check passed
src/conf/domain_capabilities.h                     |   4 +
src/qemu/qemu_capabilities.c                       | 127 +++++++++++++++++++--
src/qemu/qemu_monitor_json.c                       |  31 +++--
tests/domaincapsschemadata/qemu_2.12.0.x86_64.xml  |   5 +-
tests/qemucapabilitiesdata/caps_2.12.0.x86_64.xml  |   6 +
tests/qemucapabilitiesdata/caps_3.0.0.x86_64.xml   |   1 -
...args => launch-security-sev.x86_64-2.12.0.args} |  19 +--
tests/qemuxml2argvtest.c                           |   4 +-
8 files changed, 164 insertions(+), 33 deletions(-)
rename tests/qemuxml2argvdata/{launch-security-sev.args => launch-security-sev.x86_64-2.12.0.args} (54%)
[libvirt] [PATCH v2 0/4] Fix a SIGSEGV in libvirtd when querying AMD SEV info
Posted by Erik Skultety 5 years, 10 months ago
This series fixes the following BZ:
https://bugzilla.redhat.com/show_bug.cgi?id=1612009

TL;DR:
We don't format SEV platform data (PDH, certificate chain,...) into our qemu
caps cache which poses a problem after libvirtd restart when we restore from
the cache and get a segfault upon issuing virNodeGetSEVInfo.

Since v1:
- reworked patch 3 so that qemuMonitorJSONGetSEVCapabilities returns more
values in order to distinguish between error and whether SEV is actually
supported
- added the missing backtrace to patch 4
- no other patches were changed apart from patch 3

Erik Skultety (4):
  tests: sev: Test launch-security with specific QEMU version
  qemu: Define and use a auto cleanup function with virSEVCapability
  qemu: Fix probing of AMD SEV support
  qemu: caps: Format SEV platform data into qemuCaps cache

 src/conf/domain_capabilities.h                     |   4 +
 src/qemu/qemu_capabilities.c                       | 127 +++++++++++++++++++--
 src/qemu/qemu_monitor_json.c                       |  31 +++--
 tests/domaincapsschemadata/qemu_2.12.0.x86_64.xml  |   5 +-
 tests/qemucapabilitiesdata/caps_2.12.0.x86_64.xml  |   6 +
 tests/qemucapabilitiesdata/caps_3.0.0.x86_64.xml   |   1 -
 ...args => launch-security-sev.x86_64-2.12.0.args} |  19 +--
 tests/qemuxml2argvtest.c                           |   4 +-
 8 files changed, 164 insertions(+), 33 deletions(-)
 rename tests/qemuxml2argvdata/{launch-security-sev.args => launch-security-sev.x86_64-2.12.0.args} (54%)

--
2.14.4

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list