- Devel - libvirt List Archives

[libvirt] [PATCH v2] qemu: set fake reboot flag only in acpi mode
by Nikolay Shirokovskiy 02 Aug '16

02 Aug '16

First this fixes the same issue as e2b86f580. Only difference is that reboot is done via shutdown function with reboot semantics. Second 8be502fd tells us that we need to always set fake reboot flag if shutdown/reboot will trigger shutdown event. To put it simple we need to set it false if agent is used. References: e2b86f580 - fixes guest shutdown becames reboot after API reboot thru agent 8be502fd - fixes API shutdown thru agent becames reboot after API reboot thru acpi (more on https://www.redhat.com/archives/libvir-list/2015-April/msg00715.html) --- src/qemu/qemu_driver.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c index 2089359..8855387 100644 --- a/src/qemu/qemu_driver.c +++ b/src/qemu/qemu_driver.c @@ -1997,9 +1997,9 @@ static int qemuDomainShutdownFlags(virDomainPtr dom, unsigned int flags) useAgent = false; } - qemuDomainSetFakeReboot(driver, vm, isReboot); if (useAgent) { + qemuDomainSetFakeReboot(driver, vm, false); qemuDomainObjEnterAgent(vm); ret = qemuAgentShutdown(priv->agent, agentFlag); qemuDomainObjExitAgent(vm); @@ -2018,6 +2018,7 @@ static int qemuDomainShutdownFlags(virDomainPtr dom, unsigned int flags) goto endjob; } + qemuDomainSetFakeReboot(driver, vm, isReboot); qemuDomainObjEnterMonitor(driver, vm); ret = qemuMonitorSystemPowerdown(priv->mon); if (qemuDomainObjExitMonitor(driver, vm) < 0) @@ -2091,6 +2092,7 @@ qemuDomainReboot(virDomainPtr dom, unsigned int flags) } if (useAgent) { + qemuDomainSetFakeReboot(driver, vm, false); qemuDomainObjEnterAgent(vm); ret = qemuAgentShutdown(priv->agent, agentFlag); qemuDomainObjExitAgent(vm); -- 1.8.3.1

2 2

[libvirt] [PATCH 0/2] Fixes for broken build
by John Ferlan 02 Aug '16

02 Aug '16

Pushed under the build breaker rule John Ferlan (2): libxl: Fix broken build from libxlDomainCleanup libxl: Fix broken build attach/detach controller device src/libxl/libxl_domain.c | 2 +- src/libxl/libxl_driver.c | 4 ++++ 2 files changed, 5 insertions(+), 1 deletion(-) -- 2.7.4

1 2

[libvirt] [PATCH V2 0/7] libxl: add support of pvusb controller
by Chunyan Liu 02 Aug '16

02 Aug '16

This patch series is to add pvusb controller support in libxl driver. It should be applied on previous pvusb device support patch series. --- Changes: * drop pvusb1 and pvusb2 model * add check in qemu device post-parse to report error of unsupported 'qusb1' and 'qusb2' model Chunyan Liu (7): extend usb controller model to support xen pvusb libxl: support USB controllers in creation time libxl: support usb controller hotplug libxl: check available controller and port when hotplugging USB device xenconfig: add conversion of usb controller config to and from xml xlconfigtest: add test for usb controller conversion qemuDomainDeviceDefPostParse: add USB controller model check docs/formatdomain.html.in | 4 +- docs/schemas/domaincommon.rng | 2 + src/conf/domain_conf.c | 2 + src/conf/domain_conf.h | 2 + src/libxl/libxl_conf.c | 84 ++++++++++++++++ src/libxl/libxl_conf.h | 4 + src/libxl/libxl_driver.c | 176 +++++++++++++++++++++++++++++++++ src/qemu/qemu_command.c | 2 + src/qemu/qemu_domain.c | 13 +++ src/xenconfig/xen_xl.c | 190 ++++++++++++++++++++++++++++++++++++ tests/xlconfigdata/test-usbctrl.cfg | 13 +++ tests/xlconfigdata/test-usbctrl.xml | 31 ++++++ tests/xlconfigtest.c | 1 + 13 files changed, 523 insertions(+), 1 deletion(-) create mode 100644 tests/xlconfigdata/test-usbctrl.cfg create mode 100644 tests/xlconfigdata/test-usbctrl.xml -- 2.1.4

3 15

[libvirt] [PATCH 0/5] admin: Fix updating of the client limits
by Erik Skultety 02 Aug '16

02 Aug '16

This series slightly refactors the code in virnetserver.c in order to resolve https://bugzilla.redhat.com/show_bug.cgi?id=1357776. The problem resides in updating of the client limits which even though updated, thus "permitting" new connections to be established, do not re-enable polling for the socket file descriptor which results in all new connections still queueing on the socket. Erik Skultety (5): rpc: virnetserver: Rename ClientSetProcessingControls to ClientSetLimits rpc: virnetserver: Move virNetServerCheckLimits which is static up in the file rpc: virnetserver: Add code to CheckLimits to handle suspending of services admin: rpc: virnetserver: Fix updating of the client limits rpc: virnetserver: Remove dead code checking the client limits daemon/admin_server.c | 4 +-- src/rpc/virnetserver.c | 97 +++++++++++++++++++++++--------------------------- src/rpc/virnetserver.h | 6 ++-- 3 files changed, 50 insertions(+), 57 deletions(-) -- 2.5.5

2 8

[libvirt] [PATCH] qemu: only report errno in trace message on failure
by Daniel P. Berrange 02 Aug '16

02 Aug '16

Avoid reporting a stale errno value when the syscall succeeds, instead always pass 0. Signed-off-by: Daniel P. Berrange <berrange(a)redhat.com> --- src/qemu/qemu_monitor.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/qemu/qemu_monitor.c b/src/qemu/qemu_monitor.c index 58c04d5..6c9695e 100644 --- a/src/qemu/qemu_monitor.c +++ b/src/qemu/qemu_monitor.c @@ -528,12 +528,12 @@ qemuMonitorIOWrite(qemuMonitorPtr mon) PROBE(QEMU_MONITOR_IO_WRITE, "mon=%p buf=%s len=%zu ret=%d errno=%d", - mon, buf, len, done, errno); + mon, buf, len, done, done < 0 ? errno : 0); if (mon->msg->txFD != -1) { PROBE(QEMU_MONITOR_IO_SEND_FD, "mon=%p fd=%d ret=%d errno=%d", - mon, mon->msg->txFD, done, errno); + mon, mon->msg->txFD, done, done < 0 ? errno : 0); } if (done < 0) { -- 2.7.4

2 1

[libvirt] [PATCH] Post-release version bump to 2.2.0
by Michal Privoznik 02 Aug '16

02 Aug '16

Signed-off-by: Michal Privoznik <mprivozn(a)redhat.com> --- Pushed under trivial rule. configure.ac | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure.ac b/configure.ac index 2c81c95..8d7d63e 100644 --- a/configure.ac +++ b/configure.ac @@ -16,7 +16,7 @@ dnl You should have received a copy of the GNU Lesser General Public dnl License along with this library. If not, see dnl <http://www.gnu.org/licenses/>. -AC_INIT([libvirt], [2.1.0], [libvir-list(a)redhat.com] [], [http://libvirt.org] +AC_INIT([libvirt], [2.2.0], [libvir-list(a)redhat.com] [], [http://libvirt.org] AC_CONFIG_SRCDIR([src/libvirt.c]) AC_CONFIG_AUX_DIR([build-aux]) AC_CONFIG_HEADERS([config.h]) -- 2.8.4

1 0

[libvirt] [PATCH] Fix unbalanced quotation marks in messages
by Yuri Chornoivan 02 Aug '16

02 Aug '16

Hi, Thanks for reviewing this minor patch that fixes some glitches found while translating libvirt. Best regards, Yuri

2 1

[libvirt] [PATCH] device_conf: Let compiler decide on inlining functions
by Michal Privoznik 02 Aug '16

02 Aug '16

There's no point in forcing some functions to be both 'static' and 'inline' at the same time at the header file level. This leads to a situation where just linking the file defines those function (potentially needlessly). Instead, we should let the compiler (linker) decide whether the function are small enough to be inlined or not. Signed-off-by: Michal Privoznik <mprivozn(a)redhat.com> --- src/conf/device_conf.c | 24 ++++++++++++++++++++++++ src/conf/device_conf.h | 21 +++------------------ src/libvirt_private.syms | 3 +++ 3 files changed, 30 insertions(+), 18 deletions(-) diff --git a/src/conf/device_conf.c b/src/conf/device_conf.c index f58b9d0..f34b6c6 100644 --- a/src/conf/device_conf.c +++ b/src/conf/device_conf.c @@ -78,6 +78,30 @@ int virPCIDeviceAddressIsValid(virPCIDeviceAddressPtr addr, } +bool +virPCIDeviceAddressIsEmpty(const virPCIDeviceAddress *addr) +{ + return !(addr->domain || addr->bus || addr->slot); +} + + +bool +virDeviceInfoPCIAddressWanted(const virDomainDeviceInfo *info) +{ + return info->type == VIR_DOMAIN_DEVICE_ADDRESS_TYPE_NONE || + (info->type == VIR_DOMAIN_DEVICE_ADDRESS_TYPE_PCI && + virPCIDeviceAddressIsEmpty(&info->addr.pci)); +} + + +bool +virDeviceInfoPCIAddressPresent(const virDomainDeviceInfo *info) +{ + return info->type == VIR_DOMAIN_DEVICE_ADDRESS_TYPE_PCI && + !virPCIDeviceAddressIsEmpty(&info->addr.pci); +} + + int virPCIDeviceAddressParseXML(xmlNodePtr node, virPCIDeviceAddressPtr addr) diff --git a/src/conf/device_conf.h b/src/conf/device_conf.h index 8443de6..98eaf3f 100644 --- a/src/conf/device_conf.h +++ b/src/conf/device_conf.h @@ -148,26 +148,11 @@ typedef struct _virDomainDeviceInfo { int virPCIDeviceAddressIsValid(virPCIDeviceAddressPtr addr, bool report); -static inline bool -virPCIDeviceAddressIsEmpty(const virPCIDeviceAddress *addr) -{ - return !(addr->domain || addr->bus || addr->slot); -} +bool virPCIDeviceAddressIsEmpty(const virPCIDeviceAddress *addr); -static inline bool -virDeviceInfoPCIAddressWanted(const virDomainDeviceInfo *info) -{ - return info->type == VIR_DOMAIN_DEVICE_ADDRESS_TYPE_NONE || - (info->type == VIR_DOMAIN_DEVICE_ADDRESS_TYPE_PCI && - virPCIDeviceAddressIsEmpty(&info->addr.pci)); -} +bool virDeviceInfoPCIAddressWanted(const virDomainDeviceInfo *info); -static inline bool -virDeviceInfoPCIAddressPresent(const virDomainDeviceInfo *info) -{ - return info->type == VIR_DOMAIN_DEVICE_ADDRESS_TYPE_PCI && - !virPCIDeviceAddressIsEmpty(&info->addr.pci); -} +bool virDeviceInfoPCIAddressPresent(const virDomainDeviceInfo *info); int virPCIDeviceAddressParseXML(xmlNodePtr node, virPCIDeviceAddressPtr addr); diff --git a/src/libvirt_private.syms b/src/libvirt_private.syms index 89c4511..8c7aef0 100644 --- a/src/libvirt_private.syms +++ b/src/libvirt_private.syms @@ -76,10 +76,13 @@ virCPUModeTypeToString; # conf/device_conf.h +virDeviceInfoPCIAddressPresent; +virDeviceInfoPCIAddressWanted; virInterfaceLinkFormat; virInterfaceLinkParseXML; virPCIDeviceAddressEqual; virPCIDeviceAddressFormat; +virPCIDeviceAddressIsEmpty; virPCIDeviceAddressIsValid; virPCIDeviceAddressParseXML; -- 2.8.4

2 1

[libvirt] [PATCH 0/2] virpci: support driver_override sysfs interface
by Jim Fehlig 02 Aug '16

02 Aug '16

This series fixes a regression using Xen's pciback with kernels >= 3.16 containing https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/driv… It makes use of the driver_override sysfs interface introduced in kernel 3.12 https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/driv… This approach was suggested by Laine, falling back to using the new_id interface in the absence of driver_override https://www.redhat.com/archives/libvir-list/2016-June/msg02129.html The first patch simplifies the existing logic in virPCIDeviceBindToStub, making the second patch (which actually fixes the bug) a bit easier to code (and hopefully review). Jim Fehlig (2): virpci: simplify virPCIDeviceBindToStub virpci: support driver_override sysfs interface src/util/virpci.c | 253 +++++++++++++++++++++++++++++++++++++----------------- 1 file changed, 173 insertions(+), 80 deletions(-) -- 2.1.4

2 7

[libvirt] [PATCH 0/3] qemu: Don't error out on missing optional disk
by Martin Kletzander 01 Aug '16

01 Aug '16

More info in PATCH 3/3. Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1168453 Martin Kletzander (3): qemu: Make qemuDomainCheckDiskStartupPolicy self-contained qemu: Remove unnecessary label and its only reference qemu: Fix support for startupPolicy with volume/pool disks src/qemu/qemu_domain.c | 36 +++++++++++++++++++++--------------- src/qemu/qemu_domain.h | 5 +++-- src/qemu/qemu_process.c | 20 +++++--------------- 3 files changed, 29 insertions(+), 32 deletions(-) -- 2.9.2

2 4

[libvirt] [PATCH v2] storage: Document wiping formatted volume types
by Martin Kletzander 01 Aug '16

01 Aug '16

When wiping a volume we just rewrite all the data of the volume, not only the content. Since format gets overridden, we need to recreate the volume. However we can't do that for every possible format out there. Since it was only coded for the ploop volume type, let's document what might be the consequences instead of forbidding it for every other format out there. Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=868771 Signed-off-by: Martin Kletzander <mkletzan(a)redhat.com> --- v2: - Reworded according to John v1: - https://www.redhat.com/archives/libvir-list/2016-August/msg00050.html src/libvirt-storage.c | 15 ++++++++++----- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/src/libvirt-storage.c b/src/libvirt-storage.c index cebe02fdda66..48996ba4c9c8 100644 --- a/src/libvirt-storage.c +++ b/src/libvirt-storage.c @@ -1725,11 +1725,16 @@ virStorageVolDelete(virStorageVolPtr vol, * @vol: pointer to storage volume * @flags: extra flags; not used yet, so callers should always pass 0 * - * Ensure data previously on a volume is not accessible to future - * reads. Also note, that depending on the actual volume - * representation, this call may not really overwrite the - * physical location of the volume. For instance, files stored - * journaled, log structured, copy-on-write, versioned, and + * Ensure data previously on a volume is not accessible to future reads. + * + * The data to be wiped may include the format and possibly size information, + * so non-raw images might become raw with a different size. It is storage + * backend dependent whether the format and size information is regenerated + * once the initial volume wipe is completed. + * + * Depending on the actual volume representation, this call may not + * overwrite the physical location of the volume. For instance, files + * stored journaled, log structured, copy-on-write, versioned, and * network file systems are known to be problematic. * * Returns 0 on success, or -1 on error -- 2.9.2

2 1

[libvirt] Question about LSN-2016-0001
by Jim Fehlig 01 Aug '16

01 Aug '16

I've noticed the behavior described by this LSN with libvirt+Xen. Config containing <graphics type='vnc' passwd=''/> allows any client to connect with no authentication check. I asked about this on the Xen security list and was told that "libxl interprets an empty password in the caller's configuration to mean that passwordless access should be permitted". The libvirt domXML docs are not clear on semantics of empty vnc password, only stating "The passwd attribute provides a VNC password in clear text". Should the libvirt domXML vnc passwd documentation be amended to define the semantics of an empty string in the passwd attribute? Is the behavior hypervisor-dependent as the documentation in qemu.conf suggests? Regards, Jim

2 4

[libvirt] [PATCH] storage: Document wiping formatted volume types
by Martin Kletzander 01 Aug '16

01 Aug '16

When wiping a volume we just rewrite all the data of the volume, not only the content. Since format gets overriden, we need to recreate the volume. However we can't do that for every possible format there. And since it was only coded for the ploop volume type, let's document what might be the consequences instead of forbidding it for every other format out there. Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=868771 Signed-off-by: Martin Kletzander <mkletzan(a)redhat.com> --- src/libvirt-storage.c | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/src/libvirt-storage.c b/src/libvirt-storage.c index cebe02fdda66..2be95a5adcaf 100644 --- a/src/libvirt-storage.c +++ b/src/libvirt-storage.c @@ -1732,6 +1732,12 @@ virStorageVolDelete(virStorageVolPtr vol, * journaled, log structured, copy-on-write, versioned, and * network file systems are known to be problematic. * + * NB the data includes the format (and possibly size information), so + * non-raw images might become raw with different size from libvirt's + * POV. Due to the fact that not all information for re-creating the + * volume can be kept for every volume type, the volume might be kept + * in that state and not re-created. + * * Returns 0 on success, or -1 on error */ int -- 2.9.2

2 2

[libvirt] [PATCH 0/3] storage: Clean up some volume wiping code
by Martin Kletzander 01 Aug '16

01 Aug '16

Just a cleanup so that some functionality is more readable (about what's being done for ploop volumes when wiping them) and extensible in the future. Feel free to NACK if I'm the only one that finds this more readable. Martin Kletzander (3): storage: Move functions around storage: Use path instead of volume as an argument storage: Clean up volume wiping src/storage/storage_backend.c | 193 +++++++++++++++++++++++------------------- 1 file changed, 104 insertions(+), 89 deletions(-) -- 2.9.2

2 6

[libvirt] [PATCH 0/2] admin: sasl: Save the SASL username to client's identity
by Erik Skultety 01 Aug '16

01 Aug '16

Erik Skultety (2): admin: Retrieve the SASL context for both local and remote connection daemon: sasl: Don't forget to save SASL username to client's identity daemon/admin_server.c | 14 +++++++------- daemon/remote.c | 6 ++++++ 2 files changed, 13 insertions(+), 7 deletions(-) -- 2.5.5

2 3

[libvirt] [PATCH] wireshark dissector compile fix
by Benedikt Spranger 01 Aug '16

01 Aug '16

Compiling libvirt wireshark dissector on a debian based system failed: CC wireshark/src/wireshark_src_libvirt_la-packet-libvirt.lo In file included from ../../libvirt/tools/wireshark/src/packet-libvirt.c:27:0: /usr/include/wireshark/epan/proto.h:40:18: fatal error: glib.h: No such file or directory compilation terminated. Makefile:2595: recipe for target 'wireshark/src/wireshark_src_libvirt_la-packet-libvirt.lo' failed Add an explicit dependency to glib. Benedikt Spranger (1): wireshark: glib compile fix m4/virt-wireshark.m4 | 2 ++ tools/Makefile.am | 4 ++-- 2 files changed, 4 insertions(+), 2 deletions(-) -- 2.8.1

3 5

[libvirt] [java binding] Retrieve VM IP address
by Fabien DUMINY 01 Aug '16

01 Aug '16

After reading https://github.com/fog/fog-libvirt/pull/11, I realize it should possible to get ip address of a domain from java binding ? How can I use that new feature from java? Thanks. Regards, Fabien.

2 1

[libvirt] [PATCH v2] qemu: remove panic dev models s390 and pseries when migrating
by Boris Fiuczynski 01 Aug '16

01 Aug '16

The panic devices with models s390 and pseries are autogenerated. For backwards compatibility reasons the devices are to be removed when migrating. Signed-off-by: Boris Fiuczynski <fiuczy(a)linux.vnet.ibm.com> --- src/qemu/qemu_domain.c | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c index ceac22f..82fa817 100644 --- a/src/qemu/qemu_domain.c +++ b/src/qemu/qemu_domain.c @@ -3310,6 +3310,15 @@ qemuDomainDefFormatBuf(virQEMUDriverPtr driver, virDomainControllerDefFree(usb); } + /* Remove the panic device for selected models if present */ + for (i = 0; i < def->npanics; i++) { + if (def->panics[i]->model == VIR_DOMAIN_PANIC_MODEL_S390 || + def->panics[i]->model == VIR_DOMAIN_PANIC_MODEL_PSERIES) { + VIR_DELETE_ELEMENT(def->panics, i, def->npanics); + i--; + } + } + for (i = 0; i < def->nchannels; i++) qemuDomainChrDefDropDefaultPath(def->channels[i], driver); } -- 2.9.0

2 2

[libvirt] [libvirt-glib][PATCH 0/2] Support python3
by Michal Privoznik 01 Aug '16

01 Aug '16

/usr/bin/python defaults to python3 on my system. While trying to build libvirt-glib I've noticed some errors in configure (first patch), and some missing implementation (second patch). Michal Privoznik (2): configure: Detect python in compatible fashion python: Introduce support for python3 configure.ac | 6 +++--- python/libvirt-glib.c | 44 ++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 47 insertions(+), 3 deletions(-) -- 2.8.4

3 4

[libvirt] [PATCH 0/2] a couple of schema fixes
by Nikolay Shirokovskiy 01 Aug '16

01 Aug '16

Nikolay Shirokovskiy (2): schema: add missed alias element to memory device schema: fix resolved interfaces of network type docs/schemas/domaincommon.rng | 43 +++++++++++++++++++++++++++++++++--------- docs/schemas/network.rng | 5 +---- docs/schemas/networkcommon.rng | 6 ++++++ 3 files changed, 41 insertions(+), 13 deletions(-) -- 1.8.3.1

3 5

[libvirt] [PATCH v2] qemu: fix domain id after domainCreateWithFlags()
by Sascha Silbe 01 Aug '16

01 Aug '16

Ever since virDomainCreateWithFlags() was introduced by de3aadaa [drivers: add virDomainCreateWithFlags if virDomainCreate exists], the domain ID retrieved with virDomainGetID() was incorrect for several drivers after virDomainCreateWithFlags() was called. The API consumer had to look up the domain anew to retrieve the correct ID. For the ESX driver, this was fixed in 6139b274 [esx: Update ID after starting a domain]. For the openvz driver, it was fixed in fd81a097 [openvzDomainCreateWithFlags: set domain id to the correct value]. The test driver, the OpenNebula driver (removed in the meantime) and the vbox driver were already updating the domain ID correctly in domainCreate(). Copy over the ID in qemuDomainCreateWithFlags() to fix this for the qemu driver, too. Fixes: de3aadaa ("drivers: add virDomainCreateWithFlags if virDomainCreate exists") Reported-by: Marc Hartmayer <mhartmay(a)linux.vnet.ibm.com> Signed-off-by: Sascha Silbe <silbe(a)linux.vnet.ibm.com> Tested-by: Marc Hartmayer <mhartmay(a)linux.vnet.ibm.com> Reviewed-by: Marc Hartmayer <mhartmay(a)linux.vnet.ibm.com> --- v1→v2: - Adjusted commit message. remoteDomainCreate() is actually fine since it explicitly queries the ID in a second RPC, only remoteDomainCreateWithFlags() is broken. Found and mentioned several other commits that fixed this for several of the other hypervisor drivers. src/qemu/qemu_driver.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c index 1820f85..43242e0 100644 --- a/src/qemu/qemu_driver.c +++ b/src/qemu/qemu_driver.c @@ -7100,6 +7100,7 @@ qemuDomainCreateWithFlags(virDomainPtr dom, unsigned int flags) QEMU_ASYNC_JOB_START) < 0) goto endjob; + dom->id = vm->def->id; ret = 0; endjob: -- 1.9.1

2 1

[libvirt] [PATCH] wireshark: Drop glib dependency
by Michal Privoznik 01 Aug '16

01 Aug '16

The only function that we currently use from glib is g_sprintf(). That's a very big gun for such small target. Not only that, but we've silently relied on wireshark dragging in the glib. Replace the g_sprintf() with plain sprinf() so that we can drop the glib dependency. Signed-off-by: Michal Privoznik <mprivozn(a)redhat.com> --- tools/wireshark/src/packet-libvirt.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/tools/wireshark/src/packet-libvirt.c b/tools/wireshark/src/packet-libvirt.c index 569a0a1..9eadf4f 100644 --- a/tools/wireshark/src/packet-libvirt.c +++ b/tools/wireshark/src/packet-libvirt.c @@ -27,8 +27,6 @@ #include <wireshark/epan/proto.h> #include <wireshark/epan/packet.h> #include <wireshark/epan/dissectors/packet-tcp.h> -#include <glib.h> -#include <glib/gprintf.h> #ifdef HAVE_RPC_TYPES_H # include <rpc/types.h> #endif @@ -113,7 +111,7 @@ format_xdr_bytes(guint8 *bytes, guint32 length) for (i = 0; i < length; i++) { /* We know that buf has enough size to contain 2 * length + '\0' characters. */ - g_sprintf(buf, "%02x", bytes[i]); + sprintf(buf, "%02x", bytes[i]); buf += 2; } return buf - length*2; -- 2.8.4

2 1

[libvirt] [PATCH] tests: qemu: Don't leak security manager object
by Peter Krempa 01 Aug '16

01 Aug '16

==2064442== 200 (88 direct, 112 indirect) bytes in 1 blocks are definitely lost in loss record 54 of 73 ==2064442== at 0x4C2E0F0: calloc (vg_replace_malloc.c:711) ==2064442== by 0x18E75B80: virAllocVar (viralloc.c:560) ==2064442== by 0x18EC43B0: virObjectNew (virobject.c:193) ==2064442== by 0x18EC476E: virObjectLockableNew (virobject.c:219) ==2064442== by 0x1906BC73: virSecurityManagerNewDriver (security_manager.c:93) ==2064442== by 0x1906C076: virSecurityManagerNewStack (security_manager.c:115) ==2064442== by 0x43CC39: qemuTestDriverInit (testutilsqemu.c:548) ==2064442== by 0x4337ED: mymain (qemumonitorjsontest.c:2440) ==2064442== by 0x43BABE: virTestMain (testutils.c:982) ==2064442== by 0x43A490: main (qemumonitorjsontest.c:2558) --- tests/testutilsqemu.c | 1 + 1 file changed, 1 insertion(+) diff --git a/tests/testutilsqemu.c b/tests/testutilsqemu.c index 1cf72a9..133de29 100644 --- a/tests/testutilsqemu.c +++ b/tests/testutilsqemu.c @@ -475,6 +475,7 @@ void qemuTestDriverFree(virQEMUDriver *driver) virObjectUnref(driver->xmlopt); virObjectUnref(driver->caps); virObjectUnref(driver->config); + virObjectUnref(driver->securityManager); } int qemuTestCapsCacheInsert(virQEMUCapsCachePtr cache, const char *binary, -- 2.9.2

2 1

[libvirt] [PATCH] spec: Require perl-XML-XPath during build
by Martin Kletzander 01 Aug '16

01 Aug '16

Some time ago I went through the trouble to make support for perl module requirements in our bootstrap. Even though we require it in the bootstrap, it was not listed as a build requirement in the specfile. Signed-off-by: Martin Kletzander <mkletzan(a)redhat.com> --- libvirt.spec.in | 1 + 1 file changed, 1 insertion(+) diff --git a/libvirt.spec.in b/libvirt.spec.in index cc5d72209873..b182f4521453 100644 --- a/libvirt.spec.in +++ b/libvirt.spec.in @@ -269,6 +269,7 @@ BuildRequires: /usr/bin/pod2man %endif BuildRequires: git BuildRequires: perl +BuildRequires: perl-XML-XPath BuildRequires: python %if %{with_systemd} BuildRequires: systemd-units -- 2.9.2

3 4

[libvirt] [PATCH 0/2] qemu: caps: Sanitize storage of machine type related data
by Peter Krempa 01 Aug '16

01 Aug '16

Peter Krempa (2): qemu: capabilities: Drop unused function virQEMUCapsGetMachineTypes qemu: caps: Sanitize storage of machine type related data src/qemu/qemu_capabilities.c | 125 ++++++++++++++++--------------------------- src/qemu/qemu_capabilities.h | 2 - 2 files changed, 46 insertions(+), 81 deletions(-) -- 2.9.2

2 5