[CRIU] [PATCH 3/3] proc-parse: Drop deleted postfix from the mountpoint root

Andrew Vagin avagin at gmail.com
Tue Aug 11 01:37:59 PDT 2015


On Mon, Aug 10, 2015 at 11:34:27AM +0300, Cyrill Gorcunov wrote:
> This address only a part of a problem with deleted mountpoint,
> so rather a fast fix to not block development, but warning
> issued to nag us to not forget resolve it before the release.

The result will be incorrect. Do we really want it? This issue can be
easy fixed in libcontainer which can mount /dev/null from a host file
system.

> 
> Signed-off-by: Cyrill Gorcunov <gorcunov at openvz.org>
> ---
>  proc_parse.c | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> diff --git a/proc_parse.c b/proc_parse.c
> index 00e2b7bc6d20..a8addcded1ea 100644
> --- a/proc_parse.c
> +++ b/proc_parse.c
> @@ -1025,6 +1025,14 @@ static int parse_mountinfo_ent(char *str, struct mount_info *new, char **fsname)
>  	cure_path(new->mountpoint);
>  	cure_path(new->root);
>  
> +	/*
> +	 * FIXME: Revisit and make a proper restore
> +	 * of deleted roots. https://github.com/xemul/criu/issues/9
> +	 * for details.
> +	 */
> +	if (strip_deleted(new->root))
> +		pr_warn("Dropped postfix from the root");
> +
>  	new->mountpoint = xrealloc(new->mountpoint, strlen(new->mountpoint) + 1);
>  	if (!new->mountpoint)
>  		goto err;
> -- 
> 2.4.3
> 


More information about the CRIU mailing list