Peter Maydell <peter.mayd...@linaro.org> 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 <peter.mayd...@linaro.org>
Reviewed-by: Alex Bennée <alex.ben...@linaro.org> > --- > exec.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/exec.c b/exec.c > index ffa1099547..bd8833fc9d 100644 > --- a/exec.c > +++ b/exec.c > @@ -1124,6 +1124,7 @@ void cpu_abort(CPUState *cpu, const char *fmt, ...) > struct sigaction act; > sigfillset(&act.sa_mask); > act.sa_handler = SIG_DFL; > + act.sa_flags = 0; > sigaction(SIGABRT, &act, NULL); > } > #endif -- Alex Bennée
