[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] vl: fix -sandbox parsing crash when seccomp sup
From: |
Andrea Bolognani |
Subject: |
Re: [Qemu-devel] [PATCH] vl: fix -sandbox parsing crash when seccomp support is disabled |
Date: |
Mon, 29 Apr 2019 16:36:11 +0200 |
User-agent: |
Evolution 3.30.5 (3.30.5-1.fc29) |
On Mon, 2019-04-29 at 15:47 +0200, Marc-André Lureau wrote:
> $ ./x86_64-softmmu/qemu-system-x86_64 -sandbox off
> qemu-system-x86_64: -sandbox off: There is no option group 'sandbox'
> Segmentation fault
>
> Commit 5780760f5e ("seccomp: check TSYNC host capability") wrapped one
> use of the sandbox option group to produce a sensible error, it didn't
> do the same for another call to qemu_opts_parse_noisily():
>
> (gdb) bt
> at util/qemu-option.c:829
> #0 0x00000000105b36d8 in opts_parse (list=0x0, params=0x3ffffffffab5 "off",
> permit_abbrev=true, defaults=false, errp=0x3ffffffff080)
> at util/qemu-option.c:829
> #1 0x00000000105b3b74 in qemu_opts_parse_noisily (list=<optimized out>,
> params=<optimized out>, permit_abbrev=<optimized out>) at
> util/qemu-option.c:890
> #2 0x0000000010024964 in main (argc=<optimized out>, argv=<optimized out>,
> envp=<optimized out>) at vl.c:3589
>
> Fixes: 5780760f5ea6163939a5dabe7427318b4f07d1a2
> Cc: address@hidden
> Cc: address@hidden
> Signed-off-by: Marc-André Lureau <address@hidden>
> ---
> vl.c | 18 ++++++++++--------
> 1 file changed, 10 insertions(+), 8 deletions(-)
This fixes the crash for me, so
Tested-by: Andrea Bolognani <address@hidden>
--
Andrea Bolognani / Red Hat / Virtualization