Created attachment 26194 [details] Don't dereference null when generating the log message PersistentValve.invoke() has the code: if (session != null && ...){ } else { session.isValid() // line 186 It's quite possible that session is null in the log message.
This was fixed as part of the general clean-up undertaken when getting trunk to build with Checkstyle's unused imports check enabled. The fix will be in 7.0.5 onwards.
The patch was proposed for 6.0 and 5.5.
Fixed in 6.0 by r1028554, will be in 6.0.30.
Fixed in 5.5 by r1028556, will be in 5.5.32.