<?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>169034</bug_id>
          
          <creation_ts>2017-03-01 06:02:55 -0800</creation_ts>
          <short_desc>Incorrect RELEASE_ASSERT in JSGlobalObject::addStaticGlobals()</short_desc>
          <delta_ts>2017-03-10 12:09:35 -0800</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>WebKit Nightly Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>104114</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Tomas Popela">tpopela</reporter>
          <assigned_to name="Tomas Popela">tpopela</assigned_to>
          <cc>commit-queue</cc>
    
    <cc>keith_miller</cc>
    
    <cc>mark.lam</cc>
    
    <cc>mcatanzaro</cc>
    
    <cc>msaboff</cc>
    
    <cc>saam</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1282247</commentid>
    <comment_count>0</comment_count>
    <who name="Tomas Popela">tpopela</who>
    <bug_when>2017-03-01 06:02:55 -0800</bug_when>
    <thetext>There is an incorrect RELEASE_ASSERT in JSGlobalObject::addStaticGlobals(GlobalPropertyInfo* globals, int count):

RELEASE_ASSERT(offset = startOffset + i);

it should not assign to offset, but compare to offset.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1282248</commentid>
    <comment_count>1</comment_count>
      <attachid>303064</attachid>
    <who name="Tomas Popela">tpopela</who>
    <bug_when>2017-03-01 06:04:38 -0800</bug_when>
    <thetext>Created attachment 303064
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1282305</commentid>
    <comment_count>2</comment_count>
      <attachid>303064</attachid>
    <who name="Mark Lam">mark.lam</who>
    <bug_when>2017-03-01 09:49:23 -0800</bug_when>
    <thetext>Comment on attachment 303064
Patch

r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1282665</commentid>
    <comment_count>3</comment_count>
      <attachid>303064</attachid>
    <who name="Tomas Popela">tpopela</who>
    <bug_when>2017-03-02 01:42:18 -0800</bug_when>
    <thetext>Comment on attachment 303064
Patch

Clearing flags on attachment: 303064

Committed r213275: &lt;http://trac.webkit.org/changeset/213275&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1282666</commentid>
    <comment_count>4</comment_count>
    <who name="Tomas Popela">tpopela</who>
    <bug_when>2017-03-02 01:42:26 -0800</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1285879</commentid>
    <comment_count>5</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2017-03-10 12:09:35 -0800</bug_when>
    <thetext>&lt;rdar://problem/30978527&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>303064</attachid>
            <date>2017-03-01 06:04:38 -0800</date>
            <delta_ts>2017-03-02 01:42:18 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-169034-20170301150141.patch</filename>
            <type>text/plain</type>
            <size>1675</size>
            <attacher name="Tomas Popela">tpopela</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjEzMjE2CmRpZmYgLS1naXQgYS9Tb3VyY2UvSmF2YVNjcmlw
dENvcmUvQ2hhbmdlTG9nIGIvU291cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwppbmRleCBj
OGNkYWY4NzNjODRiMjdkNTNmYzM2YWRjOTM4OWI3YTY5YjMyYjAyLi5jZWI4NzQ2MDQ3ZTFlNTlk
NDgwNGRhNjY3ZjI2ZmQwY2QxNzZmN2VhIDEwMDY0NAotLS0gYS9Tb3VyY2UvSmF2YVNjcmlwdENv
cmUvQ2hhbmdlTG9nCisrKyBiL1NvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKQEAgLTEs
NSArMSwxNyBAQAogMjAxNy0wMy0wMSAgVG9tYXMgUG9wZWxhICA8dHBvcGVsYUByZWRoYXQuY29t
PgogCisgICAgICAgIEluY29ycmVjdCBSRUxFQVNFX0FTU0VSVCBpbiBKU0dsb2JhbE9iamVjdDo6
YWRkU3RhdGljR2xvYmFscygpCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3df
YnVnLmNnaT9pZD0xNjkwMzQKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4K
KworICAgICAgICBJdCBzaG91bGQgbm90IGFzc2lnbiB0byBvZmZzZXQsIGJ1dCBjb21wYXJlIHRv
IG9mZnNldC4KKworICAgICAgICAqIHJ1bnRpbWUvSlNHbG9iYWxPYmplY3QuY3BwOgorICAgICAg
ICAoSlNDOjpKU0dsb2JhbE9iamVjdDo6YWRkU3RhdGljR2xvYmFscyk6CisKKzIwMTctMDMtMDEg
IFRvbWFzIFBvcGVsYSAgPHRwb3BlbGFAcmVkaGF0LmNvbT4KKwogICAgICAgICBMZWFrIHVuZGVy
IE9wdGlvbnM6OnNldE9wdGlvbnMKICAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hv
d19idWcuY2dpP2lkPTE2OTAyOQogCmRpZmYgLS1naXQgYS9Tb3VyY2UvSmF2YVNjcmlwdENvcmUv
cnVudGltZS9KU0dsb2JhbE9iamVjdC5jcHAgYi9Tb3VyY2UvSmF2YVNjcmlwdENvcmUvcnVudGlt
ZS9KU0dsb2JhbE9iamVjdC5jcHAKaW5kZXggNzZjOTY1Y2FlMWRjMzJjYTg4YjdmZWNmNzc2NTMw
ZjQ2OWM1NzVkNC4uNWI4YjhkMjM0YTY0NzNiNGY0NGM1ZDI4ZGUxMmJlODk3N2Q2YzZmNCAxMDA2
NDQKLS0tIGEvU291cmNlL0phdmFTY3JpcHRDb3JlL3J1bnRpbWUvSlNHbG9iYWxPYmplY3QuY3Bw
CisrKyBiL1NvdXJjZS9KYXZhU2NyaXB0Q29yZS9ydW50aW1lL0pTR2xvYmFsT2JqZWN0LmNwcApA
QCAtMTMxNiw3ICsxMzE2LDcgQEAgdm9pZCBKU0dsb2JhbE9iamVjdDo6YWRkU3RhdGljR2xvYmFs
cyhHbG9iYWxQcm9wZXJ0eUluZm8qIGdsb2JhbHMsIGludCBjb3VudCkKICAgICAgICAgewogICAg
ICAgICAgICAgQ29uY3VycmVudEpTTG9ja2VyIGxvY2tlcihzeW1ib2xUYWJsZSgpLT5tX2xvY2sp
OwogICAgICAgICAgICAgU2NvcGVPZmZzZXQgb2Zmc2V0ID0gc3ltYm9sVGFibGUoKS0+dGFrZU5l
eHRTY29wZU9mZnNldChsb2NrZXIpOwotICAgICAgICAgICAgUkVMRUFTRV9BU1NFUlQob2Zmc2V0
ID0gc3RhcnRPZmZzZXQgKyBpKTsKKyAgICAgICAgICAgIFJFTEVBU0VfQVNTRVJUKG9mZnNldCA9
PSBzdGFydE9mZnNldCArIGkpOwogICAgICAgICAgICAgU3ltYm9sVGFibGVFbnRyeSBuZXdFbnRy
eShWYXJPZmZzZXQob2Zmc2V0KSwgZ2xvYmFsLmF0dHJpYnV0ZXMpOwogICAgICAgICAgICAgbmV3
RW50cnkucHJlcGFyZVRvV2F0Y2goKTsKICAgICAgICAgICAgIHdhdGNocG9pbnRTZXQgPSBuZXdF
bnRyeS53YXRjaHBvaW50U2V0KCk7Cg==
</data>

          </attachment>
      

    </bug>

</bugzilla>