Re: [Qemu-devel] [PATCH] seccomp: add shmctl(), mlock(), and munlock() to the syscall whitelist

[Paul Moore <pmoore@redhat.com>]

On Wednesday, February 26, 2014 10:25:01 AM Paul Moore wrote:
> Additional testing reveals that PulseAudio requires shmctl() and the
> mlock()/munlock() syscalls on some systems/configurations.  As before,
> on systems that do require these syscalls, the problem can be seen with
> the following command line:
> 
>   # qemu -monitor stdio  -sandbox on \
>          -device intel-hda -device hda-duplex
> 
> Signed-off-by: Paul Moore <pmoore@redhat.com>
> ---
>  qemu-seccomp.c |    5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/qemu-seccomp.c b/qemu-seccomp.c
> index caa926e..3db1e9b 100644
> --- a/qemu-seccomp.c
> +++ b/qemu-seccomp.c
> @@ -225,7 +225,10 @@ static const struct QemuSeccompSyscall
> seccomp_whitelist[] = { { SCMP_SYS(fchmod), 240 },
>      { SCMP_SYS(shmget), 240 },
>      { SCMP_SYS(shmat), 240 },
> -    { SCMP_SYS(shmdt), 240 }
> +    { SCMP_SYS(shmdt), 240 },
> +    { SCMP_SYS(shmctl), 240 },
> +    { SCMP_SYS(mlock), 240 },
> +    { SCMP_SYS(munlock), 240 }
>  };
> 
>  int seccomp_start(void)

Bump to bring this back the forefront of everyone's minds.  Can we get this 
merged?

-- 
paul moore
security and virtualization @ redhat