[Qemu-devel] [PATCH] target-arm/abi32: check for segfault in do_kernel_trap

2017-01-07 Thread Seraphime Kirkovski
n the case where src == dest: the code doesn't check whether put_user_u32 succeeds. This fixes both problems by sending a SIGSEGV when the destination address is inaccessible. Signed-off-by: Seraphime Kirkovski --- > As the patchew robot notes, our coding style wants braces on all >

[Qemu-devel] [PATCHv2] target-arm/abi32: check for segfault in do_kernel_trap

2017-01-06 Thread Seraphime Kirkovski
n the case where src == dest: the code doesn't check whether put_user_u32 succeeds. This fixes both problems by sending a SIGSEGV when the destination address is inaccessible. Signed-off-by: Seraphime Kirkovski --- This accounts for Peter Maydell's remarks. The refactoring here extra

Re: [Qemu-devel] [RESEND PATCH] target-arm/abi32: check for segfault in do_kernel_trap

2017-01-06 Thread Seraphime Kirkovski
do_kernel_trap more readable. On the other hand, this would clutter a bit git history. If it is acceptable, I will send a patch with those changes. If not, will try to find a cleaner solution without creating new functions. Cheers, Seraphime Kirkovski

[Qemu-devel] [RESEND PATCH] target-arm/abi32: check for segfault in do_kernel_trap

2017-01-02 Thread Seraphime Kirkovski
n the case where src == dest: the code doesn't check whether put_user_u32 succeeds. This fixes both problems by sending a SIGSEGV when the destination address is inaccessible. Signed-off-by: Seraphime Kirkovski --- linux-user/main.c | 24 +++- 1 file changed, 19 insert

[Qemu-devel] [PATCH] target-arm/abi32: check for segfault in do_kernel_trap

2016-12-19 Thread Seraphime Kirkovski
n the case where src == dest: the code doesn't check whether put_user_u32 succeeds. This fixes both problems by sending a SIGSEGV when the destination address is inaccessible. Signed-off-by: Seraphime Kirkovski --- linux-user/main.c | 24 +++- 1 file changed, 19 insert