]> xenbits.xensource.com Git - people/liuw/libxenctrl-split/libvirt.git/commit
qemu: don't fill in nicindexes for session mode libvirtd
authorLaine Stump <laine@laine.org>
Tue, 10 Mar 2015 06:09:24 +0000 (02:09 -0400)
committerLaine Stump <laine@laine.org>
Tue, 10 Mar 2015 11:53:10 +0000 (07:53 -0400)
commit705242f8809dc2222c35c64d5408dd6b0cc94cf8
treedbbd23790f90ed7099ace209c369b64de4521f53
parenta663c34ad015e7911a885a4f1badef54a9ea2677
qemu: don't fill in nicindexes for session mode libvirtd

Commit 4bbe1029f fixed a problem in commit f7afeddc by moving the call
to virNetDevGetIndex() to a location common to all interface types (so
that the nicindex array would be filled in for macvtap as well as tap
interfaces), but the location was *too* common, as the original call
to virNetDevGetIndex() had been in a section qualified by "if
(cfg->privileged)". The result was that the "fixed" libvirtd would try
to call virNetDevGetIndex() even for session mode libvirtd, and end up
failing with the log message:

  Unable to open control socket: Operation not permitted

To remedy that, this patch qualifies the call to virNetDevGetIndex()
in its new location with cfg->privileged.

This resolves https://bugzilla.redhat.com/show_bug.cgi?id=1198244
src/qemu/qemu_command.c