* [Qemu-devel] [PATCH] seccomp: add additional asynchronous I/O syscalls
@ 2013-07-15 19:32 Paul Moore
2013-07-23 13:57 ` Paul Moore
2013-08-02 12:35 ` Anthony Liguori
0 siblings, 2 replies; 5+ messages in thread
From: Paul Moore @ 2013-07-15 19:32 UTC (permalink / raw)
To: qemu-devel, qemu-stable; +Cc: coreyb, otubo
A previous commit, "seccomp: add the asynchronous I/O syscalls to the
whitelist", added several asynchronous I/O syscalls but left out the
io_submit() and io_cancel() syscalls. This patch corrects this by
adding the two missing asynchronous I/O syscalls.
Signed-off-by: Paul Moore <pmoore@redhat.com>
---
qemu-seccomp.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/qemu-seccomp.c b/qemu-seccomp.c
index ca123bf..173d185 100644
--- a/qemu-seccomp.c
+++ b/qemu-seccomp.c
@@ -33,6 +33,7 @@ static const struct QemuSeccompSyscall seccomp_whitelist[] = {
{ SCMP_SYS(socketcall), 250 },
#endif
{ SCMP_SYS(read), 249 },
+ { SCMP_SYS(io_submit), 249 },
{ SCMP_SYS(brk), 248 },
{ SCMP_SYS(clone), 247 },
{ SCMP_SYS(mmap), 247 },
@@ -231,6 +232,7 @@ static const struct QemuSeccompSyscall seccomp_whitelist[] = {
{ SCMP_SYS(recvmmsg), 241 },
{ SCMP_SYS(prlimit64), 241 },
{ SCMP_SYS(waitid), 241 },
+ { SCMP_SYS(io_cancel), 241 },
{ SCMP_SYS(io_setup), 241 },
{ SCMP_SYS(io_destroy), 241 }
};
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [Qemu-devel] [PATCH] seccomp: add additional asynchronous I/O syscalls
2013-07-15 19:32 [Qemu-devel] [PATCH] seccomp: add additional asynchronous I/O syscalls Paul Moore
@ 2013-07-23 13:57 ` Paul Moore
2013-07-24 18:02 ` Eduardo Otubo
2013-08-02 12:35 ` Anthony Liguori
1 sibling, 1 reply; 5+ messages in thread
From: Paul Moore @ 2013-07-23 13:57 UTC (permalink / raw)
To: qemu-devel, qemu-stable; +Cc: coreyb, otubo
On Monday, July 15, 2013 03:32:01 PM Paul Moore wrote:
> A previous commit, "seccomp: add the asynchronous I/O syscalls to the
> whitelist", added several asynchronous I/O syscalls but left out the
> io_submit() and io_cancel() syscalls. This patch corrects this by
> adding the two missing asynchronous I/O syscalls.
>
> Signed-off-by: Paul Moore <pmoore@redhat.com>
A gentle nudge so this fix doesn't get forgotten.
> ---
> qemu-seccomp.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/qemu-seccomp.c b/qemu-seccomp.c
> index ca123bf..173d185 100644
> --- a/qemu-seccomp.c
> +++ b/qemu-seccomp.c
> @@ -33,6 +33,7 @@ static const struct QemuSeccompSyscall seccomp_whitelist[]
> = { { SCMP_SYS(socketcall), 250 },
> #endif
> { SCMP_SYS(read), 249 },
> + { SCMP_SYS(io_submit), 249 },
> { SCMP_SYS(brk), 248 },
> { SCMP_SYS(clone), 247 },
> { SCMP_SYS(mmap), 247 },
> @@ -231,6 +232,7 @@ static const struct QemuSeccompSyscall
> seccomp_whitelist[] = { { SCMP_SYS(recvmmsg), 241 },
> { SCMP_SYS(prlimit64), 241 },
> { SCMP_SYS(waitid), 241 },
> + { SCMP_SYS(io_cancel), 241 },
> { SCMP_SYS(io_setup), 241 },
> { SCMP_SYS(io_destroy), 241 }
> };
--
paul moore
security and virtualization @ redhat
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [Qemu-devel] [PATCH] seccomp: add additional asynchronous I/O syscalls
2013-07-23 13:57 ` Paul Moore
@ 2013-07-24 18:02 ` Eduardo Otubo
2013-07-29 22:12 ` Paul Moore
0 siblings, 1 reply; 5+ messages in thread
From: Eduardo Otubo @ 2013-07-24 18:02 UTC (permalink / raw)
To: Paul Moore; +Cc: coreyb, qemu-devel, qemu-stable
On 07/23/2013 10:57 AM, Paul Moore wrote:
> On Monday, July 15, 2013 03:32:01 PM Paul Moore wrote:
>> A previous commit, "seccomp: add the asynchronous I/O syscalls to the
>> whitelist", added several asynchronous I/O syscalls but left out the
>> io_submit() and io_cancel() syscalls. This patch corrects this by
>> adding the two missing asynchronous I/O syscalls.
>>
>> Signed-off-by: Paul Moore <pmoore@redhat.com>
>
> A gentle nudge so this fix doesn't get forgotten.
Reviewed and tested.
Reviewed-by: Eduardo Otubo <otubo@linux.vnet.ibm.com>
>
>> ---
>> qemu-seccomp.c | 2 ++
>> 1 file changed, 2 insertions(+)
>>
>> diff --git a/qemu-seccomp.c b/qemu-seccomp.c
>> index ca123bf..173d185 100644
>> --- a/qemu-seccomp.c
>> +++ b/qemu-seccomp.c
>> @@ -33,6 +33,7 @@ static const struct QemuSeccompSyscall seccomp_whitelist[]
>> = { { SCMP_SYS(socketcall), 250 },
>> #endif
>> { SCMP_SYS(read), 249 },
>> + { SCMP_SYS(io_submit), 249 },
>> { SCMP_SYS(brk), 248 },
>> { SCMP_SYS(clone), 247 },
>> { SCMP_SYS(mmap), 247 },
>> @@ -231,6 +232,7 @@ static const struct QemuSeccompSyscall
>> seccomp_whitelist[] = { { SCMP_SYS(recvmmsg), 241 },
>> { SCMP_SYS(prlimit64), 241 },
>> { SCMP_SYS(waitid), 241 },
>> + { SCMP_SYS(io_cancel), 241 },
>> { SCMP_SYS(io_setup), 241 },
>> { SCMP_SYS(io_destroy), 241 }
>> };
--
Eduardo Otubo
IBM Linux Technology Center
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [Qemu-devel] [PATCH] seccomp: add additional asynchronous I/O syscalls
2013-07-24 18:02 ` Eduardo Otubo
@ 2013-07-29 22:12 ` Paul Moore
0 siblings, 0 replies; 5+ messages in thread
From: Paul Moore @ 2013-07-29 22:12 UTC (permalink / raw)
To: qemu-devel; +Cc: coreyb, qemu-stable, Eduardo Otubo
On Wednesday, July 24, 2013 03:02:23 PM Eduardo Otubo wrote:
> On 07/23/2013 10:57 AM, Paul Moore wrote:
> > On Monday, July 15, 2013 03:32:01 PM Paul Moore wrote:
> >> A previous commit, "seccomp: add the asynchronous I/O syscalls to the
> >> whitelist", added several asynchronous I/O syscalls but left out the
> >> io_submit() and io_cancel() syscalls. This patch corrects this by
> >> adding the two missing asynchronous I/O syscalls.
> >>
> >> Signed-off-by: Paul Moore <pmoore@redhat.com>
> >
> > A gentle nudge so this fix doesn't get forgotten.
>
> Reviewed and tested.
>
> Reviewed-by: Eduardo Otubo <otubo@linux.vnet.ibm.com>
Any chance of merging this patch?
> >> ---
> >>
> >> qemu-seccomp.c | 2 ++
> >> 1 file changed, 2 insertions(+)
> >>
> >> diff --git a/qemu-seccomp.c b/qemu-seccomp.c
> >> index ca123bf..173d185 100644
> >> --- a/qemu-seccomp.c
> >> +++ b/qemu-seccomp.c
> >> @@ -33,6 +33,7 @@ static const struct QemuSeccompSyscall
> >> seccomp_whitelist[] = { { SCMP_SYS(socketcall), 250 },
> >>
> >> #endif
> >>
> >> { SCMP_SYS(read), 249 },
> >>
> >> + { SCMP_SYS(io_submit), 249 },
> >>
> >> { SCMP_SYS(brk), 248 },
> >> { SCMP_SYS(clone), 247 },
> >> { SCMP_SYS(mmap), 247 },
> >>
> >> @@ -231,6 +232,7 @@ static const struct QemuSeccompSyscall
> >> seccomp_whitelist[] = { { SCMP_SYS(recvmmsg), 241 },
> >>
> >> { SCMP_SYS(prlimit64), 241 },
> >> { SCMP_SYS(waitid), 241 },
> >>
> >> + { SCMP_SYS(io_cancel), 241 },
> >>
> >> { SCMP_SYS(io_setup), 241 },
> >> { SCMP_SYS(io_destroy), 241 }
> >>
> >> };
--
paul moore
security and virtualization @ redhat
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [Qemu-devel] [PATCH] seccomp: add additional asynchronous I/O syscalls
2013-07-15 19:32 [Qemu-devel] [PATCH] seccomp: add additional asynchronous I/O syscalls Paul Moore
2013-07-23 13:57 ` Paul Moore
@ 2013-08-02 12:35 ` Anthony Liguori
1 sibling, 0 replies; 5+ messages in thread
From: Anthony Liguori @ 2013-08-02 12:35 UTC (permalink / raw)
To: Paul Moore, qemu-devel, qemu-stable; +Cc: coreyb, Eduardo Otubo
Applied. Thanks.
Regards,
Anthony Liguori
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2013-08-02 12:36 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-07-15 19:32 [Qemu-devel] [PATCH] seccomp: add additional asynchronous I/O syscalls Paul Moore
2013-07-23 13:57 ` Paul Moore
2013-07-24 18:02 ` Eduardo Otubo
2013-07-29 22:12 ` Paul Moore
2013-08-02 12:35 ` Anthony Liguori
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).