[lxc-devel] [PATCH] don't hardcode the path to criu when checking versions

Serge Hallyn serge.hallyn at ubuntu.com
Wed Jun 3 14:36:36 UTC 2015


Quoting Tycho Andersen (tycho.andersen at canonical.com):
> We use the right path when actually execing criu to checkpoint and restore, but
> when checking versions we didn't. Let's use the right path.
> 
> Reported-by: Dietmar Maurer <dietmar at proxmox.com>
> Signed-off-by: Tycho Andersen <tycho.andersen at canonical.com>

Acked-by: Serge E. Hallyn <serge.hallyn at ubuntu.com>

> ---
>  src/lxc/criu.c | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/src/lxc/criu.c b/src/lxc/criu.c
> index c331adf..c6b1863 100644
> --- a/src/lxc/criu.c
> +++ b/src/lxc/criu.c
> @@ -223,13 +223,15 @@ static bool criu_version_ok()
>  
>  	if (pid == 0) {
>  		char *args[] = { "criu", "--version", NULL };
> +		char *path;
>  		close(pipes[0]);
>  
>  		close(STDERR_FILENO);
>  		if (dup2(pipes[1], STDOUT_FILENO) < 0)
>  			exit(1);
>  
> -		execv("/usr/local/sbin/criu", args);
> +		path = on_path("criu", NULL);
> +		execv(path, args);
>  		exit(1);
>  	} else {
>  		FILE *f;
> -- 
> 2.1.4
> 
> _______________________________________________
> lxc-devel mailing list
> lxc-devel at lists.linuxcontainers.org
> http://lists.linuxcontainers.org/listinfo/lxc-devel


More information about the lxc-devel mailing list