[libvirt] [PATCH] qemu: Fix dynamic_ownership qemu.conf setting
Martin Kletzander
mkletzan at redhat.com
Tue Sep 29 04:46:18 UTC 2015
On Mon, Sep 28, 2015 at 07:53:42PM -0400, Cole Robinson wrote:
>Commit 307fb904 (Sep 10) added a 'privileged' variable when creating
>the DAC driver:
>
>@@ -153,6 +157,7 @@ virSecurityManagerNewDAC(const char *virtDriver,
> bool defaultConfined,
> bool requireConfined,
> bool dynamicOwnership,
>+ bool privileged,
> virSecurityManagerDACChownCallback chownCallback)
>
>But argument order is mixed up at the caller, swapping dynamicOwnership
>and privileged values. This corrects the argument order
>
I'm pretty sure I noticed this during review. I must've failed to
inform the author because I see no reply to the patch that caused it.
ACK and safe for freeze!
>https://bugzilla.redhat.com/show_bug.cgi?id=1266628
>---
> src/qemu/qemu_driver.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
>index 7025c51..aff1915 100644
>--- a/src/qemu/qemu_driver.c
>+++ b/src/qemu/qemu_driver.c
>@@ -431,8 +431,8 @@ qemuSecurityInit(virQEMUDriverPtr driver)
> cfg->allowDiskFormatProbing,
> cfg->securityDefaultConfined,
> cfg->securityRequireConfined,
>- virQEMUDriverIsPrivileged(driver),
> cfg->dynamicOwnership,
>+ virQEMUDriverIsPrivileged(driver),
> qemuSecurityChownCallback)))
> goto error;
> if (!stack) {
>--
>2.5.0
>
>--
>libvir-list mailing list
>libvir-list at redhat.com
>https://www.redhat.com/mailman/listinfo/libvir-list
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20150929/77aadfce/attachment-0001.sig>
More information about the libvir-list
mailing list