]> git.ipfire.org Git - thirdparty/squid.git/log
thirdparty/squid.git
11 years agoDocs: Update http(s)_port protocol= details
Amos Jeffries [Fri, 26 Jul 2013 05:21:29 +0000 (23:21 -0600)] 
Docs: Update http(s)_port protocol= details

11 years agoUpdate the http(s)_port directives protocol= parameter
Amos Jeffries [Fri, 26 Jul 2013 02:06:25 +0000 (20:06 -0600)] 
Update the http(s)_port directives protocol= parameter

... to use AnyP::ProtocolVersion for internal storage instead of opaque
string text.

This both limits the possible parameter values to one of HTTP, HTTP/1.1,
HTTPS or HTTPS/1.1 and ensures that URLs generated from that protocol
parameter value are http:// and https:// URL.
Other values will cause Squid to abort.

11 years agoRelease Notes: fix typo in ./configure options
Amos Jeffries [Thu, 25 Jul 2013 13:59:35 +0000 (07:59 -0600)] 
Release Notes: fix typo in ./configure options

11 years agoMake GCC 4.8 happy with libip code
Amos Jeffries [Thu, 25 Jul 2013 13:06:58 +0000 (07:06 -0600)] 
Make GCC 4.8 happy with libip code

Invalid casting seems to confuse the ABI generator and results in
illegal instruction faults when the unit tests is run.
The class API is already const-correct so there is no need for the cast
to occur, and it should not be done on a non-pointer type anyway.

Also, fixes a missing "struct" type identifier found along the way.

11 years agoFix assert with side effects in ServerStateData::handleAdaptedHeader
Nathan Hoad [Thu, 25 Jul 2013 07:21:54 +0000 (01:21 -0600)] 
Fix assert with side effects in ServerStateData::handleAdaptedHeader

setConsumerIfNotLate conditionally schedules some async calls, so the
call definitely shouldn't be removed by the precompiler.

11 years agoFix mgr:info report 'Largest file desc currently in use' with SMP
Amos Jeffries [Thu, 25 Jul 2013 03:42:06 +0000 (21:42 -0600)] 
Fix mgr:info report 'Largest file desc currently in use' with SMP

The biggest FD in use should not be the sum of biggest FD in each worker,
but the maximum across all workers.

11 years agoBranch 3.4.0.0 and disable snapshots 3.1 and 3.2
Automatic source maintenance [Mon, 22 Jul 2013 09:22:09 +0000 (03:22 -0600)] 
Branch 3.4.0.0 and disable snapshots 3.1 and 3.2

11 years agoRelease Notes: fix typo
Amos Jeffries [Mon, 22 Jul 2013 05:15:37 +0000 (23:15 -0600)] 
Release Notes: fix typo

11 years agoRelease notes for multicast DNS support
Amos Jeffries [Mon, 22 Jul 2013 02:13:50 +0000 (20:13 -0600)] 
Release notes for multicast DNS support

11 years agoRelease notes for rev.12945
Amos Jeffries [Mon, 22 Jul 2013 01:57:39 +0000 (19:57 -0600)] 
Release notes for rev.12945

11 years agoFix return value of ConfigParser::strtokFile()
Amos Jeffries [Mon, 22 Jul 2013 01:26:09 +0000 (19:26 -0600)] 
Fix return value of ConfigParser::strtokFile()

11 years agoRelease Notes: document squid.conf quoting changes
Amos Jeffries [Sun, 21 Jul 2013 23:23:13 +0000 (17:23 -0600)] 
Release Notes: document squid.conf quoting changes

11 years agoRelease Notes: update squid.conf directive status
Amos Jeffries [Sun, 21 Jul 2013 23:02:09 +0000 (17:02 -0600)] 
Release Notes: update squid.conf directive status

11 years agoQuoted values in squid.conf
Christos Tsantilas [Sun, 21 Jul 2013 19:24:35 +0000 (22:24 +0300)] 
Quoted values in squid.conf

This patch :
  - adds support for quoted values in the entire squid.conf
  - warn about or prohibit values that can no longer be interpreted as
    either quoted strings or simple tokens
  - support parameters("/path/to/file.name") syntax to load external
    configuration files
  - Checks if macros allowed in "double quoted" values.
  - replaces the strtok() calls with calls to the new ConfigParser::NextToken()
  - modify strtokFile to use new ConfigParser::NextToken()
  - Add the new configuration_includes_quoted_values configuration option, to
    control the squid parser behaviour. If set to on Squid will recognize each
    "quoted string" after a configuration directive as a single parameter

This is a Measurement Factory project

11 years agoBug 3879: assertion failed ConnStateData::validatePinnedConnection
Martin Stolle [Thu, 18 Jul 2013 05:02:23 +0000 (23:02 -0600)] 
Bug 3879: assertion failed ConnStateData::validatePinnedConnection

11 years agoDocumented space() methods to explain why bugs like 3869 keep happening.
Alex Rousskov [Thu, 18 Jul 2013 02:07:31 +0000 (20:07 -0600)] 
Documented space() methods to explain why bugs like 3869 keep happening.

Hopefully we will not repeat these mistakes with StringNG.

11 years agoSourceFormat Enforcement
Automatic source maintenance [Tue, 16 Jul 2013 00:13:03 +0000 (18:13 -0600)] 
SourceFormat Enforcement

11 years agoLog TOS and Netfilter marks set by Squid
Christos Tsantilas [Mon, 15 Jul 2013 15:47:00 +0000 (18:47 +0300)] 
Log TOS and Netfilter marks set by Squid

This patch add new logformat codes to log TOS/DSCP values and netfilter marks
for client and server connections. If multiple outgoing connections were used,
the last used connection value logged.
The values printed in hexadecimal form.
The logformat codes are:
   %>tos    Client connection tos mark set by Squid
   %<tos    Server connection tos mark set by Squid
   %>nfmark Client connection netfilter mark set by Squid
   %<nfmark Server connection netfilter mark set by Squid

This patch also modify qos related code to set Comm::Connection::nfmark and
Comm::Connection::tos members in Ip::Qos::setSockNfmark and Ip::Qos::setSockTos
methods. The Comm::Connection members are now set only if the tos and nfmark
set successfuly.

This is a Measurement Factory project

11 years agoBug 2287: Better fix for unsupported HTP version handling
Amos Jeffries [Mon, 15 Jul 2013 13:41:51 +0000 (07:41 -0600)] 
Bug 2287: Better fix for unsupported HTP version handling

RFC 2616 section 10.5.6 is more specifically about MAJOR versions.
Remove the criterion limiting 1.x minor version numbers.

11 years agoReload into If-None-Match
Christos Tsantilas [Mon, 15 Jul 2013 07:49:43 +0000 (10:49 +0300)] 
Reload into If-None-Match

This patch sends an If-None-Match  request, when we need to re-validate
if a cached object which has a strong ETag is still valid.

This is also done in the cases an HTTP client request contains HTTP
headers prohibiting a from-cache response (i.e., a "reload" request).
The use of If-None-Match request in this context violates RFC 2616 and
requires using reload-into-ims option within refresh_pattern squid.conf
directive.

The exact definition of a "reload request" and the adjustment/removal of
"reload" headers is the same as currently used for reload-into-ims
option support. This patch is not modifying that code/logic, just adding
an If-None-Match header in addition to the IMS header that Squid already
adds.

This is a Measurement Factory Project

11 years agoFix stupid typo in rev.12941
Amos Jeffries [Sun, 14 Jul 2013 03:45:23 +0000 (21:45 -0600)] 
Fix stupid typo in rev.12941

11 years agoFix external ACL user:pass detail logging after adaptation
Nathan Hoad [Sun, 14 Jul 2013 03:03:53 +0000 (21:03 -0600)] 
Fix external ACL user:pass detail logging after adaptation

When a request is successfully adapted, the external ACL username and
password are now inherited with this patch. This means the
LFT_USER_NAME log token can display the username from an external ACL
if available, for adapted requests.

The HttpRequest will inherit the password for good measure as well -
while none too useful, it seems strange to inherit the username but
not the password.

11 years agoFix assert() with side effects
Amos Jeffries [Sun, 14 Jul 2013 03:00:32 +0000 (21:00 -0600)] 
Fix assert() with side effects

assert() may be completely removed by precompiler when optimizing. Such
that the code conditional never gets run.

11 years agoPrep for 3.3.8 and 3.2.13
Amos Jeffries [Sat, 13 Jul 2013 13:08:28 +0000 (07:08 -0600)] 
Prep for 3.3.8 and 3.2.13

11 years agoBetter handling of strange port values in Host:
Amos Jeffries [Sat, 13 Jul 2013 12:19:45 +0000 (00:19 +1200)] 
Better handling of strange port values in Host:

We can do better than just producing errors about invalid port details
and treatign it as port-0.
We can instead undo the port separation and pass it through as part of
the host name to be verified with the default port number properly
assumed.

11 years agoBug 3869: assertion failed: MemBuf.cc:272: size < capacity
Amos Jeffries [Fri, 12 Jul 2013 16:08:09 +0000 (10:08 -0600)] 
Bug 3869: assertion failed: MemBuf.cc:272: size < capacity

11 years agoPrep for 3.2.12 and 3.3.7
Amos Jeffries [Thu, 11 Jul 2013 04:38:47 +0000 (22:38 -0600)] 
Prep for 3.2.12 and 3.3.7

11 years agoProtect against buffer overrun in DNS query generation
Nathan Hoad [Wed, 10 Jul 2013 12:38:36 +0000 (06:38 -0600)] 
Protect against buffer overrun in DNS query generation

see SQUID-2013:2.

This bug has been present as long as the internal DNS component however
most code reaching this point is passing through URL validation first.
With Squid-3.2 Host header verification using DNS directly we may have
problems.

11 years agoSourceFormat Enforcement
Automatic source maintenance [Wed, 10 Jul 2013 00:13:08 +0000 (18:13 -0600)] 
SourceFormat Enforcement

11 years agoRelease Notes: more 3.4 documentation
Amos Jeffries [Tue, 9 Jul 2013 13:29:05 +0000 (07:29 -0600)] 
Release Notes: more 3.4 documentation

11 years agoAdd storeid_file_rewrite helper
Alan Mizrahi [Tue, 9 Jul 2013 11:15:51 +0000 (05:15 -0600)] 
Add storeid_file_rewrite helper

Based on work by Eliezer Croitoru <eliezer@ngtech.co.il>

This program acts as a Store-ID helper program, rewriting URLs passed
by Squid into storage-ids that can be used to achieve better caching
for websites that use different URLs for the same content.

It takes a text file with two tab separated columns.
 Column 1: Regular expression to match against the URL
 Column 2: Rewrite rule to generate a Store-ID

Rewrite rules are matched in the same order as they appear in the file.
So for best performance, sort it in order of frequency of occurrence.

11 years agoSupport IPv6 NAT interception on Linux
Alexis Robert [Tue, 9 Jul 2013 10:04:39 +0000 (22:04 +1200)] 
Support IPv6 NAT interception on Linux

NAT support has been included for IPv6 in Linux 3.7 (along with
REDIRECT/DNAT rules), as well as IP6T_SO_ORIGINAL_DST in Linux 3.8.
Add support for transparent proxies over IPv6.

There is a bug in linux/netfilter_ipv6/ip6_tables.h on C++ compilers,
the bug report and patch to fix it can be found at
  https://lkml.org/lkml/2012/9/30/146.
It is only used for the constant IP6T_SO_ORIGINAL_DST. We attempt to use
the official header whenever possible but if it is detected missing or
broken we define our own version of the option.

IPv6 is now permitted on any http_port or https_port in squid.conf
however on older Linux systems and Unix systems without the required NAT
support Squid will fail when accepting the traffic.

Also, this removes the blocker checks preventing BSD systems using NAT
interception on IPv6 ports. Several version of PF have long since
supported IPv6 NAT operations although it was discouraged, such support
is not easily detected though so results WILL vary by operating system.

11 years agoBug 3876: mDNS support segfault when using --disable-ipv6
Amos Jeffries [Tue, 9 Jul 2013 05:31:57 +0000 (17:31 +1200)] 
Bug 3876: mDNS support segfault when using --disable-ipv6

When IPv6 is disabled the mDNS IPv6 multicast group gets rejected by
idnsAddnameserver() resulting in invalid pointers for the remaining
mDNS NS setup operations.

Convert the hard-coded mDNS nameserver count to dynamic global count and
elide the relevant NS when IPv6 support disabled.

11 years agoCleanup: remove duplicate includes
Amos Jeffries [Sat, 6 Jul 2013 15:11:43 +0000 (09:11 -0600)] 
Cleanup: remove duplicate includes

Remove duplicate includes of system files already pulled in by compat/types.h

11 years agoRelease Notes: initial 3.4 changelog
Amos Jeffries [Thu, 4 Jul 2013 08:05:33 +0000 (02:05 -0600)] 
Release Notes: initial 3.4 changelog

11 years agoBootstrapped
Automatic source maintenance [Thu, 4 Jul 2013 00:10:48 +0000 (18:10 -0600)] 
Bootstrapped

11 years agoFix build on FreeBSD 9.x platform with clang
Christos Tsantilas [Wed, 3 Jul 2013 17:34:42 +0000 (20:34 +0300)] 
Fix build on FreeBSD 9.x platform with clang

Add the "/usr/local/*" directories to include files  and libraries  search
paths, for clang compiler too.

11 years agoBug 3297: Fix openSSL related build failures
Christos Tsantilas [Sun, 30 Jun 2013 15:54:22 +0000 (18:54 +0300)] 
Bug 3297: Fix openSSL related build failures

- The SSL_CTX_new in newer openSSL releases requires a const
'SSL_METHOD *' argument and in older releases requires non const
'SSL_METHD *' argument. Currently we are trying to identify openSSL
version using the OPENSSL_VERSION_NUMBER macro define but we are failing
to correctly identify all cases.

- sk_OPENSSL_PSTRING_value is buggy in early openSSL-1.0.0? releases
causing compile errors to squid.

This is a Measurement Factory project

11 years agoPrep for 3.3.6
Amos Jeffries [Sun, 30 Jun 2013 08:52:51 +0000 (02:52 -0600)] 
Prep for 3.3.6

11 years agoBug 3762: remove bogus WARNING in cache.log
Amos Jeffries [Sat, 29 Jun 2013 14:43:23 +0000 (08:43 -0600)] 
Bug 3762: remove bogus WARNING in cache.log

The warning is bogus for several reasons:
* it appears with memory-only cache configurations
* it only checks the size of first SwapDir (as seen in bug 3762)
* very large memory spaces are now possible which may make disk appear
  small by comparison.

Its usefulness in detecting memory and disk misconfigurations has long
been almosy nil. Removing this entirey to resolve the bogus noise in
the above mentinoed legitimate configurations.

11 years agoBug 3842: Segfault during connection close in ConnStateData::stopReceiving()
Amos Jeffries [Sat, 29 Jun 2013 11:37:08 +0000 (05:37 -0600)] 
Bug 3842: Segfault during connection close in ConnStateData::stopReceiving()

11 years agoCleanup: Remove last CBDATA_CLASS() macros
Amos Jeffries [Sat, 29 Jun 2013 04:21:47 +0000 (22:21 -0600)] 
Cleanup: Remove last CBDATA_CLASS() macros

Update the final CBDATA_CLASS() macro usages to CBDATA_CLASS2() and
remove the now defunct macro.

11 years agoCleanup: update several CBDATA classes part 2
Amos Jeffries [Thu, 27 Jun 2013 15:58:46 +0000 (09:58 -0600)] 
Cleanup: update several CBDATA classes part 2

Removes several cbdataAlloc/cbdataFree by updating classes to use
CBDATA_CLASS2 macro..

11 years agoFix Ip::Address::operator =(sockaddr_storage)
Alexis Robert [Mon, 24 Jun 2013 07:42:35 +0000 (01:42 -0600)] 
Fix Ip::Address::operator =(sockaddr_storage)

The memcpy() for AF_INET6 is using a length of sizeof(sockaddr_in) instead
of sizeof(sockaddr_in6), so squid was trying to connect to truncatured IPv6
addresses with strange ports.

11 years agoRedirector helpers related changes
Christos Tsantilas [Thu, 20 Jun 2013 19:21:02 +0000 (22:21 +0300)] 
Redirector helpers related changes

- The redirectStateData handlers requires the HelperReply::Okay helper reply
  result code else will drop the helper reply, but we are always pass to them
  the HelperReply::Unknown reply result code

- The NotePairs are not support "=" operator. This patch replaces a such command
  using the NotePairs::append member, and also adds unimplemented private
  = operator and copy constructor to prevent developers from using it.

11 years agoIntegrate libTrie into squid build process
Francesco Chemolli [Thu, 20 Jun 2013 10:01:48 +0000 (12:01 +0200)] 
Integrate libTrie into squid build process

11 years agoMerged from trunk
Francesco Chemolli [Thu, 20 Jun 2013 09:56:11 +0000 (11:56 +0200)] 
Merged from trunk

11 years agoCleanup: update several CBDATA classes
Amos Jeffries [Wed, 19 Jun 2013 04:59:56 +0000 (22:59 -0600)] 
Cleanup: update several CBDATA classes

Removes several cbdataAlloc/cbdataFree by updating classes to use
CBDATA_CLASS2 macro..

11 years agoAdd Master Transaction class
Amos Jeffries [Tue, 18 Jun 2013 23:26:17 +0000 (17:26 -0600)] 
Add Master Transaction class

... to store and propigate the shared state used end-to-end through Squid
for logging or server-side component input. This excludes Job and Call
pointers, but does include any 'factual' data regarding the traansaction.

11 years agoMake sure %<tt includes all [failed] connection attempts.
Alex Rousskov [Tue, 18 Jun 2013 22:30:39 +0000 (16:30 -0600)] 
Make sure %<tt includes all [failed] connection attempts.

The old code was using zero n_tries to detect the first connection attempt,
but n_tries is not incremented when we are opening a new connection rather
than reusing an old one. Perhaps n_tries should be updated differently as
well, but this change simply makes %<tt (hier.total_response_time) management
independent from that [complex] counter.

11 years agoSending root certificate for validation
Christos Tsantilas [Tue, 18 Jun 2013 10:23:41 +0000 (13:23 +0300)] 
Sending root certificate for validation

This patch modify squid cert validation subsystem to sent to cert validator
helper the complete certificates chain, not only the certificates sent by
web server. This is may not be possible in all cases, for example  in cases
where the root certificate is not stored localy.

Also this patch includes a small optimization, it checks for domain mismatch
error only when the checked (current) certificate is the server certificate.

This is a Measurement Factory project

11 years agoDeprecate log_icap and log_access configuration directives
Christos Tsantilas [Tue, 18 Jun 2013 06:22:13 +0000 (09:22 +0300)] 
Deprecate log_icap and log_access configuration directives

The log_icap and log_access are not really needed to control requests logging.
Someone can use acls  with access_log and icap_log configuration directives
for this purpose.
Also currently the requests denied for logging using the log_access access list
will not be accounted for in performance counters.

This patch:
 - removes log_icap and log_access options from configuration file.
 - adds the "stats_collection" access list to control performane counters
   accounting.

This is a Measurement Factory project

11 years agoSourceLayout: Add anyp/forward.h predefinition
Amos Jeffries [Thu, 13 Jun 2013 11:08:58 +0000 (05:08 -0600)] 
SourceLayout: Add anyp/forward.h predefinition

Provide the AnyP:: namespace predefines to reduce dependencies.

11 years agoRemoved useless include compiler flags
Francesco Chemolli [Tue, 11 Jun 2013 15:16:07 +0000 (17:16 +0200)] 
Removed useless include compiler flags

11 years agoMerged from trunk
Francesco Chemolli [Tue, 11 Jun 2013 13:15:01 +0000 (15:15 +0200)] 
Merged from trunk

11 years agoLibTrie refactro: remove sub-configure invocation
Francesco Chemolli [Tue, 11 Jun 2013 13:13:23 +0000 (15:13 +0200)] 
LibTrie refactro: remove sub-configure invocation

11 years agoFix icap_log and log_icap acls checking
Christos Tsantilas [Tue, 11 Jun 2013 10:04:48 +0000 (13:04 +0300)] 
Fix icap_log and log_icap acls checking

The HTTP reply related acls does not check for icap_log

11 years agoFix request headers logging for icap_log
Christos Tsantilas [Tue, 11 Jun 2013 09:55:05 +0000 (12:55 +0300)] 
Fix request headers logging for icap_log

The %http::>h does not print the original request headers but the adapted
request headers. Also the %http::>ha does not print adapted headers.

11 years agoFix wrong check inside Format::Format::assemble
Christos Tsantilas [Tue, 11 Jun 2013 09:25:07 +0000 (12:25 +0300)] 
Fix wrong check inside Format::Format::assemble

We are checking for al->request but we are using al->adapted_request. This is
may cause crashes if the al->adapted_request is NULL.

11 years agoMinor fix for cert validator helper messages parsing
Christos Tsantilas [Tue, 11 Jun 2013 09:22:45 +0000 (12:22 +0300)] 
Minor fix for cert validator helper messages parsing

Allow "host=" cert validation message parameter exist anywhere in the message.
Currently supposed to be always the first parameter of the message.

11 years agoSourceFormat Enforcement
Automatic source maintenance [Tue, 11 Jun 2013 08:11:30 +0000 (02:11 -0600)] 
SourceFormat Enforcement

11 years agoSupport forwarding intercepted but not bumped connections to cache_peers.
Alex Rousskov [Mon, 10 Jun 2013 20:46:08 +0000 (14:46 -0600)] 
Support forwarding intercepted but not bumped connections to cache_peers.

When talking to a cache_peer (i.e., sending a CONNECT request before tunneling
the transaction), tunnel code is using a clever hack: Squid does not parse
the CONNECT response from peer but blindly forwards it to the client. This
works great and simplifies code a lot, except when the client connection
was intercepted and, hence, the client did not send a CONNECT request and
is not expecting a CONNECT response.

In those situations, we now accumulate, parse, and strip the peer CONNECT
response (or close connection on errors).

The existing tunnel I/O code is too simple to accommodate that task -- it
cannot accumulate read data (its I/O buffers work in lockstep fashion, writing
everything it reads before reading again). Instead of rewriting the entire
tunnel code to use more complex buffers, I added a temporary accumulation
buffer for the CONNECT response. That buffer is not allocated unless it is
needed and does not grow beyond SQUID_TCP_SO_RCVBUF size, just like the
simple buffers.

11 years agoBug 3722: Invalid markup in Armenian hy ERR_ONLY_IF_CACHED_MISS
Richard Wall [Mon, 10 Jun 2013 11:04:53 +0000 (23:04 +1200)] 
Bug 3722: Invalid markup in Armenian hy ERR_ONLY_IF_CACHED_MISS

11 years agoInstruct clang not to treat unused command line arguments as errors
Francesco Chemolli [Sun, 9 Jun 2013 20:35:58 +0000 (22:35 +0200)] 
Instruct clang not to treat unused command line arguments as errors

11 years agoRelease Notes: fix typo
Amos Jeffries [Sun, 9 Jun 2013 12:36:06 +0000 (06:36 -0600)] 
Release Notes: fix typo

11 years agoFix detection of concurrent ACLChecklist checks, avoiding !accessList asserts.
Alex Rousskov [Sat, 8 Jun 2013 23:21:23 +0000 (17:21 -0600)] 
Fix detection of concurrent ACLChecklist checks, avoiding !accessList asserts.

Concurrent checks are not supported, but it is possible for the same
ACLChecklist to be used for a sequence of checks, alternating fastCheck(void)
and fastCheck(list) calls. We needed a different/dedicated mechanism to detect
check concurrency (added ACLChecklist::occupied_), and we needed to preserve
(and then restore) pre-set accessList during fastCheck(list) checks.

11 years agoKill some old unused & unmaintained code for sending an error response
Henrik Nordstrom [Fri, 7 Jun 2013 08:49:36 +0000 (20:49 +1200)] 
Kill some old unused & unmaintained code for sending an error response

... on timeout before seeing a request

Ported from 2.7 rev.11516

11 years agoSourceLayout: shuffle forward.h/cc to FwdState.h/cc
Amos Jeffries [Fri, 7 Jun 2013 04:35:25 +0000 (22:35 -0600)] 
SourceLayout: shuffle forward.h/cc to FwdState.h/cc

Our convenience libraries are using the filename forward.h for forward
declarations of symbols. This clashes with the old deprecated naming
of src/forward.h at times.

Rename the src/forward.* files to FwdState.* inline with current coding
guidelines and add a source maintenance check to avoid this problem in
future.

11 years agoFix configure with --disable-internal-dns compile error
Zhanpeng Chen [Fri, 7 Jun 2013 02:32:36 +0000 (20:32 -0600)] 
Fix configure with --disable-internal-dns compile error

dns.cc: In function variable_list* snmp_netDnsFn(variable_list*,s=nint*):
dns.cc:150:5: error: snmpDebugOid was not declared in this scope

11 years agoSourceFormat Enforcement
Automatic source maintenance [Fri, 7 Jun 2013 00:18:11 +0000 (18:18 -0600)] 
SourceFormat Enforcement

11 years agoDo not log bogus ERRORs when url_rewrite_access bypasses url_rewriter.
Alex Rousskov [Thu, 6 Jun 2013 16:43:29 +0000 (10:43 -0600)] 
Do not log bogus ERRORs when url_rewrite_access bypasses url_rewriter.

The code uses HelperReply() object as a fake reply when url_rewrite_access
ACLs did not match. That fake reply had Unknown result code, which made Squid
log ERRORs to cache.log. We now use Error result code, just like
store_id_access does.

TODO: Bypass the overheads of creating and processing a fake reply by moving
all post-processing actions and checks into a new dedicated method and calling
that method when url_rewrite_access does not match. Do the same to the StoreID
code.

11 years agobasic_sasl_auth: Fix helper auto-detection
Amos Jeffries [Thu, 6 Jun 2013 15:39:53 +0000 (03:39 +1200)] 
basic_sasl_auth: Fix helper auto-detection

The helpers update in rev.12782 revealed SASL detection errors in the
use of ./configure script variables. This fixes the SASL library checks
and updates them to use the configure variable naming scheme.

11 years agoPolished icap_service and ecap_service documentation.
Alex Rousskov [Thu, 6 Jun 2013 14:21:25 +0000 (08:21 -0600)] 
Polished icap_service and ecap_service documentation.

11 years agoTying validation errors to certificates
Christos Tsantilas [Thu, 6 Jun 2013 13:53:16 +0000 (16:53 +0300)] 
Tying validation errors to certificates

When Squid sends errors to the certificate validation daemon, the daemon cannot
tell which certificate caused which error. This is especially bad because the
validator has to return that same information in the response (the response
format requires the validator to match the error to the certificate).
This patch adjust the validation request format to provide that information
using a set of the following key=value pairs:

    error_name_N=the name of the certificate error number N
    error_cert_N=the ID of the certificate which caused error_name_N

where N is non-negative integer. N values start from zero and increase
sequentially.

This is a Measurement Factory project

11 years agoRedo r12887 (bug 2066 fix) which introduced several related bugs:
Alex Rousskov [Wed, 5 Jun 2013 15:38:09 +0000 (09:38 -0600)] 
Redo r12887 (bug 2066 fix) which introduced several related bugs:

  - fatal() if chroot is given
  - no chdir to / after chroot if coredump_dir is given
  - wrong detection of "none" coredump_dir
  - chdir to uninitialized pathbuf if no chroot is given

This emergency fix helps Squid start better but may need more testing/work.

11 years agoFixed r12859 (Boolean ACLs) bugs related to ACL parsing and dumping:
Christos Tsantilas [Wed, 5 Jun 2013 15:02:11 +0000 (09:02 -0600)] 
Fixed r12859 (Boolean ACLs) bugs related to ACL parsing and dumping:

- BoolOps.cc:94: "false" assertion when parsing ssl_bump directive.

- Infinite loop with RAM growth in dump_wordlist when dumping Squid
  configuration for cache manager.

- Segfault on not configured (nil) access lists in dump_acl_access.

11 years agoSourceFormat Enforcement
Automatic source maintenance [Wed, 5 Jun 2013 00:13:22 +0000 (18:13 -0600)] 
SourceFormat Enforcement

11 years agoDrop Ip::Address(Ip::Address *) constructor entirely
Amos Jeffries [Tue, 4 Jun 2013 05:12:39 +0000 (23:12 -0600)] 
Drop Ip::Address(Ip::Address *) constructor entirely

Has been deprecated for a while and appears to no longer be required by
any of the current code. The reference& copy-constructor can easily be
used in its place.

11 years agoFix NULL-dereference added in rev.12779
Amos Jeffries [Tue, 4 Jun 2013 04:21:48 +0000 (22:21 -0600)] 
Fix NULL-dereference added in rev.12779

With the change of helper responses from Notes to NotePairs the errNote
in NTLM ERR/NA responses was altered to a potentially NULL char*, and
allowed to be printed in debugs() level 4.

This updated NTLM and Negotiate halpers to print the helper response as
"Result: ..." in identical fashion.

  Detected by Coverity Scan. Issue 1020655.

11 years agoBug 2066: squid does not do chdir() after chroot()
Amos Jeffries [Tue, 4 Jun 2013 04:09:31 +0000 (22:09 -0600)] 
Bug 2066: squid does not do chdir() after chroot()

The earlier workaround applied only fixed 1 of the 3 places performing
chroot().
 This makes chroot and chdir integral parts of setting up Squids running
directory and alters teh chroot() calls to be mainSetCwnd() calls. Which
fixes several potential problems with core dumps from squid -z or -k
executions ending up in unexpected locations, regardless of whether
chroot() and coredump_dir are configured.

 Detected by Coverity Scan. Issue 740335.

11 years agoFix assert unsigned-vs-0 comparison
Amos Jeffries [Tue, 4 Jun 2013 03:55:55 +0000 (21:55 -0600)] 
Fix assert unsigned-vs-0 comparison

Nodes::size_type is unsigned and cannot be <0.

 Detected by Coverity Scan. Issue 1026516.

11 years agoFix string termination in snmplib
Amos Jeffries [Tue, 4 Jun 2013 02:59:47 +0000 (20:59 -0600)] 
Fix string termination in snmplib

strncpy() does not null-terminate if the source string is larger or equal
to the destination buffer length.

 Detected by Coverity Scan. Issue 1025738.

11 years agoFix typo-omission in rev.12882
Amos Jeffries [Mon, 3 Jun 2013 15:37:00 +0000 (09:37 -0600)] 
Fix typo-omission in rev.12882

11 years agoFix basic_pam_auth helper detection
Amos Jeffries [Mon, 3 Jun 2013 14:38:26 +0000 (08:38 -0600)] 
Fix basic_pam_auth helper detection

The m4_include() for this helpers test script was omitted from rev.12782.
Add that in, and shuffle the PAM-specific configure tests into the helper
required.m4 script.

11 years agoPolish: update Ip::Address to follow Squid coding guidelines
Amos Jeffries [Mon, 3 Jun 2013 14:05:16 +0000 (08:05 -0600)] 
Polish: update Ip::Address to follow Squid coding guidelines

* lower-case initial word for camelCase method names
* _ suffix for private variables.
* upper-case for static methods
* InitAddrInfo() and FreeAddrInfo() are static, do not use as methods

Not all methods are camelCased due to meaning irregularities and there
are other guidelines not being followed which also need to be fixed.

11 years agoPrepare external_acl_type format codes for libformat upgrade
Amos Jeffries [Sun, 2 Jun 2013 15:17:28 +0000 (03:17 +1200)] 
Prepare external_acl_type format codes for libformat upgrade

Add upgrade warnings for the %> and %< header codes which will change
radically in a future version when libformat is integrated.

Also, while we are at it support the other logformat codes which map 1:1
but silently for now and only on parse.

11 years agoFix incorrect external_acl_type codes
Amos Jeffries [Sun, 2 Jun 2013 14:32:18 +0000 (02:32 +1200)] 
Fix incorrect external_acl_type codes

Documentation describes %USER_CA_CERT_* codes for outputing the CA cert
attributes. However the directive parser and internals were all
referencing it as %CA_CERT_*.

This updates the internals to match documentation, and adds an upgrade
notice for any installations using the old token name.

11 years agoSupport multicast DNS
Amos Jeffries [Sun, 2 Jun 2013 11:47:05 +0000 (05:47 -0600)] 
Support multicast DNS

Resolve .local domain names using mDNS one-shot queries ahead of regular
recursive DNS qeuries.

* adds the mDNS multicast group IPs as always-present entries in the
  nameservers list.

* filters each request. ".local" lookups are permitted to both the mDNS
  resolvers and the recursive resolvers, other requests are only
  permitted to the regular recursive resolvers.

11 years agoCleanup: remove several HTTPMSGLOCK/HTTPMSGUNLOCK
Amos Jeffries [Sat, 1 Jun 2013 15:20:56 +0000 (09:20 -0600)] 
Cleanup: remove several HTTPMSGLOCK/HTTPMSGUNLOCK

Convert the TunnelStateData and UrnStateData HttpRequest* to Pointer.

11 years agoRelease Notes: initial details for Store-ID and Annotations
Amos Jeffries [Sat, 1 Jun 2013 13:28:13 +0000 (07:28 -0600)] 
Release Notes: initial details for Store-ID and Annotations

11 years agoBug 3854: pt3: redo pt2 fix and revert rev.12587
Amos Jeffries [Sat, 1 Jun 2013 13:05:38 +0000 (07:05 -0600)] 
Bug 3854: pt3: redo pt2 fix and revert rev.12587

The fix in rev.12857 works for Unix and Linux systems. But BSD based
systems it results in TextException errors via sub-includes from the stub

11 years agoFix typo in rev.12859
Amos Jeffries [Sat, 1 Jun 2013 10:01:13 +0000 (04:01 -0600)] 
Fix typo in rev.12859

cbdataReference() is a macro with a design which means it requires usage
in the form of an assignment. ie a = cbdataReference(B); , or as a
parameter in equivalent to a variable.

Not using it in either of those manners results in syntax errors about a
',' from any strict compiler, and many older versions of GCC.

11 years agoPolish: more debug info from squidclient
Amos Jeffries [Fri, 31 May 2013 09:29:21 +0000 (03:29 -0600)] 
Polish: more debug info from squidclient

11 years agoAdd some missing includes
Amos Jeffries [Fri, 31 May 2013 06:42:09 +0000 (00:42 -0600)] 
Add some missing includes

11 years agoCleanup: remove useless includes
Amos Jeffries [Thu, 30 May 2013 14:58:49 +0000 (08:58 -0600)] 
Cleanup: remove useless includes

11 years agoAdd missing include dlink.h for mem.cc
Amos Jeffries [Thu, 30 May 2013 14:58:11 +0000 (08:58 -0600)] 
Add missing include dlink.h for mem.cc

11 years agoCleanup: remove useless includes
Amos Jeffries [Thu, 30 May 2013 10:42:01 +0000 (04:42 -0600)] 
Cleanup: remove useless includes

11 years agoRemove useless includes
Amos Jeffries [Thu, 30 May 2013 10:14:55 +0000 (04:14 -0600)] 
Remove useless includes

11 years agoRemove useless includes
Amos Jeffries [Thu, 30 May 2013 10:10:29 +0000 (04:10 -0600)] 
Remove useless includes

11 years agoRemove useless includes
Amos Jeffries [Thu, 30 May 2013 09:57:42 +0000 (03:57 -0600)] 
Remove useless includes