<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>131250</bug_id>
          
          <creation_ts>2014-04-04 17:59:42 -0700</creation_ts>
          <short_desc>Need to sanitize Date code to ensure we adhere to ES5 15.9.1.14</short_desc>
          <delta_ts>2014-04-04 18:14:15 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>JavaScriptCore</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>NEW</bug_status>
          <resolution></resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Mark Lam">mark.lam</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>mark.lam</cc>
    
    <cc>mhahnenberg</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>997772</commentid>
    <comment_count>0</comment_count>
    <who name="Mark Lam">mark.lam</who>
    <bug_when>2014-04-04 17:59:42 -0700</bug_when>
    <thetext>So far, we&apos;ve added strategic checks on the time milliseconds value in the Date object.  However, the spec is unclear about whether that limit applies to the UTC time, or Local time, or both.  Currently, we conservatively clip the value for both.

We also don&apos;t have a guarantee that we&apos;ve done adequate limit checks to ensure we don&apos;t have underflow / overflow issues in the Date math code.  It would be great if there&apos;s something we can do to ensure that we don&apos;t silently underflow / overflow in the math thereby resulting in bogus Date strings.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>