[CRIU] [PATCH] kerndat: include config.h before using CONFIG_* contants

Dmitry Safonov dsafonov at virtuozzo.com
Tue Jun 27 12:45:55 MSK 2017


On 06/27/2017 06:50 AM, Andrei Vagin wrote:
> From: Andrei Vagin <avagin at virtuozzo.com>
> 
> Otherwise someone can include kerndat.h before config.h
> and get another kerndat structure. For example, proc_parse
> doesn't include config.h and Cyrill met this problem.
> 
> Cc: Dmitry Safonov <dsafonov at virtuozzo.com>
> Cc: Cyrill Gorcunov <gorcunov at openvz.org>
> Signed-off-by: Andrei Vagin <avagin at virtuozzo.com>

Thanks, LGTM:
Reviewed-by: Dmitry Safonov <dsafonov at virtuozzo.com>

> ---
>   criu/include/kerndat.h | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/criu/include/kerndat.h b/criu/include/kerndat.h
> index 2035f79..8eaa06d 100644
> --- a/criu/include/kerndat.h
> +++ b/criu/include/kerndat.h
> @@ -3,6 +3,7 @@
>   
>   #include <stdbool.h>
>   #include "int.h"
> +#include "config.h"
>   #ifdef CONFIG_VDSO
>   #include "util-vdso.h"
>   #endif
> 
-- 
              Dmitry


More information about the CRIU mailing list