summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorEray Aslan <eras@gentoo.org>2016-04-18 15:40:46 +0000
committerEray Aslan <eras@gentoo.org>2016-04-18 15:40:46 +0000
commit39f4399ca5ae5f49641b3d93ca95c82af3b8c961 (patch)
tree6e738748af975bc5515355ca44b7a8ed7601f50a /net-proxy/squid/files
parentprofiles/prefix/cygwin/2.0: for coreutils add p.use kill (diff)
downloadgentoo-39f4399ca5ae5f49641b3d93ca95c82af3b8c961.tar.gz
gentoo-39f4399ca5ae5f49641b3d93ca95c82af3b8c961.tar.bz2
gentoo-39f4399ca5ae5f49641b3d93ca95c82af3b8c961.zip
net-proxy/squid: Include upstream fix for varyEvaluateMatch
Fixes upstream bug 4481 Package-Manager: portage-2.2.28
Diffstat (limited to 'net-proxy/squid/files')
-rw-r--r--net-proxy/squid/files/squid-3.5-14022.patch50
1 files changed, 50 insertions, 0 deletions
diff --git a/net-proxy/squid/files/squid-3.5-14022.patch b/net-proxy/squid/files/squid-3.5-14022.patch
new file mode 100644
index 000000000000..4b598b643b72
--- /dev/null
+++ b/net-proxy/squid/files/squid-3.5-14022.patch
@@ -0,0 +1,50 @@
+------------------------------------------------------------
+revno: 14022
+revision-id: squid3@treenet.co.nz-20160411131130-z2lp1uj94sovgthy
+parent: squid3@treenet.co.nz-20160411124138-zsrf0ltaxlj25k2x
+fixes bug: http://bugs.squid-cache.org/show_bug.cgi?id=4481
+committer: Amos Jeffries <squid3@treenet.co.nz>
+branch nick: 3.5
+timestamp: Tue 2016-04-12 01:11:30 +1200
+message:
+ Bug 4481: varyEvaluateMatch: Oops. Not a Vary match on second attempt
+------------------------------------------------------------
+# Bazaar merge directive format 2 (Bazaar 0.90)
+# revision_id: squid3@treenet.co.nz-20160411131130-z2lp1uj94sovgthy
+# target_branch: http://bzr.squid-cache.org/bzr/squid3/3.5
+# testament_sha1: 91af3a71dfc316577c5661733fc0c9fdeeb8ba83
+# timestamp: 2016-04-11 13:50:55 +0000
+# source_branch: http://bzr.squid-cache.org/bzr/squid3/3.5
+# base_revision_id: squid3@treenet.co.nz-20160411124138-\
+# zsrf0ltaxlj25k2x
+#
+# Begin patch
+=== modified file 'src/StoreMetaVary.cc'
+--- src/StoreMetaVary.cc 2016-04-01 06:15:31 +0000
++++ src/StoreMetaVary.cc 2016-04-11 13:11:30 +0000
+@@ -22,6 +22,9 @@
+ /* XXX separate this mutator from the query */
+ /* Assume the object is OK.. remember the vary request headers */
+ e->mem_obj->vary_headers.assign(static_cast<const char *>(value), length);
++ /* entries created before SBuf vary handling may include string terminator */
++ static const SBuf nul("\0", 1);
++ e->mem_obj->vary_headers.trim(nul);
+ return true;
+ }
+
+
+=== modified file 'src/store_swapmeta.cc'
+--- src/store_swapmeta.cc 2016-04-01 06:15:31 +0000
++++ src/store_swapmeta.cc 2016-04-11 13:11:30 +0000
+@@ -89,9 +89,7 @@
+ SBuf vary(e->mem_obj->vary_headers);
+
+ if (!vary.isEmpty()) {
+- // TODO: do we still need +1 here? StoreMetaVary::checkConsistency
+- // no longer relies on nul-termination, but other things might.
+- t = StoreMeta::Factory(STORE_META_VARY_HEADERS, vary.length() + 1, vary.c_str());
++ t = StoreMeta::Factory(STORE_META_VARY_HEADERS, vary.length(), vary.c_str());
+
+ if (!t) {
+ storeSwapTLVFree(TLV);
+