Do not assign nil values in ACLFilledChecklist::syncAle() (#240)

Being explicit about nil parameters protects (future) code from
dereferencing them.

diff --git a/src/acl/FilledChecklist.cc b/src/acl/FilledChecklist.cc
index b5ce7d21ac38958ce72e5c2ad1de349d37c0e5c3..27c49a8111405f53c26761f1289264e6ed8a9647 100644 (file)
--- a/src/acl/FilledChecklist.cc
+++ b/src/acl/FilledChecklist.cc
@@ -132,11 +132,11 @@ ACLFilledChecklist::syncAle(HttpRequest *adaptedRequest, const char *logUri) con
 {
     if (!al)
         return;
-    if (!al->adapted_request) {
+    if (adaptedRequest && !al->adapted_request) {
         al->adapted_request = adaptedRequest;
         HTTPMSGLOCK(al->adapted_request);
     }
-    if (al->url.isEmpty())
+    if (logUri && al->url.isEmpty())
         al->url = logUri;
 }