Cleanup: fix assertion in Store unit tests

The old Squid String implementation cannot handle appending nullptr or
negative lengths. So if the test code using CapturingStoreEntry ever
tries to append such it will crash instead of working like a StoreEntry
should.

diff --git a/src/tests/CapturingStoreEntry.h b/src/tests/CapturingStoreEntry.h
index e8f010b54f58057a8bb70adbed6a14751f983501..7736d53254a9ae5442258390ef4cdf316cbebd03 100644 (file)
--- a/src/tests/CapturingStoreEntry.h
+++ b/src/tests/CapturingStoreEntry.h
@@ -35,6 +35,8 @@ public:
     }
 
     virtual void append(char const * buf, int len) {
+        if (!buf || len < 0) // old 'String' can't handle these cases
+            return;
         _appended_text.append(buf, len);
     }
 };