On Wed, Jul 20, 2011 at 08:56:35PM +0200, Stefan Weil wrote: > cppcheck reports this error: > > qemu/hw/sh_intc.c:390: error: Possible null pointer dereference: > s - otherwise it is redundant to check if s is null at line 385 > > If s were NULL, the printf() statement would crash. > Setting braces fixes this bug. > > Signed-off-by: Stefan Weil <w...@mail.berlios.de> > --- > hw/sh_intc.c | 9 +++++---- > 1 files changed, 5 insertions(+), 4 deletions(-)
Aurelien Jarno is listed as active maintainer for this code. Patches should go through him. Stefan