From mboxrd@z Thu Jan 1 00:00:00 1970 From: Matthias Fischer To: development@lists.ipfire.org Subject: [PATCH] squid 3.4: remove obsolete patch files Date: Tue, 10 May 2016 18:50:25 +0200 Message-ID: <1462899025-9354-1-git-send-email-matthias.fischer@ipfire.org> MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="===============6770928185830127153==" List-Id: --===============6770928185830127153== Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Obsolete, no longer needed. Signed-off-by: Matthias Fischer --- src/patches/squid-3.4-13233.patch | 274 ------------------------------------= -- src/patches/squid-3.4-13234.patch | 33 ----- src/patches/squid-3.4-13235.patch | 97 -------------- 3 files changed, 404 deletions(-) delete mode 100644 src/patches/squid-3.4-13233.patch delete mode 100644 src/patches/squid-3.4-13234.patch delete mode 100644 src/patches/squid-3.4-13235.patch diff --git a/src/patches/squid-3.4-13233.patch b/src/patches/squid-3.4-13233.= patch deleted file mode 100644 index d657838..0000000 --- a/src/patches/squid-3.4-13233.patch +++ /dev/null @@ -1,274 +0,0 @@ ------------------------------------------------------------- -revno: 13233 -revision-id: squid3(a)treenet.co.nz-20160420063907-hcnf4qmcg5hbjc11 -parent: squid3(a)treenet.co.nz-20160330141410-t6p2dhzr8ri36fap -committer: Amos Jeffries -branch nick: 3.4 -timestamp: Wed 2016-04-20 18:39:07 +1200 -message: - cachemgr.cgi: use dynamic MemBuf for internal content generation - =20 - Using a fixed size buffer limits how big content lines can be. Modern - HTTP is fast reaching the point where such limits are problematic. - Also fixes incorrect uses of snprintf() by removing them. ------------------------------------------------------------- -# Bazaar merge directive format 2 (Bazaar 0.90) -# revision_id: squid3(a)treenet.co.nz-20160420063907-hcnf4qmcg5hbjc11 -# target_branch: http://bzr.squid-cache.org/bzr/squid3/3.4 -# testament_sha1: 161e86814f6f14d74557a3fa169b37b6601c08a1 -# timestamp: 2016-04-20 06:50:57 +0000 -# source_branch: http://bzr.squid-cache.org/bzr/squid3/3.4 -# base_revision_id: squid3(a)treenet.co.nz-20160330141410-\ -# t6p2dhzr8ri36fap -#=20 -# Begin patch -=3D=3D=3D modified file 'src/tests/stub_cbdata.cc' ---- src/tests/stub_cbdata.cc 2012-11-01 10:31:28 +0000 -+++ src/tests/stub_cbdata.cc 2016-04-20 06:39:07 +0000 -@@ -5,7 +5,13 @@ - #include "tests/STUB.h" -=20 - void cbdataRegisterWithCacheManager(void) STUB -- -+void *cbdataInternalAlloc(cbdata_type type, const char *, int sz) { -+ return xcalloc(1, sz); -+} -+void *cbdataInternalFree(void *p, const char *, int) { -+ xfree(p); -+ return nullptr; -+} - #if USE_CBDATA_DEBUG - void *cbdataInternalAllocDbg(cbdata_type type, const char *, int) STUB_RETV= AL(NULL) - void *cbdataInternalFreeDbg(void *p, const char *, int) STUB_RETVAL(NULL) - -=3D=3D=3D modified file 'src/tests/stub_mem.cc' ---- src/tests/stub_mem.cc 2012-08-29 07:29:35 +0000 -+++ src/tests/stub_mem.cc 2016-04-20 06:39:07 +0000 -@@ -5,7 +5,7 @@ - #include "squid.h" -=20 - #define STUB_API "stub_mem.cc" --#include "STUB.h" -+#include "tests/STUB.h" - #include "Mem.h" -=20 - void - -=3D=3D=3D modified file 'tools/Makefile.am' ---- tools/Makefile.am 2014-04-06 04:37:08 +0000 -+++ tools/Makefile.am 2016-04-20 06:39:07 +0000 -@@ -35,15 +35,24 @@ - stub_debug.cc: $(top_srcdir)/src/tests/stub_debug.cc - cp $(top_srcdir)/src/tests/stub_debug.cc . -=20 -+MemBuf.cc: $(top_srcdir)/src/MemBuf.cc -+ cp $(top_srcdir)/src/MemBuf.cc $@ -+ - time.cc: $(top_srcdir)/src/time.cc - cp $(top_srcdir)/src/time.cc . -=20 -+stub_cbdata.cc: $(top_srcdir)/src/tests/stub_cbdata.cc -+ cp $(top_srcdir)/src/tests/stub_cbdata.cc $@ -+ -+stub_mem.cc: $(top_srcdir)/src/tests/stub_mem.cc -+ cp $(top_srcdir)/src/tests/stub_mem.cc $@ -+ - # stock tools for unit tests - library independent versions of dlink_list - # etc. - # globals.cc is needed by test_tools.cc. - # Neither of these should be disted from here. - TESTSOURCES=3D test_tools.cc --CLEANFILES +=3D test_tools.cc stub_debug.cc time.cc -+CLEANFILES +=3D test_tools.cc MemBuf.cc stub_debug.cc time.cc stub_cbdata.c= c stub_mem.cc -=20 - ## ##### helper-mux ##### -=20 -@@ -74,7 +83,10 @@ - libexec_PROGRAMS =3D cachemgr$(CGIEXT) -=20 - cachemgr__CGIEXT__SOURCES =3D cachemgr.cc \ -+ MemBuf.cc \ -+ stub_cbdata.cc \ - stub_debug.cc \ -+ stub_mem.cc \ - test_tools.cc \ - time.cc -=20 - -=3D=3D=3D modified file 'tools/cachemgr.cc' ---- tools/cachemgr.cc 2015-01-09 10:32:57 +0000 -+++ tools/cachemgr.cc 2016-04-20 06:39:07 +0000 -@@ -35,6 +35,7 @@ - #include "getfullhostname.h" - #include "html_quote.h" - #include "ip/Address.h" -+#include "MemBuf.h" - #include "rfc1123.h" - #include "rfc1738.h" - #include "util.h" -@@ -460,8 +461,8 @@ - return url; - } -=20 --static const char * --munge_menu_line(const char *buf, cachemgr_request * req) -+static void -+munge_menu_line(MemBuf &out, const char *buf, cachemgr_request * req) - { - char *x; - const char *a; -@@ -469,15 +470,14 @@ - const char *p; - char *a_url; - char *buf_copy; -- static char html[2 * 1024]; -- -- if (strlen(buf) < 1) -- return buf; -- -- if (*buf !=3D ' ') -- return buf; -- -- buf_copy =3D x =3D xstrdup(buf); -+ -+ const char bufLen =3D strlen(buf); -+ if (bufLen < 1 || *buf !=3D ' ') { -+ out.append(buf, bufLen); -+ return; -+ } -+ -+ buf_copy =3D x =3D xstrndup(buf, bufLen); -=20 - a =3D xstrtok(&x, '\t'); -=20 -@@ -489,59 +489,56 @@ -=20 - /* no reason to give a url for a disabled action */ - if (!strcmp(p, "disabled")) -- snprintf(html, sizeof(html), "
  • %s (disabled).\n", d, a_url); -+ out.Printf("
  • %s (disabled).\n", d, a_url); - else - /* disable a hidden action (requires a password, but password is no= t in squid.conf) */ - if (!strcmp(p, "hidden")) -- snprintf(html, sizeof(html), "
  • %s (hidden)= .\n", d, a_url); -+ out.Printf("
  • %s (hidden).= \n", d, a_url); - else - /* disable link if authentication is required and we have no pa= ssword */ - if (!strcmp(p, "protected") && !req->passwd) -- snprintf(html, sizeof(html), "
  • %s (req= uires authentication).\n", -- d, menu_url(req, "authenticate"), a_url); -+ out.Printf("
  • %s (requires authentication).\n", -+ d, menu_url(req, "authenticate"), a_url); - else - /* highlight protected but probably available entries */ - if (!strcmp(p, "protected")) -- snprintf(html, sizeof(html), "
  • %s\n", -- a_url, d); -+ out.Printf("
  • %s\n", -+ a_url, d); -=20 - /* public entry or unknown type of protection */ - else -- snprintf(html, sizeof(html), "
  • %s\n", a_url, d); -+ out.Printf("
  • %s= \n", a_url, d); -=20 - xfree(a_url); -=20 - xfree(buf_copy); -- -- return html; - } -=20 --static const char * --munge_other_line(const char *buf, cachemgr_request * req) -+static void -+munge_other_line(MemBuf &out, const char *buf, cachemgr_request *) - { - static const char *ttags[] =3D {"td", "th"}; -=20 -- static char html[4096]; - static int table_line_num =3D 0; - static int next_is_header =3D 0; - int is_header =3D 0; - const char *ttag; - char *buf_copy; - char *x, *p; -- int l =3D 0; - /* does it look like a table? */ -=20 - if (!strchr(buf, '\t') || *buf =3D=3D '\t') { - /* nope, just text */ -- snprintf(html, sizeof(html), "%s%s", -- table_line_num ? "\n
    " : "", html_quote(buf));
-+        if (table_line_num)
-+            out.append("\n
    ", 14);
-+        out.Printf("%s", html_quote(buf));
-         table_line_num =3D 0;
--        return html;
-+        return;
-     }
-=20
-     /* start html table */
-     if (!table_line_num) {
--        l +=3D snprintf(html + l, sizeof(html) - l, "
    \n");
-+        out.append("
    \n=
", 46);
-         next_is_header =3D 0;
-     }
-=20
-@@ -551,7 +548,7 @@
-     ttag =3D ttags[is_header];
-=20
-     /* record starts */
--    l +=3D snprintf(html + l, sizeof(html) - l, "");
-+    out.append("", 4);
-=20
-     /* substitute '\t' */
-     buf_copy =3D x =3D xstrdup(buf);
-@@ -568,18 +565,17 @@
-             ++x;
-         }
-=20
--        l +=3D snprintf(html + l, sizeof(html) - l, "<%s colspan=3D\"%d\" a=
lign=3D\"%s\">%s",
--                      ttag, column_span,
--                      is_header ? "center" : is_number(cell) ? "right" : "l=
eft",
--                      html_quote(cell), ttag);
-+        out.Printf("<%s colspan=3D\"%d\" align=3D\"%s\">%s",
-+                   ttag, column_span,
-+                   is_header ? "center" : is_number(cell) ? "right" : "left=
",
-+                   html_quote(cell), ttag);
-     }
-=20
-     xfree(buf_copy);
-     /* record ends */
--    snprintf(html + l, sizeof(html) - l, "\n");
-+    out.append("\n", 6);
-     next_is_header =3D is_header && strstr(buf, "\t\t");
-     ++table_line_num;
--    return html;
- }
-=20
- static const char *
-@@ -736,14 +732,18 @@
-             /* yes, fall through, we do not want to loose the first line */
-=20
-         case isBody:
-+        {
-             /* interpret [and reformat] cache response */
--
-+            MemBuf out;
-+            out.init();
-             if (parse_menu)
--                fputs(munge_menu_line(buf, req), stdout);
-+                munge_menu_line(out, buf, req);
-             else
--                fputs(munge_other_line(buf, req), stdout);
-+                munge_other_line(out, buf, req);
-=20
--            break;
-+            fputs(out.buf, stdout);
-+        }
-+        break;
-=20
-         case isForward:
-             /* forward: no modifications allowed */
-
diff --git a/src/patches/squid-3.4-13234.patch b/src/patches/squid-3.4-13234.=
patch
deleted file mode 100644
index 6f701fa..0000000
--- a/src/patches/squid-3.4-13234.patch
+++ /dev/null
@@ -1,33 +0,0 @@
-------------------------------------------------------------
-revno: 13234
-revision-id: squid3(a)treenet.co.nz-20160420101437-36eofkldxfku61kj
-parent: squid3(a)treenet.co.nz-20160420063907-hcnf4qmcg5hbjc11
-committer: Amos Jeffries 
-branch nick: 3.4
-timestamp: Wed 2016-04-20 22:14:37 +1200
-message:
-  nullptr is a C++11 feature
-------------------------------------------------------------
-# Bazaar merge directive format 2 (Bazaar 0.90)
-# revision_id: squid3(a)treenet.co.nz-20160420101437-36eofkldxfku61kj
-# target_branch: http://bzr.squid-cache.org/bzr/squid3/3.4
-# testament_sha1: 0f9f2a5b29a7ef02befe2a4e2e6357ef4bcffbce
-# timestamp: 2016-04-20 10:51:03 +0000
-# source_branch: http://bzr.squid-cache.org/bzr/squid3/3.4
-# base_revision_id: squid3(a)treenet.co.nz-20160420063907-\
-#   hcnf4qmcg5hbjc11
-#=20
-# Begin patch
-=3D=3D=3D modified file 'src/tests/stub_cbdata.cc'
---- src/tests/stub_cbdata.cc	2016-04-20 06:39:07 +0000
-+++ src/tests/stub_cbdata.cc	2016-04-20 10:14:37 +0000
-@@ -10,7 +10,7 @@
- }
- void *cbdataInternalFree(void *p, const char *, int) {
-     xfree(p);
--    return nullptr;
-+    return NULL;
- }
- #if USE_CBDATA_DEBUG
- void *cbdataInternalAllocDbg(cbdata_type type, const char *, int) STUB_RETV=
AL(NULL)
-
diff --git a/src/patches/squid-3.4-13235.patch b/src/patches/squid-3.4-13235.=
patch
deleted file mode 100644
index e380225..0000000
--- a/src/patches/squid-3.4-13235.patch
+++ /dev/null
@@ -1,97 +0,0 @@
-------------------------------------------------------------
-revno: 13235
-revision-id: squid3(a)treenet.co.nz-20160420111514-4hpxglbn9k15l5sa
-parent: squid3(a)treenet.co.nz-20160420101437-36eofkldxfku61kj
-committer: Amos Jeffries 
-branch nick: 3.4
-timestamp: Wed 2016-04-20 23:15:14 +1200
-message:
-  Fix several ESI element construction issues
- =20
-  * Do not wrap active logic in assert().
- =20
-  * Fix localbuf array bounds checking.
- =20
-  * Add Must() conditions to verify array writes will succeed
-------------------------------------------------------------
-# Bazaar merge directive format 2 (Bazaar 0.90)
-# revision_id: squid3(a)treenet.co.nz-20160420111514-4hpxglbn9k15l5sa
-# target_branch: http://bzr.squid-cache.org/bzr/squid3/3.4
-# testament_sha1: e95687b13c98667ab09966e7f94d511ca3e6ad96
-# timestamp: 2016-04-20 11:18:22 +0000
-# source_branch: http://bzr.squid-cache.org/bzr/squid3/3.4
-# base_revision_id: squid3(a)treenet.co.nz-20160420101437-\
-#   36eofkldxfku61kj
-#=20
-# Begin patch
-=3D=3D=3D modified file 'src/esi/Esi.cc'
---- src/esi/Esi.cc	2013-06-27 15:58:46 +0000
-+++ src/esi/Esi.cc	2016-04-20 11:15:14 +0000
-@@ -991,7 +991,7 @@
-     ESIElement::Pointer element;
-     int specifiedattcount =3D attrCount * 2;
-     char *position;
--    assert (ellen < sizeof (localbuf)); /* prevent unexpected overruns. */
-+    Must(ellen < sizeof(localbuf)); /* prevent unexpected overruns. */
-=20
-     debugs(86, 5, "ESIContext::Start: element '" << el << "' with " << spec=
ifiedattcount << " tags");
-=20
-@@ -1005,15 +1005,17 @@
-         /* Spit out elements we aren't interested in */
-         localbuf[0] =3D '<';
-         localbuf[1] =3D '\0';
--        assert (xstrncpy (&localbuf[1], el, sizeof(localbuf) - 2));
-+        xstrncpy(&localbuf[1], el, sizeof(localbuf) - 2);
-         position =3D localbuf + strlen (localbuf);
-=20
-         for (i =3D 0; i < specifiedattcount && attr[i]; i +=3D 2) {
-+            Must(static_cast(position - localbuf) < sizeof(localbuf=
) - 1);
-             *position =3D ' ';
-             ++position;
-             /* TODO: handle thisNode gracefully */
--            assert (xstrncpy (position, attr[i], sizeof(localbuf) + (positi=
on - localbuf)));
-+            xstrncpy(position, attr[i], sizeof(localbuf) - (position - loca=
lbuf));
-             position +=3D strlen (position);
-+            Must(static_cast(position - localbuf) < sizeof(localbuf=
) - 2);
-             *position =3D '=3D';
-             ++position;
-             *position =3D '\"';
-@@ -1022,18 +1024,21 @@
-             char ch;
-             while ((ch =3D *chPtr++) !=3D '\0') {
-                 if (ch =3D=3D '\"') {
--                    assert( xstrncpy(position, """, sizeof(localbuf) +=
 (position-localbuf)) );
-+                    Must(static_cast(position - localbuf) < sizeof(=
localbuf) - 6);
-+                    xstrncpy(position, """, sizeof(localbuf) - (positi=
on-localbuf));
-                     position +=3D 6;
-                 } else {
-+                    Must(static_cast(position - localbuf) < sizeof(=
localbuf) - 1);
-                     *position =3D ch;
-                     ++position;
-                 }
-             }
--            position +=3D strlen (position);
-+            Must(static_cast(position - localbuf) < sizeof(localbuf=
) - 1);
-             *position =3D '\"';
-             ++position;
-         }
-=20
-+        Must(static_cast(position - localbuf) < sizeof(localbuf) - =
2);
-         *position =3D '>';
-         ++position;
-         *position =3D '\0';
-@@ -1119,11 +1124,11 @@
-     switch (ESIElement::IdentifyElement (el)) {
-=20
-     case ESIElement::ESI_ELEMENT_NONE:
--        assert (ellen < sizeof (localbuf)); /* prevent unexpected overruns.=
 */
-+        Must(ellen < sizeof(localbuf) - 3); /* prevent unexpected overruns.=
 */
-         /* Add elements we aren't interested in */
-         localbuf[0] =3D '<';
-         localbuf[1] =3D '/';
--        assert (xstrncpy (&localbuf[2], el, sizeof(localbuf) - 3));
-+        xstrncpy(&localbuf[2], el, sizeof(localbuf) - 3);
-         position =3D localbuf + strlen (localbuf);
-         *position =3D '>';
-         ++position;
-
--=20
2.8.2


--===============6770928185830127153==--