[CRIU] [PATCH] seccomp: Use own seccomp_metadata_t type
Andrey Vagin
avagin at virtuozzo.com
Tue May 22 23:14:43 MSK 2018
https://travis-ci.org/criupatchwork/criu/builds/380591639?utm_source=github_status&utm_medium=notification
On Fri, May 18, 2018 at 11:41:49AM +0300, Cyrill Gorcunov wrote:
> On fedora rawhide seccomp_metadata for some
> reason is not defined (while in kernel it introduced
> together with PTRACE_SECCOMP_GET_METADATA). So
> lets do a trick for a while -- define own alias.
> Once system headers get settled down we might find
> more suitable solution. Because it's a part of kernel
> API we're on the safe side.
>
> Signed-off-by: Cyrill Gorcunov <gorcunov at gmail.com>
> ---
> compel/include/uapi/ptrace.h | 4 ++--
> criu/seccomp.c | 2 +-
> 2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/compel/include/uapi/ptrace.h b/compel/include/uapi/ptrace.h
> index fcc2d62b08e4..85c0a84e308d 100644
> --- a/compel/include/uapi/ptrace.h
> +++ b/compel/include/uapi/ptrace.h
> @@ -53,10 +53,10 @@
> #ifndef PTRACE_SECCOMP_GET_METADATA
> # define PTRACE_SECCOMP_GET_METADATA 0x420d
>
> -struct seccomp_metadata {
> +typedef struct {
> uint64_t filter_off; /* Input: which filter */
> uint64_t flags; /* Output: filter's flags */
> -};
> +} seccomp_metadata_t;
>
> #endif /* PTRACE_SECCOMP_GET_METADATA */
>
> diff --git a/criu/seccomp.c b/criu/seccomp.c
> index d5da6475fcf8..78a87352d133 100644
> --- a/criu/seccomp.c
> +++ b/criu/seccomp.c
> @@ -131,7 +131,7 @@ int seccomp_dump_thread(pid_t tid_real, ThreadCoreEntry *thread_core)
>
> static int collect_filter(struct seccomp_entry *entry)
> {
> - struct seccomp_metadata meta_buf, *meta = &meta_buf;
> + seccomp_metadata_t meta_buf, *meta = &meta_buf;
> struct seccomp_filter_chain *chain, *prev;
> struct sock_filter buf[BPF_MAXINSNS];
> size_t i;
> --
> 2.14.3
>
More information about the CRIU
mailing list