The checkpoint file header now has an 11-character string
containing the name of the active LSM, following the uts
info, and a variable length buffer type conaining LSM-specific
version information (for instance a sha1sum of policy).
Handle these.

Also add a -k (--keeplsm) flag to tell restart to set the
RESTART_KEEP_LSM flag to sys_restart().

Changelog:
oct 15: separate out from container config section patch
oct 05: 1. move keep_lsm into arg struct
        2. read a separate container config section
        3. use CHECKPOINT_LSM_NAME_MAX

Signed-off-by: Serge E. Hallyn <se...@us.ibm.com>
---
 restart.c |   39 +++++++++++++++++++++++++++++++++++++--
 1 files changed, 37 insertions(+), 2 deletions(-)

diff --git a/restart.c b/restart.c
index fa786ef..588ab97 100644
--- a/restart.c
+++ b/restart.c
@@ -68,6 +68,7 @@ static char usage_str[] =
 "     --signal=SIG       send SIG to root task on SIGINT (default: SIGKILL\n"
 "                        to container root, SIGINT otherwise)\n"
 "  -w,--wait             wait for root task to termiate (default)\n"
+"  -k,--keeplsm          Try to recreate original LSM labels on all objects\n"
 "     --show-status      show exit status of root task (implies -w)\n"
 "     --copy-status      imitate exit status of root task (implies -w)\n"
 "  -W,--no-wait          do not wait for root task to terminate\n"
@@ -350,6 +351,7 @@ struct args {
        int copy_status;
        char *freezer;
        char *input;
+       int keep_lsm;
 };
 
 static void usage(char *str)
@@ -380,6 +382,7 @@ static void parse_args(struct args *args, int argc, char 
*argv[])
                { "self",       no_argument,            NULL, 6},
                { "signal",     required_argument,      NULL, 4 },
                { "inspect",    no_argument,            NULL, 5 },
+               { "keeplsm",    no_argument,            NULL, 'k' },
                { "input",      required_argument,      NULL, 'i' },
                { "root",       required_argument,      NULL, 'r' },
                { "wait",       no_argument,            NULL, 'w' },
@@ -391,7 +394,7 @@ static void parse_args(struct args *args, int argc, char 
*argv[])
                { "debug",      no_argument,            NULL, 'd' },
                { NULL,         0,                      NULL, 0 }
        };
-       static char optc[] = "hdvpPwWF:r:i:";
+       static char optc[] = "hdvpkPwWF:r:i:";
 
        int sig;
 
@@ -446,6 +449,9 @@ static void parse_args(struct args *args, int argc, char 
*argv[])
                case 'w':
                        args->wait = 1;
                        break;
+               case 'k':
+                       args->keep_lsm = RESTART_KEEP_LSM;
+                       break;
                case 'W':
                        args->wait = 0;
                        break;
@@ -936,6 +942,7 @@ static int ckpt_coordinator(struct ckpt_ctx *ctx)
        if (ctx->args->freezer)
                flags |= RESTART_FROZEN;
 
+       flags |= ctx->args->keep_lsm;
        ret = restart(root_pid, STDIN_FILENO, flags);
 
        if (ret < 0) {
@@ -1584,6 +1591,8 @@ static int ckpt_make_tree(struct ckpt_ctx *ctx, struct 
task *task)
        if (task->flags & (TASK_GHOST | TASK_DEAD))
                flags |= RESTART_GHOST;
 
+       flags |= ctx->args->keep_lsm;
+
        /* on success this doesn't return */
        ckpt_dbg("about to call sys_restart(), flags %#lx\n", flags);
        ret = restart(0, STDIN_FILENO, flags);
@@ -2116,10 +2125,23 @@ static int ckpt_read_header_arch(struct ckpt_ctx *ctx)
 
 static int ckpt_read_container(struct ckpt_ctx *ctx)
 {
+       int ret;
        struct ckpt_hdr_container *h;
+       char *ptr;
 
        h = (struct ckpt_hdr_container *) ctx->container;
-       return ckpt_read_obj_type(ctx, h, sizeof(*h), CKPT_HDR_CONTAINER);
+       ret = ckpt_read_obj_type(ctx, h, sizeof(*h), CKPT_HDR_CONTAINER);
+       if (ret < 0)
+               return ret;
+
+       ptr = (char *) h;
+       ptr += ((struct ckpt_hdr *) ptr)->len;
+       ret = ckpt_read_obj_buffer(ctx, ptr, CHECKPOINT_LSM_NAME_MAX + 1);
+       if (ret < 0)
+               return ret;
+
+       ptr += ((struct ckpt_hdr *) ptr)->len;
+       return ckpt_read_obj_type(ctx, ptr, 200, CKPT_HDR_LSM_INFO);
 }
 
 static int ckpt_read_tree(struct ckpt_ctx *ctx)
@@ -2197,9 +2219,22 @@ static int ckpt_write_header_arch(struct ckpt_ctx *ctx)
 static int ckpt_write_container(struct ckpt_ctx *ctx)
 {
        char *ptr;
+       int ret;
 
        ptr = (char *) ctx->container;
        /* write the container info section */
+       ret = ckpt_write_obj(ctx, (struct ckpt_hdr *) ptr);
+       if (ret < 0)
+               return ret;
+
+       /* write the lsm name buffer */
+       ptr += ((struct ckpt_hdr *) ptr)->len;
+       ret = ckpt_write_obj(ctx, (struct ckpt_hdr *) ptr);
+       if (ret < 0)
+               return ret;
+
+       /* write the lsm policy section */
+       ptr += ((struct ckpt_hdr *) ptr)->len;
        return ckpt_write_obj(ctx, (struct ckpt_hdr *) ptr);
 }
 
-- 
1.6.1.1

_______________________________________________
Containers mailing list
contain...@lists.linux-foundation.org
https://lists.linux-foundation.org/mailman/listinfo/containers

_______________________________________________
Devel mailing list
Devel@openvz.org
https://openvz.org/mailman/listinfo/devel

Reply via email to