Fix possible NULL-dereference handling annotations

  Detected by Coverity Scan. Issue 1127088

diff --git a/src/auth/UserRequest.cc b/src/auth/UserRequest.cc
index d6b6bf1cab2bce6323d2208a6c59a54ae22354de..d4bf5e64527cfdbdd83acb125267b101a2616f04 100644 (file)
--- a/src/auth/UserRequest.cc
+++ b/src/auth/UserRequest.cc
@@ -258,7 +258,7 @@ authTryGetUser(Auth::UserRequest::Pointer auth_user_request, ConnStateData * con
         res = conn->getAuth();
 
     // attach the credential notes from helper to the transaction
-    if (res != NULL && res->user() != NULL) {
+    if (request != NULL && res != NULL && res->user() != NULL) {
         // XXX: we have no access to the transaction / AccessLogEntry so cant SyncNotes().
         // workaround by using anything already set in HttpRequest
         // OR use new and rely on a later Sync copying these to AccessLogEntry