Code change by Eric Albert, reviewd by me.
authormjs <mjs@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 17 Aug 2004 23:19:59 +0000 (23:19 +0000)
committermjs <mjs@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 17 Aug 2004 23:19:59 +0000 (23:19 +0000)
<rdar://problem/3571960> washingtonpost.com claims I don't have cookies enabled and won't let me read articles

        * kjs/date_object.cpp:
        (timetUsingCF): Clamp time to LONG_MAX (getting rid of time_t
entirely would be even better, but is not required to fix this bug.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@7280 268f45cc-cd09-0410-ab3c-d52691b4dbfc

JavaScriptCore/ChangeLog
JavaScriptCore/kjs/date_object.cpp

index f20bc9c..bc7a8d0 100644 (file)
@@ -1,3 +1,13 @@
+2004-08-16  Maciej Stachowiak  <mjs@apple.com>
+
+        Code change by Eric Albert, reviewd by me.
+
+       <rdar://problem/3571960> washingtonpost.com claims I don't have cookies enabled and won't let me read articles
+
+        * kjs/date_object.cpp:
+        (timetUsingCF): Clamp time to LONG_MAX (getting rid of time_t
+       entirely would be even better, but is not required to fix this bug.
+
 === Safari-157 ===
 
 2004-08-16  Richard Williamson   <rjw@apple.com>
index e81796e..fe1201b 100644 (file)
@@ -175,8 +175,12 @@ static time_t timetUsingCF(struct tm *tm, CFTimeZoneRef timeZone)
     }
 
     CFAbsoluteTime absoluteTime = CFGregorianDateGetAbsoluteTime(date, timeZone);
+       CFTimeInterval interval = absoluteTime + kCFAbsoluteTimeIntervalSince1970;
+       if (interval > LONG_MAX) {
+               interval = LONG_MAX;
+       }
 
-    return (time_t)(absoluteTime + kCFAbsoluteTimeIntervalSince1970);
+    return (time_t) interval;
 }
 
 static time_t mktimeUsingCF(struct tm *tm)