From ca4bb32d449fe18bd9ab8f4c8e779585d6fdfd7a Mon Sep 17 00:00:00 2001 From: "dveditz@cruzio.com" Date: Mon, 7 May 2007 04:14:01 -0700 Subject: [PATCH] bug 373228 incorporate additional review comments after initial checkin, r=dwitte, a=release-drivers --- netwerk/cookie/src/nsCookieService.cpp | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/netwerk/cookie/src/nsCookieService.cpp b/netwerk/cookie/src/nsCookieService.cpp index b1067985aabe..f63130ed9b6b 100644 --- a/netwerk/cookie/src/nsCookieService.cpp +++ b/netwerk/cookie/src/nsCookieService.cpp @@ -2028,12 +2028,8 @@ nsCookieService::CheckPath(nsCookieAttributes &aCookieAttributes, } } - } else { - if (aCookieAttributes.path.Length() > kMaxBytesPerPath || - aCookieAttributes.path.FindChar('\t') != kNotFound ) - return PR_FALSE; - #if 0 + } else { /** * The following test is part of the RFC2109 spec. Loosely speaking, it says that a site * cannot set a cookie for a path that it is not on. See bug 155083. However this patch @@ -2049,6 +2045,10 @@ nsCookieService::CheckPath(nsCookieAttributes &aCookieAttributes, #endif } + if (aCookieAttributes.path.Length() > kMaxBytesPerPath || + aCookieAttributes.path.FindChar('\t') != kNotFound ) + return PR_FALSE; + return PR_TRUE; }