[libvirt] [sandbox PATCH 1/4] Make sure the sandbox state dir and config can be accessed

Cédric Bosdonnat cbosdonnat at suse.com
Mon Jun 15 13:43:43 UTC 2015


When running a KVM sandbox as root, the qemu process will run as
another user (likely qemu). We need to make sure this user can access
the vmlinux and initrd.img, sandbox.cfg and mounts.cfg files.
---
 libvirt-sandbox/libvirt-sandbox-config.c              | 2 +-
 libvirt-sandbox/libvirt-sandbox-context-interactive.c | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/libvirt-sandbox/libvirt-sandbox-config.c b/libvirt-sandbox/libvirt-sandbox-config.c
index 087b5ce..8991043 100644
--- a/libvirt-sandbox/libvirt-sandbox-config.c
+++ b/libvirt-sandbox/libvirt-sandbox-config.c
@@ -2258,7 +2258,7 @@ gboolean gvir_sandbox_config_save_to_path(GVirSandboxConfig *config,
     if (!(data = g_key_file_to_data(file, &len, error)))
         goto cleanup;
 
-    if (!(os = G_OUTPUT_STREAM(g_file_create(f, G_FILE_CREATE_PRIVATE, NULL, error))))
+    if (!(os = G_OUTPUT_STREAM(g_file_create(f, G_FILE_CREATE_NONE, NULL, error))))
         goto cleanup;
 
     if (!g_output_stream_write_all(os, data, len, NULL, NULL, error))
diff --git a/libvirt-sandbox/libvirt-sandbox-context-interactive.c b/libvirt-sandbox/libvirt-sandbox-context-interactive.c
index cec7965..78b2fbd 100644
--- a/libvirt-sandbox/libvirt-sandbox-context-interactive.c
+++ b/libvirt-sandbox/libvirt-sandbox-context-interactive.c
@@ -217,8 +217,8 @@ static gboolean gvir_sandbox_context_interactive_start(GVirSandboxContext *ctxt,
                                                         error)))
         goto cleanup;
 
-    g_mkdir_with_parents(statedir, 0700);
-    g_mkdir_with_parents(configdir, 0700);
+    g_mkdir_with_parents(statedir, 0755);
+    g_mkdir_with_parents(configdir, 0755);
 
     unlink(configfile);
     if (!gvir_sandbox_config_save_to_path(config, configfile, error))
-- 
2.1.4




More information about the libvir-list mailing list