<?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>33371</bug_id>
          
          <creation_ts>2010-01-08 03:02:20 -0800</creation_ts>
          <short_desc>Infinite recursion in RuntimeObjectImp::getOwnPropertyNames()</short_desc>
          <delta_ts>2010-01-14 12:13:42 -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>WebCore JavaScript</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>PC</rep_platform>
          <op_sys>OS X 10.5</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</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="Kent Hansen">kent.hansen</reporter>
          <assigned_to name="Kent Hansen">kent.hansen</assigned_to>
          <cc>commit-queue</cc>
    
    <cc>darin</cc>
    
    <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>178590</commentid>
    <comment_count>0</comment_count>
    <who name="Kent Hansen">kent.hansen</who>
    <bug_when>2010-01-08 03:02:20 -0800</bug_when>
    <thetext>This causes a crash if Object.keys is called on a plugin object, for example.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>178596</commentid>
    <comment_count>1</comment_count>
      <attachid>46126</attachid>
    <who name="Kent Hansen">kent.hansen</who>
    <bug_when>2010-01-08 03:12:13 -0800</bug_when>
    <thetext>Created attachment 46126
Proposed patch

Patch needs to be adapted once the patch for https://bugs.webkit.org/show_bug.cgi?id=32242 is landed.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>178598</commentid>
    <comment_count>2</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2010-01-08 03:15:46 -0800</bug_when>
    <thetext>style-queue ran check-webkit-style on attachment 46126 without any errors.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>178704</commentid>
    <comment_count>3</comment_count>
      <attachid>46126</attachid>
    <who name="Geoffrey Garen">ggaren</who>
    <bug_when>2010-01-08 10:11:24 -0800</bug_when>
    <thetext>Comment on attachment 46126
Proposed patch

r=me, but this can&apos;t land until the other patch does.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>180134</commentid>
    <comment_count>4</comment_count>
      <attachid>46439</attachid>
    <who name="Kent Hansen">kent.hansen</who>
    <bug_when>2010-01-13 02:18:04 -0800</bug_when>
    <thetext>Created attachment 46439
Revised patch

Yay, that other monster patch finally landed. :)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>180144</commentid>
    <comment_count>5</comment_count>
    <who name="Kent Hansen">kent.hansen</who>
    <bug_when>2010-01-13 03:30:01 -0800</bug_when>
    <thetext>(In reply to comment #4)
&gt; Created an attachment (id=46439) [details]
&gt; Revised patch

I didn&apos;t remove the unused mode parameter name. Style bot didn&apos;t complain (and check-webkit-style doesn&apos;t if I remove the name, either). Is it an unwritten style rule that the name should/should not be present if the argument is unused, or is it a dontcare?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>180157</commentid>
    <comment_count>6</comment_count>
      <attachid>46445</attachid>
    <who name="Kent Hansen">kent.hansen</who>
    <bug_when>2010-01-13 04:38:06 -0800</bug_when>
    <thetext>Created attachment 46445
Revised patch (remove unused parameter name)

Ouch, with the unused parameter name, Safari didn&apos;t build for me (warnings treated as errors).
Whereas on the Qt port, unused parameter warnings are just disabled. That&apos;s a bit odd.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>180255</commentid>
    <comment_count>7</comment_count>
      <attachid>46445</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2010-01-13 09:26:18 -0800</bug_when>
    <thetext>Comment on attachment 46445
Revised patch (remove unused parameter name)

Since this patch includes a new regression test, it also needs to include the expected results for that test. The patch otherwise looks great!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>180594</commentid>
    <comment_count>8</comment_count>
    <who name="Kent Hansen">kent.hansen</who>
    <bug_when>2010-01-14 01:16:02 -0800</bug_when>
    <thetext>(In reply to comment #7)
&gt; (From update of attachment 46445 [details])
&gt; Since this patch includes a new regression test, it also needs to include the
&gt; expected results for that test. The patch otherwise looks great!

The test will continue to say SUCCESS if all of it succeeds (including the new part), like before. But it now has a new potential FAILURE output.
So the expected results don&apos;t need to be changed, unless you want me to change the test to have separate PASS output for each &quot;part&quot; of the test.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>180610</commentid>
    <comment_count>9</comment_count>
      <attachid>46445</attachid>
    <who name="Kent Hansen">kent.hansen</who>
    <bug_when>2010-01-14 01:48:31 -0800</bug_when>
    <thetext>Comment on attachment 46445
Revised patch (remove unused parameter name)

Confused about Darin&apos;s comment (see my response in comment #8), setting review? again in hope of being advised before I do any more changes to the test.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>180797</commentid>
    <comment_count>10</comment_count>
    <who name="Darin Adler">darin</who>
    <bug_when>2010-01-14 11:01:24 -0800</bug_when>
    <thetext>(In reply to comment #8)
&gt; The test will continue to say SUCCESS if all of it succeeds (including the new
&gt; part), like before. But it now has a new potential FAILURE output.
&gt; So the expected results don&apos;t need to be changed, unless you want me to change
&gt; the test to have separate PASS output for each &quot;part&quot; of the test.

OK. My mistake.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>180845</commentid>
    <comment_count>11</comment_count>
      <attachid>46445</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-01-14 12:13:36 -0800</bug_when>
    <thetext>Comment on attachment 46445
Revised patch (remove unused parameter name)

Clearing flags on attachment: 46445

Committed r53285: &lt;http://trac.webkit.org/changeset/53285&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>180846</commentid>
    <comment_count>12</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-01-14 12:13:42 -0800</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>46126</attachid>
            <date>2010-01-08 03:12:13 -0800</date>
            <delta_ts>2010-01-13 02:18:04 -0800</delta_ts>
            <desc>Proposed patch</desc>
            <filename>getOwnPropertyNamesInfiniteRecursion.diff</filename>
            <type>text/plain</type>
            <size>3444</size>
            <attacher name="Kent Hansen">kent.hansen</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL0NoYW5nZUxvZyBiL0xheW91dFRlc3RzL0NoYW5nZUxv
ZwppbmRleCBhZTNmYzgxLi4zM2NiMTc1IDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0cy9DaGFuZ2VM
b2cKKysrIGIvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTQgQEAKKzIwMTAtMDEt
MDggIEtlbnQgSGFuc2VuICA8a2VudC5oYW5zZW5Abm9raWEuY29tPgorCisgICAgICAgIFJldmll
d2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIEluZmluaXRlIHJlY3Vyc2lvbiBpbiBS
dW50aW1lT2JqZWN0SW1wOjpnZXRPd25Qcm9wZXJ0eU5hbWVzKCkKKyAgICAgICAgaHR0cHM6Ly9i
dWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTMzMzcxCisKKyAgICAgICAgVGVzdCB0aGF0
IE9iamVjdC5rZXlzIHdvcmtzIG9uIGEgcGx1Z2luIG9iamVjdC4KKworICAgICAgICAqIHBsdWdp
bnMvbmV0c2NhcGUtZW51bWVyYXRlLmh0bWw6CisKIDIwMTAtMDEtMDcgIEFmb25zbyBSLiBDb3N0
YSBKci4gIDxhZm9uc28uY29zdGFAb3BlbmJvc3NhLm9yZz4KIAogICAgICAgICBSZXZpZXdlZCBi
eSBEYXJpbiBBZGxlci4KZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL3BsdWdpbnMvbmV0c2NhcGUt
ZW51bWVyYXRlLmh0bWwgYi9MYXlvdXRUZXN0cy9wbHVnaW5zL25ldHNjYXBlLWVudW1lcmF0ZS5o
dG1sCmluZGV4IDhkM2EwMTkuLjU5Yzg5MjIgMTAwNjQ0Ci0tLSBhL0xheW91dFRlc3RzL3BsdWdp
bnMvbmV0c2NhcGUtZW51bWVyYXRlLmh0bWwKKysrIGIvTGF5b3V0VGVzdHMvcGx1Z2lucy9uZXRz
Y2FwZS1lbnVtZXJhdGUuaHRtbApAQCAtMjksNiArMjksMTIgQEAgZnVuY3Rpb24gcnVuVGVzdCgp
CiAgICAgICAgIHJldHVybjsKICAgICB9CiAKKyAgICAvLyBOb3cgdHJ5IGVudW1lcmF0aW5nIGEg
cGx1Z2luIG9iamVjdCdzIHByb3BlcnRpZXMgdXNpbmcgT2JqZWN0LmtleXMKKyAgICBpZiAoT2Jq
ZWN0LmtleXMocGx1Z2luLnRlc3RPYmplY3QpLnRvU3RyaW5nKCkgIT0gJ2ZvbyxiYXInKSB7Cisg
ICAgICAgIHJlc3VsdC5pbm5lckhUTUwgPSAnRkFJTFVSRTogUGx1Z2luIG9iamVjdCBwcm9wZXJ0
aWVzIHJldHVybmVkIGJ5IE9iamVjdC5rZXlzIHdlcmUgJyArIHByb3BOYW1lcy50b1N0cmluZygp
OworICAgICAgICByZXR1cm47CisgICAgfQorCiAgICAgcmVzdWx0LmlubmVySFRNTCA9ICdTVUND
RVNTJzsKIH0KIDwvc2NyaXB0PgpkaWZmIC0tZ2l0IGEvV2ViQ29yZS9DaGFuZ2VMb2cgYi9XZWJD
b3JlL0NoYW5nZUxvZwppbmRleCA2ODI3NDA4Li4yNmUwOTkwIDEwMDY0NAotLS0gYS9XZWJDb3Jl
L0NoYW5nZUxvZworKysgYi9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE3IEBACisyMDEw
LTAxLTA4ICBLZW50IEhhbnNlbiAgPGtlbnQuaGFuc2VuQG5va2lhLmNvbT4KKworICAgICAgICBS
ZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBJbmZpbml0ZSByZWN1cnNpb24g
aW4gUnVudGltZU9iamVjdEltcDo6Z2V0T3duUHJvcGVydHlOYW1lcygpCisgICAgICAgIGh0dHBz
Oi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0zMzM3MQorCisgICAgICAgIFJ1bnRp
bWVPYmplY3RJbXAgc2hvdWxkIG5vdCByZWltcGxlbWVudCBnZXRQcm9wZXJ0eU5hbWVzKCk7Cisg
ICAgICAgIG1vdmUgdGhlIGltcGxlbWVudGF0aW9uIHRvIGdldE93blByb3BlcnR5TmFtZXMoKS4K
KworICAgICAgICAqIGJyaWRnZS9ydW50aW1lX29iamVjdC5jcHA6CisgICAgICAgIChKU0M6OlJ1
bnRpbWVPYmplY3RJbXA6OmdldE93blByb3BlcnR5TmFtZXMpOgorICAgICAgICAqIGJyaWRnZS9y
dW50aW1lX29iamVjdC5oOgorCiAyMDEwLTAxLTA3ICBNaWtlIEJlbHNoZSAgPG1iZWxzaGVAY2hy
b21pdW0ub3JnPgogCiAgICAgICAgIFJldmlld2VkIGJ5IERhcmluIEFkbGVyLgpkaWZmIC0tZ2l0
IGEvV2ViQ29yZS9icmlkZ2UvcnVudGltZV9vYmplY3QuY3BwIGIvV2ViQ29yZS9icmlkZ2UvcnVu
dGltZV9vYmplY3QuY3BwCmluZGV4IDk1ODNmYjIuLjA3Yjc1YTcgMTAwNjQ0Ci0tLSBhL1dlYkNv
cmUvYnJpZGdlL3J1bnRpbWVfb2JqZWN0LmNwcAorKysgYi9XZWJDb3JlL2JyaWRnZS9ydW50aW1l
X29iamVjdC5jcHAKQEAgLTMwMiw3ICszMDIsNyBAQCBDb25zdHJ1Y3RUeXBlIFJ1bnRpbWVPYmpl
Y3RJbXA6OmdldENvbnN0cnVjdERhdGEoQ29uc3RydWN0RGF0YSYgY29uc3RydWN0RGF0YSkKICAg
ICByZXR1cm4gQ29uc3RydWN0VHlwZUhvc3Q7CiB9CiAKLXZvaWQgUnVudGltZU9iamVjdEltcDo6
Z2V0UHJvcGVydHlOYW1lcyhFeGVjU3RhdGUqIGV4ZWMsIFByb3BlcnR5TmFtZUFycmF5JiBwcm9w
ZXJ0eU5hbWVzKQordm9pZCBSdW50aW1lT2JqZWN0SW1wOjpnZXRPd25Qcm9wZXJ0eU5hbWVzKEV4
ZWNTdGF0ZSogZXhlYywgUHJvcGVydHlOYW1lQXJyYXkmIHByb3BlcnR5TmFtZXMpCiB7CiAgICAg
aWYgKCFtX2luc3RhbmNlKSB7CiAgICAgICAgIHRocm93SW52YWxpZEFjY2Vzc0Vycm9yKGV4ZWMp
OwpAQCAtMzE2LDExICszMTYsNiBAQCB2b2lkIFJ1bnRpbWVPYmplY3RJbXA6OmdldFByb3BlcnR5
TmFtZXMoRXhlY1N0YXRlKiBleGVjLCBQcm9wZXJ0eU5hbWVBcnJheSYgcHJvcAogICAgIGluc3Rh
bmNlLT5lbmQoKTsKIH0KIAotdm9pZCBSdW50aW1lT2JqZWN0SW1wOjpnZXRPd25Qcm9wZXJ0eU5h
bWVzKEV4ZWNTdGF0ZSogZXhlYywgUHJvcGVydHlOYW1lQXJyYXkmIHByb3BlcnR5TmFtZXMpCi17
Ci0gICAgZ2V0T3duUHJvcGVydHlOYW1lcyhleGVjLCBwcm9wZXJ0eU5hbWVzKTsKLX0KLQogSlNP
YmplY3QqIFJ1bnRpbWVPYmplY3RJbXA6OnRocm93SW52YWxpZEFjY2Vzc0Vycm9yKEV4ZWNTdGF0
ZSogZXhlYykKIHsKICAgICByZXR1cm4gdGhyb3dFcnJvcihleGVjLCBSZWZlcmVuY2VFcnJvciwg
IlRyeWluZyB0byBhY2Nlc3Mgb2JqZWN0IGZyb20gZGVzdHJveWVkIHBsdWctaW4uIik7CmRpZmYg
LS1naXQgYS9XZWJDb3JlL2JyaWRnZS9ydW50aW1lX29iamVjdC5oIGIvV2ViQ29yZS9icmlkZ2Uv
cnVudGltZV9vYmplY3QuaAppbmRleCAzOTFlMDc4Li5jODU1ZWI1IDEwMDY0NAotLS0gYS9XZWJD
b3JlL2JyaWRnZS9ydW50aW1lX29iamVjdC5oCisrKyBiL1dlYkNvcmUvYnJpZGdlL3J1bnRpbWVf
b2JqZWN0LmgKQEAgLTQ0LDcgKzQ0LDYgQEAgcHVibGljOgogICAgIHZpcnR1YWwgQ2FsbFR5cGUg
Z2V0Q2FsbERhdGEoQ2FsbERhdGEmKTsKICAgICB2aXJ0dWFsIENvbnN0cnVjdFR5cGUgZ2V0Q29u
c3RydWN0RGF0YShDb25zdHJ1Y3REYXRhJik7CiAKLSAgICB2aXJ0dWFsIHZvaWQgZ2V0UHJvcGVy
dHlOYW1lcyhFeGVjU3RhdGUqLCBQcm9wZXJ0eU5hbWVBcnJheSYpOwogICAgIHZpcnR1YWwgdm9p
ZCBnZXRPd25Qcm9wZXJ0eU5hbWVzKEV4ZWNTdGF0ZSosIFByb3BlcnR5TmFtZUFycmF5Jik7CiAK
ICAgICB2b2lkIGludmFsaWRhdGUoKTsK
</data>
<flag name="review"
          id="28346"
          type_id="1"
          status="+"
          setter="ggaren"
    />
    <flag name="commit-queue"
          id="28347"
          type_id="3"
          status="-"
          setter="kent.hansen"
    />
          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>46439</attachid>
            <date>2010-01-13 02:18:04 -0800</date>
            <delta_ts>2010-01-13 04:38:06 -0800</delta_ts>
            <desc>Revised patch</desc>
            <filename>getOwnPropertyNamesInfiniteRecursion2.diff</filename>
            <type>text/plain</type>
            <size>3615</size>
            <attacher name="Kent Hansen">kent.hansen</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL0NoYW5nZUxvZyBiL0xheW91dFRlc3RzL0NoYW5nZUxv
ZwppbmRleCBiN2Y4MjY2Li5mZTRjM2Q1IDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0cy9DaGFuZ2VM
b2cKKysrIGIvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTQgQEAKKzIwMTAtMDEt
MTMgIEtlbnQgSGFuc2VuICA8a2VudC5oYW5zZW5Abm9raWEuY29tPgorCisgICAgICAgIFJldmll
d2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIEluZmluaXRlIHJlY3Vyc2lvbiBpbiBS
dW50aW1lT2JqZWN0SW1wOjpnZXRPd25Qcm9wZXJ0eU5hbWVzKCkKKyAgICAgICAgaHR0cHM6Ly9i
dWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTMzMzcxCisKKyAgICAgICAgVGVzdCB0aGF0
IE9iamVjdC5rZXlzIHdvcmtzIG9uIGEgcGx1Z2luIG9iamVjdC4KKworICAgICAgICAqIHBsdWdp
bnMvbmV0c2NhcGUtZW51bWVyYXRlLmh0bWw6CisKIDIwMTAtMDEtMTIgIEZ1bWl0b3NoaSBVa2Fp
ICA8dWthaUBjaHJvbWl1bS5vcmc+CiAKICAgICAgICAgUmV2aWV3ZWQgYnkgRGFyaW4gQWRsZXIu
CmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9wbHVnaW5zL25ldHNjYXBlLWVudW1lcmF0ZS5odG1s
IGIvTGF5b3V0VGVzdHMvcGx1Z2lucy9uZXRzY2FwZS1lbnVtZXJhdGUuaHRtbAppbmRleCA4ZDNh
MDE5Li5hZTU1YmNmIDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0cy9wbHVnaW5zL25ldHNjYXBlLWVu
dW1lcmF0ZS5odG1sCisrKyBiL0xheW91dFRlc3RzL3BsdWdpbnMvbmV0c2NhcGUtZW51bWVyYXRl
Lmh0bWwKQEAgLTI5LDYgKzI5LDEzIEBAIGZ1bmN0aW9uIHJ1blRlc3QoKQogICAgICAgICByZXR1
cm47CiAgICAgfQogCisgICAgLy8gTm93IHRyeSBlbnVtZXJhdGluZyBhIHBsdWdpbiBvYmplY3Qn
cyBwcm9wZXJ0aWVzIHVzaW5nIE9iamVjdC5rZXlzCisgICAgdmFyIGtleXMgPSBPYmplY3Qua2V5
cyhwbHVnaW4udGVzdE9iamVjdCk7CisgICAgaWYgKGtleXMudG9TdHJpbmcoKSAhPSAnZm9vLGJh
cicpIHsKKyAgICAgICAgcmVzdWx0LmlubmVySFRNTCA9ICdGQUlMVVJFOiBQbHVnaW4gb2JqZWN0
IHByb3BlcnRpZXMgcmV0dXJuZWQgYnkgT2JqZWN0LmtleXMgd2VyZSAnICsga2V5cy50b1N0cmlu
ZygpOworICAgICAgICByZXR1cm47CisgICAgfQorCiAgICAgcmVzdWx0LmlubmVySFRNTCA9ICdT
VUNDRVNTJzsKIH0KIDwvc2NyaXB0PgpkaWZmIC0tZ2l0IGEvV2ViQ29yZS9DaGFuZ2VMb2cgYi9X
ZWJDb3JlL0NoYW5nZUxvZwppbmRleCA3YTdlMjZmLi4wMjIwN2Y1IDEwMDY0NAotLS0gYS9XZWJD
b3JlL0NoYW5nZUxvZworKysgYi9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE3IEBACisy
MDEwLTAxLTEzICBLZW50IEhhbnNlbiAgPGtlbnQuaGFuc2VuQG5va2lhLmNvbT4KKworICAgICAg
ICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBJbmZpbml0ZSByZWN1cnNp
b24gaW4gUnVudGltZU9iamVjdEltcDo6Z2V0T3duUHJvcGVydHlOYW1lcygpCisgICAgICAgIGh0
dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0zMzM3MQorCisgICAgICAgIFJ1
bnRpbWVPYmplY3RJbXAgc2hvdWxkIG5vdCByZWltcGxlbWVudCBnZXRQcm9wZXJ0eU5hbWVzKCk7
CisgICAgICAgIG1vdmUgdGhlIGltcGxlbWVudGF0aW9uIHRvIGdldE93blByb3BlcnR5TmFtZXMo
KS4KKworICAgICAgICAqIGJyaWRnZS9ydW50aW1lX29iamVjdC5jcHA6CisgICAgICAgIChKU0M6
OlJ1bnRpbWVPYmplY3RJbXA6OmdldE93blByb3BlcnR5TmFtZXMpOgorICAgICAgICAqIGJyaWRn
ZS9ydW50aW1lX29iamVjdC5oOgorCiAyMDEwLTAxLTExICBKZXJlbXkgT3Jsb3cgIDxqb3Jsb3dA
Y2hyb21pdW0ub3JnPgogCiAgICAgICAgIFJldmlld2VkIGJ5IERhcmluIEZpc2hlci4KZGlmZiAt
LWdpdCBhL1dlYkNvcmUvYnJpZGdlL3J1bnRpbWVfb2JqZWN0LmNwcCBiL1dlYkNvcmUvYnJpZGdl
L3J1bnRpbWVfb2JqZWN0LmNwcAppbmRleCA5ZmM5ZjBjLi5hMWExMmUyIDEwMDY0NAotLS0gYS9X
ZWJDb3JlL2JyaWRnZS9ydW50aW1lX29iamVjdC5jcHAKKysrIGIvV2ViQ29yZS9icmlkZ2UvcnVu
dGltZV9vYmplY3QuY3BwCkBAIC0zMDIsNyArMzAyLDcgQEAgQ29uc3RydWN0VHlwZSBSdW50aW1l
T2JqZWN0SW1wOjpnZXRDb25zdHJ1Y3REYXRhKENvbnN0cnVjdERhdGEmIGNvbnN0cnVjdERhdGEp
CiAgICAgcmV0dXJuIENvbnN0cnVjdFR5cGVIb3N0OwogfQogCi12b2lkIFJ1bnRpbWVPYmplY3RJ
bXA6OmdldFByb3BlcnR5TmFtZXMoRXhlY1N0YXRlKiBleGVjLCBQcm9wZXJ0eU5hbWVBcnJheSYg
cHJvcGVydHlOYW1lcywgRW51bWVyYXRpb25Nb2RlKQordm9pZCBSdW50aW1lT2JqZWN0SW1wOjpn
ZXRPd25Qcm9wZXJ0eU5hbWVzKEV4ZWNTdGF0ZSogZXhlYywgUHJvcGVydHlOYW1lQXJyYXkmIHBy
b3BlcnR5TmFtZXMsIEVudW1lcmF0aW9uTW9kZSBtb2RlKQogewogICAgIGlmICghbV9pbnN0YW5j
ZSkgewogICAgICAgICB0aHJvd0ludmFsaWRBY2Nlc3NFcnJvcihleGVjKTsKQEAgLTMxNiwxMSAr
MzE2LDYgQEAgdm9pZCBSdW50aW1lT2JqZWN0SW1wOjpnZXRQcm9wZXJ0eU5hbWVzKEV4ZWNTdGF0
ZSogZXhlYywgUHJvcGVydHlOYW1lQXJyYXkmIHByb3AKICAgICBpbnN0YW5jZS0+ZW5kKCk7CiB9
CiAKLXZvaWQgUnVudGltZU9iamVjdEltcDo6Z2V0T3duUHJvcGVydHlOYW1lcyhFeGVjU3RhdGUq
IGV4ZWMsIFByb3BlcnR5TmFtZUFycmF5JiBwcm9wZXJ0eU5hbWVzLCBFbnVtZXJhdGlvbk1vZGUg
bW9kZSkKLXsKLSAgICBnZXRPd25Qcm9wZXJ0eU5hbWVzKGV4ZWMsIHByb3BlcnR5TmFtZXMsIG1v
ZGUpOwotfQotCiBKU09iamVjdCogUnVudGltZU9iamVjdEltcDo6dGhyb3dJbnZhbGlkQWNjZXNz
RXJyb3IoRXhlY1N0YXRlKiBleGVjKQogewogICAgIHJldHVybiB0aHJvd0Vycm9yKGV4ZWMsIFJl
ZmVyZW5jZUVycm9yLCAiVHJ5aW5nIHRvIGFjY2VzcyBvYmplY3QgZnJvbSBkZXN0cm95ZWQgcGx1
Zy1pbi4iKTsKZGlmZiAtLWdpdCBhL1dlYkNvcmUvYnJpZGdlL3J1bnRpbWVfb2JqZWN0LmggYi9X
ZWJDb3JlL2JyaWRnZS9ydW50aW1lX29iamVjdC5oCmluZGV4IDg3YzQ3YmYuLjAyODRlNjYgMTAw
NjQ0Ci0tLSBhL1dlYkNvcmUvYnJpZGdlL3J1bnRpbWVfb2JqZWN0LmgKKysrIGIvV2ViQ29yZS9i
cmlkZ2UvcnVudGltZV9vYmplY3QuaApAQCAtNDQsNyArNDQsNiBAQCBwdWJsaWM6CiAgICAgdmly
dHVhbCBDYWxsVHlwZSBnZXRDYWxsRGF0YShDYWxsRGF0YSYpOwogICAgIHZpcnR1YWwgQ29uc3Ry
dWN0VHlwZSBnZXRDb25zdHJ1Y3REYXRhKENvbnN0cnVjdERhdGEmKTsKIAotICAgIHZpcnR1YWwg
dm9pZCBnZXRQcm9wZXJ0eU5hbWVzKEV4ZWNTdGF0ZSosIFByb3BlcnR5TmFtZUFycmF5JiwgRW51
bWVyYXRpb25Nb2RlIG1vZGUgPSBFeGNsdWRlRG9udEVudW1Qcm9wZXJ0aWVzKTsKICAgICB2aXJ0
dWFsIHZvaWQgZ2V0T3duUHJvcGVydHlOYW1lcyhFeGVjU3RhdGUqLCBQcm9wZXJ0eU5hbWVBcnJh
eSYsIEVudW1lcmF0aW9uTW9kZSBtb2RlID0gRXhjbHVkZURvbnRFbnVtUHJvcGVydGllcyk7CiAK
ICAgICB2b2lkIGludmFsaWRhdGUoKTsK
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>46445</attachid>
            <date>2010-01-13 04:38:06 -0800</date>
            <delta_ts>2010-01-14 12:13:36 -0800</delta_ts>
            <desc>Revised patch (remove unused parameter name)</desc>
            <filename>getOwnPropertyNamesInfiniteRecursion3.diff</filename>
            <type>text/plain</type>
            <size>3610</size>
            <attacher name="Kent Hansen">kent.hansen</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL0NoYW5nZUxvZyBiL0xheW91dFRlc3RzL0NoYW5nZUxv
ZwppbmRleCBiN2Y4MjY2Li5mZTRjM2Q1IDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0cy9DaGFuZ2VM
b2cKKysrIGIvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTQgQEAKKzIwMTAtMDEt
MTMgIEtlbnQgSGFuc2VuICA8a2VudC5oYW5zZW5Abm9raWEuY29tPgorCisgICAgICAgIFJldmll
d2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIEluZmluaXRlIHJlY3Vyc2lvbiBpbiBS
dW50aW1lT2JqZWN0SW1wOjpnZXRPd25Qcm9wZXJ0eU5hbWVzKCkKKyAgICAgICAgaHR0cHM6Ly9i
dWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTMzMzcxCisKKyAgICAgICAgVGVzdCB0aGF0
IE9iamVjdC5rZXlzIHdvcmtzIG9uIGEgcGx1Z2luIG9iamVjdC4KKworICAgICAgICAqIHBsdWdp
bnMvbmV0c2NhcGUtZW51bWVyYXRlLmh0bWw6CisKIDIwMTAtMDEtMTIgIEZ1bWl0b3NoaSBVa2Fp
ICA8dWthaUBjaHJvbWl1bS5vcmc+CiAKICAgICAgICAgUmV2aWV3ZWQgYnkgRGFyaW4gQWRsZXIu
CmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9wbHVnaW5zL25ldHNjYXBlLWVudW1lcmF0ZS5odG1s
IGIvTGF5b3V0VGVzdHMvcGx1Z2lucy9uZXRzY2FwZS1lbnVtZXJhdGUuaHRtbAppbmRleCA4ZDNh
MDE5Li5hZTU1YmNmIDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0cy9wbHVnaW5zL25ldHNjYXBlLWVu
dW1lcmF0ZS5odG1sCisrKyBiL0xheW91dFRlc3RzL3BsdWdpbnMvbmV0c2NhcGUtZW51bWVyYXRl
Lmh0bWwKQEAgLTI5LDYgKzI5LDEzIEBAIGZ1bmN0aW9uIHJ1blRlc3QoKQogICAgICAgICByZXR1
cm47CiAgICAgfQogCisgICAgLy8gTm93IHRyeSBlbnVtZXJhdGluZyBhIHBsdWdpbiBvYmplY3Qn
cyBwcm9wZXJ0aWVzIHVzaW5nIE9iamVjdC5rZXlzCisgICAgdmFyIGtleXMgPSBPYmplY3Qua2V5
cyhwbHVnaW4udGVzdE9iamVjdCk7CisgICAgaWYgKGtleXMudG9TdHJpbmcoKSAhPSAnZm9vLGJh
cicpIHsKKyAgICAgICAgcmVzdWx0LmlubmVySFRNTCA9ICdGQUlMVVJFOiBQbHVnaW4gb2JqZWN0
IHByb3BlcnRpZXMgcmV0dXJuZWQgYnkgT2JqZWN0LmtleXMgd2VyZSAnICsga2V5cy50b1N0cmlu
ZygpOworICAgICAgICByZXR1cm47CisgICAgfQorCiAgICAgcmVzdWx0LmlubmVySFRNTCA9ICdT
VUNDRVNTJzsKIH0KIDwvc2NyaXB0PgpkaWZmIC0tZ2l0IGEvV2ViQ29yZS9DaGFuZ2VMb2cgYi9X
ZWJDb3JlL0NoYW5nZUxvZwppbmRleCA3YTdlMjZmLi4wMjIwN2Y1IDEwMDY0NAotLS0gYS9XZWJD
b3JlL0NoYW5nZUxvZworKysgYi9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE3IEBACisy
MDEwLTAxLTEzICBLZW50IEhhbnNlbiAgPGtlbnQuaGFuc2VuQG5va2lhLmNvbT4KKworICAgICAg
ICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBJbmZpbml0ZSByZWN1cnNp
b24gaW4gUnVudGltZU9iamVjdEltcDo6Z2V0T3duUHJvcGVydHlOYW1lcygpCisgICAgICAgIGh0
dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0zMzM3MQorCisgICAgICAgIFJ1
bnRpbWVPYmplY3RJbXAgc2hvdWxkIG5vdCByZWltcGxlbWVudCBnZXRQcm9wZXJ0eU5hbWVzKCk7
CisgICAgICAgIG1vdmUgdGhlIGltcGxlbWVudGF0aW9uIHRvIGdldE93blByb3BlcnR5TmFtZXMo
KS4KKworICAgICAgICAqIGJyaWRnZS9ydW50aW1lX29iamVjdC5jcHA6CisgICAgICAgIChKU0M6
OlJ1bnRpbWVPYmplY3RJbXA6OmdldE93blByb3BlcnR5TmFtZXMpOgorICAgICAgICAqIGJyaWRn
ZS9ydW50aW1lX29iamVjdC5oOgorCiAyMDEwLTAxLTExICBKZXJlbXkgT3Jsb3cgIDxqb3Jsb3dA
Y2hyb21pdW0ub3JnPgogCiAgICAgICAgIFJldmlld2VkIGJ5IERhcmluIEZpc2hlci4KZGlmZiAt
LWdpdCBhL1dlYkNvcmUvYnJpZGdlL3J1bnRpbWVfb2JqZWN0LmNwcCBiL1dlYkNvcmUvYnJpZGdl
L3J1bnRpbWVfb2JqZWN0LmNwcAppbmRleCA5ZmM5ZjBjLi4yNmI4NWY5IDEwMDY0NAotLS0gYS9X
ZWJDb3JlL2JyaWRnZS9ydW50aW1lX29iamVjdC5jcHAKKysrIGIvV2ViQ29yZS9icmlkZ2UvcnVu
dGltZV9vYmplY3QuY3BwCkBAIC0zMDIsNyArMzAyLDcgQEAgQ29uc3RydWN0VHlwZSBSdW50aW1l
T2JqZWN0SW1wOjpnZXRDb25zdHJ1Y3REYXRhKENvbnN0cnVjdERhdGEmIGNvbnN0cnVjdERhdGEp
CiAgICAgcmV0dXJuIENvbnN0cnVjdFR5cGVIb3N0OwogfQogCi12b2lkIFJ1bnRpbWVPYmplY3RJ
bXA6OmdldFByb3BlcnR5TmFtZXMoRXhlY1N0YXRlKiBleGVjLCBQcm9wZXJ0eU5hbWVBcnJheSYg
cHJvcGVydHlOYW1lcywgRW51bWVyYXRpb25Nb2RlKQordm9pZCBSdW50aW1lT2JqZWN0SW1wOjpn
ZXRPd25Qcm9wZXJ0eU5hbWVzKEV4ZWNTdGF0ZSogZXhlYywgUHJvcGVydHlOYW1lQXJyYXkmIHBy
b3BlcnR5TmFtZXMsIEVudW1lcmF0aW9uTW9kZSkKIHsKICAgICBpZiAoIW1faW5zdGFuY2UpIHsK
ICAgICAgICAgdGhyb3dJbnZhbGlkQWNjZXNzRXJyb3IoZXhlYyk7CkBAIC0zMTYsMTEgKzMxNiw2
IEBAIHZvaWQgUnVudGltZU9iamVjdEltcDo6Z2V0UHJvcGVydHlOYW1lcyhFeGVjU3RhdGUqIGV4
ZWMsIFByb3BlcnR5TmFtZUFycmF5JiBwcm9wCiAgICAgaW5zdGFuY2UtPmVuZCgpOwogfQogCi12
b2lkIFJ1bnRpbWVPYmplY3RJbXA6OmdldE93blByb3BlcnR5TmFtZXMoRXhlY1N0YXRlKiBleGVj
LCBQcm9wZXJ0eU5hbWVBcnJheSYgcHJvcGVydHlOYW1lcywgRW51bWVyYXRpb25Nb2RlIG1vZGUp
Ci17Ci0gICAgZ2V0T3duUHJvcGVydHlOYW1lcyhleGVjLCBwcm9wZXJ0eU5hbWVzLCBtb2RlKTsK
LX0KLQogSlNPYmplY3QqIFJ1bnRpbWVPYmplY3RJbXA6OnRocm93SW52YWxpZEFjY2Vzc0Vycm9y
KEV4ZWNTdGF0ZSogZXhlYykKIHsKICAgICByZXR1cm4gdGhyb3dFcnJvcihleGVjLCBSZWZlcmVu
Y2VFcnJvciwgIlRyeWluZyB0byBhY2Nlc3Mgb2JqZWN0IGZyb20gZGVzdHJveWVkIHBsdWctaW4u
Iik7CmRpZmYgLS1naXQgYS9XZWJDb3JlL2JyaWRnZS9ydW50aW1lX29iamVjdC5oIGIvV2ViQ29y
ZS9icmlkZ2UvcnVudGltZV9vYmplY3QuaAppbmRleCA4N2M0N2JmLi4wMjg0ZTY2IDEwMDY0NAot
LS0gYS9XZWJDb3JlL2JyaWRnZS9ydW50aW1lX29iamVjdC5oCisrKyBiL1dlYkNvcmUvYnJpZGdl
L3J1bnRpbWVfb2JqZWN0LmgKQEAgLTQ0LDcgKzQ0LDYgQEAgcHVibGljOgogICAgIHZpcnR1YWwg
Q2FsbFR5cGUgZ2V0Q2FsbERhdGEoQ2FsbERhdGEmKTsKICAgICB2aXJ0dWFsIENvbnN0cnVjdFR5
cGUgZ2V0Q29uc3RydWN0RGF0YShDb25zdHJ1Y3REYXRhJik7CiAKLSAgICB2aXJ0dWFsIHZvaWQg
Z2V0UHJvcGVydHlOYW1lcyhFeGVjU3RhdGUqLCBQcm9wZXJ0eU5hbWVBcnJheSYsIEVudW1lcmF0
aW9uTW9kZSBtb2RlID0gRXhjbHVkZURvbnRFbnVtUHJvcGVydGllcyk7CiAgICAgdmlydHVhbCB2
b2lkIGdldE93blByb3BlcnR5TmFtZXMoRXhlY1N0YXRlKiwgUHJvcGVydHlOYW1lQXJyYXkmLCBF
bnVtZXJhdGlvbk1vZGUgbW9kZSA9IEV4Y2x1ZGVEb250RW51bVByb3BlcnRpZXMpOwogCiAgICAg
dm9pZCBpbnZhbGlkYXRlKCk7Cg==
</data>

          </attachment>
      

    </bug>

</bugzilla>