I feel worried about the PR from 2 reasons
1) Do we need to log these exceptions? Currently we get a LOT of them and they are basically filling up elmah with noise.
2) Why are we catching NullReferenceException as opposed to avoid null ref using if statements etc? Actually I think it is very unusual to do this for good reason which is that it can easily catch issues you didn't intend to catch.