Re: [Qemu-devel] [PATCH] exec.c: Initialize sa_flags passed to sigaction()

2018-05-16 Thread Paolo Bonzini
On 15/05/2018 20:27, Peter Maydell wrote: > Coverity points out that in the user-only version of cpu_abort() we > call sigaction() with a partially initialized struct sigaction > (CID 1005351). Correct the omission. > > Signed-off-by: Peter Maydell > --- > exec.c | 1 + > 1 file changed, 1 inser

Re: [Qemu-devel] [PATCH] exec.c: Initialize sa_flags passed to sigaction()

2018-05-15 Thread Eric Blake
On 05/15/2018 03:53 PM, Philippe Mathieu-Daudé wrote: On 05/15/2018 03:27 PM, Peter Maydell wrote: Coverity points out that in the user-only version of cpu_abort() we call sigaction() with a partially initialized struct sigaction (CID 1005351). Correct the omission. Signed-off-by: Peter Maydell

Re: [Qemu-devel] [PATCH] exec.c: Initialize sa_flags passed to sigaction()

2018-05-15 Thread Philippe Mathieu-Daudé
On 05/15/2018 03:27 PM, Peter Maydell wrote: > Coverity points out that in the user-only version of cpu_abort() we > call sigaction() with a partially initialized struct sigaction > (CID 1005351). Correct the omission. > > Signed-off-by: Peter Maydell > --- > exec.c | 1 + > 1 file changed, 1 in

Re: [Qemu-devel] [PATCH] exec.c: Initialize sa_flags passed to sigaction()

2018-05-15 Thread Alex Bennée
Peter Maydell writes: > Coverity points out that in the user-only version of cpu_abort() we > call sigaction() with a partially initialized struct sigaction > (CID 1005351). Correct the omission. > > Signed-off-by: Peter Maydell Reviewed-by: Alex Bennée > --- > exec.c | 1 + > 1 file change

[Qemu-devel] [PATCH] exec.c: Initialize sa_flags passed to sigaction()

2018-05-15 Thread Peter Maydell
Coverity points out that in the user-only version of cpu_abort() we call sigaction() with a partially initialized struct sigaction (CID 1005351). Correct the omission. Signed-off-by: Peter Maydell --- exec.c | 1 + 1 file changed, 1 insertion(+) diff --git a/exec.c b/exec.c index ffa1099547..bd