Re: [PATCH -next] mtip32xx: fix potential NULL pointer dereference in mtip_timeout_function()

2012-11-08 Thread Jens Axboe
On 2012-11-08 10:35, Wei Yongjun wrote: > From: Wei Yongjun > > The dereference to port should be moved below the NULL test. > > dpatch engine is used to auto generate this patch. > (https://github.com/weiyj/dpatch) Thanks, it definitely doesn't make sense to check for !port after having

[PATCH -next] mtip32xx: fix potential NULL pointer dereference in mtip_timeout_function()

2012-11-08 Thread Wei Yongjun
From: Wei Yongjun The dereference to port should be moved below the NULL test. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun --- drivers/block/mtip32xx/mtip32xx.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff

[PATCH -next] mtip32xx: fix potential NULL pointer dereference in mtip_timeout_function()

2012-11-08 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn The dereference to port should be moved below the NULL test. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/block/mtip32xx/mtip32xx.c | 3

Re: [PATCH -next] mtip32xx: fix potential NULL pointer dereference in mtip_timeout_function()

2012-11-08 Thread Jens Axboe
On 2012-11-08 10:35, Wei Yongjun wrote: From: Wei Yongjun yongjun_...@trendmicro.com.cn The dereference to port should be moved below the NULL test. dpatch engine is used to auto generate this patch. (https://github.com/weiyj/dpatch) Thanks, it definitely doesn't make sense to check for