From: Julia Lawall <julia@xxxxxxx> Date: Mon, 12 May 2008 22:21:25 +0200 (CEST) > If session is NULL, it is not possible to access its name field. So I have > split apart the printing of the error message to drop the printing of the > name field in this case. > > The macro PRINTK actually only evaluates its arguments starting with the > third one if the bitwise conjunction of the first two is non-zero. > Normally, this conjunction would only be non-zero if debugging mode were > turned on, but when session is NULL, the first argument in both the old and > new code is -1, and thus the bitwise conjunction is true. Perhaps a > different strategy is desired, such as using tunnel->debug, which > session->debug is initialized to, but tunnel can also be NULL, so this does > not completely solve the problem. ... > Signed-off-by: Julia Lawall <julia@xxxxxxx> Applied, thanks. -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html