Thanks for getting a fix in so quickly. I have tested the change that Amos made when closing the bug (http://www.squid-cache.org/Versions/v3/3.HEAD/changesets/squid-3-12559.patch) and confirmed that it works in my environment. Based on my past experience with OpenBSD and squid, I am inclined to think the problem exists there as well, but I'm not in a position to test that at the moment. -Nick On Tue, Jan 1, 2013 at 4:15 PM, Amos Jeffries <squid3@xxxxxxxxxxxxx> wrote: > On 2/01/2013 4:24 a.m., Andrew Beverley wrote: >> >> On Tue, 2013-01-01 at 15:15 +0000, Andrew Beverley wrote: >>> >>> Therefore, could you please try the following (untested) patch? >> >> Ah, Amos, I've just re-read the bug report and seen that you have closed >> it with a FreeBSD workaround. I couldn't see your commit, but from what >> I have read it looks like FreeBSD definitely expects a TOS value as an >> int, so that should be the correct solution. > > > I patched the function internally to store the 8-bit tos_t parameter into a > local int before sending to setsockopt() on FreeBSD. That way we still save > on 3 bytes in all the Squid objects using it. > > PS. I would still like to know if the problem occurs on other BSD variants > to extend the patch #if condition for them. > > Amos
