<?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>131449</bug_id>
          
          <creation_ts>2014-04-09 12:34:37 -0700</creation_ts>
          <short_desc>Ensure that LLINT accessing of the ProtoCallFrame is big endian friendly</short_desc>
          <delta_ts>2014-04-09 14:53:45 -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>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>
          
          <blocked>128743</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Mark Lam">mark.lam</reporter>
          <assigned_to name="Mark Lam">mark.lam</assigned_to>
          <cc>fpizlo</cc>
    
    <cc>ggaren</cc>
    
    <cc>mhahnenberg</cc>
    
    <cc>mmirman</cc>
    
    <cc>msaboff</cc>
    
    <cc>oliver</cc>
    
    <cc>tpopela</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>999256</commentid>
    <comment_count>0</comment_count>
    <who name="Mark Lam">mark.lam</who>
    <bug_when>2014-04-09 12:34:37 -0700</bug_when>
    <thetext>Patch coming.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>999257</commentid>
    <comment_count>1</comment_count>
      <attachid>228975</attachid>
    <who name="Mark Lam">mark.lam</who>
    <bug_when>2014-04-09 12:44:05 -0700</bug_when>
    <thetext>Created attachment 228975
the patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>999258</commentid>
    <comment_count>2</comment_count>
      <attachid>228975</attachid>
    <who name="Mark Hahnenberg">mhahnenberg</who>
    <bug_when>2014-04-09 12:46:48 -0700</bug_when>
    <thetext>Comment on attachment 228975
the patch

View in context: https://bugs.webkit.org/attachment.cgi?id=228975&amp;action=review

&gt; Source/JavaScriptCore/llint/LowLevelInterpreter64.asm:194
&gt; +    loadp ProtoCallFrame::paddedArgCount[protoCallFrame], temp3

Doesn&apos;t this change the behavior? We will end up loading an extra 32 bits. Was the old behavior a bug?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>999259</commentid>
    <comment_count>3</comment_count>
    <who name="Mark Lam">mark.lam</who>
    <bug_when>2014-04-09 12:50:27 -0700</bug_when>
    <thetext>(In reply to comment #2)
&gt; (From update of attachment 228975 [details])
&gt; View in context: https://bugs.webkit.org/attachment.cgi?id=228975&amp;action=review
&gt; 
&gt; &gt; Source/JavaScriptCore/llint/LowLevelInterpreter64.asm:194
&gt; &gt; +    loadp ProtoCallFrame::paddedArgCount[protoCallFrame], temp3
&gt; 
&gt; Doesn&apos;t this change the behavior? We will end up loading an extra 32 bits. Was the old behavior a bug?

Yes, the old behavior is a bug.  Alternatively, we can change ProtoCallFrame::paddedArgCount to an unsigned instead if a size_t.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>999260</commentid>
    <comment_count>4</comment_count>
      <attachid>228975</attachid>
    <who name="Mark Hahnenberg">mhahnenberg</who>
    <bug_when>2014-04-09 12:53:06 -0700</bug_when>
    <thetext>Comment on attachment 228975
the patch

View in context: https://bugs.webkit.org/attachment.cgi?id=228975&amp;action=review

&gt;&gt;&gt; Source/JavaScriptCore/llint/LowLevelInterpreter64.asm:194
&gt;&gt;&gt; +    loadp ProtoCallFrame::paddedArgCount[protoCallFrame], temp3
&gt;&gt; 
&gt;&gt; Doesn&apos;t this change the behavior? We will end up loading an extra 32 bits. Was the old behavior a bug?
&gt; 
&gt; Yes, the old behavior is a bug.  Alternatively, we can change ProtoCallFrame::paddedArgCount to an unsigned instead if a size_t.

Hmm, I&apos;m unfamiliar with ProtoCallFrame. Does the size of the struct need to be consistent between 32-bit and 64-bit? If so then we should definitely change the type to uint32_t.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>999261</commentid>
    <comment_count>5</comment_count>
      <attachid>228975</attachid>
    <who name="Mark Hahnenberg">mhahnenberg</who>
    <bug_when>2014-04-09 13:03:03 -0700</bug_when>
    <thetext>Comment on attachment 228975
the patch

View in context: https://bugs.webkit.org/attachment.cgi?id=228975&amp;action=review

&gt;&gt;&gt;&gt; Source/JavaScriptCore/llint/LowLevelInterpreter64.asm:194
&gt;&gt;&gt;&gt; +    loadp ProtoCallFrame::paddedArgCount[protoCallFrame], temp3
&gt;&gt;&gt; 
&gt;&gt;&gt; Doesn&apos;t this change the behavior? We will end up loading an extra 32 bits. Was the old behavior a bug?
&gt;&gt; 
&gt;&gt; Yes, the old behavior is a bug.  Alternatively, we can change ProtoCallFrame::paddedArgCount to an unsigned instead if a size_t.
&gt; 
&gt; Hmm, I&apos;m unfamiliar with ProtoCallFrame. Does the size of the struct need to be consistent between 32-bit and 64-bit? If so then we should definitely change the type to uint32_t.

Okay, looks like the ProtoCallFrame is the dummy call frame we use when we&apos;re about to start executing JS. I vote that we make ProtoCallFrame&apos;s arg count match the type of CallFrame, which looks like an int32_t. (Register::payload returns an int32_t, which then gets casted to a size_t inside CallFrame::argumentCountIncludingThis). So maybe uint32_t makes the most sense here.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>999265</commentid>
    <comment_count>6</comment_count>
      <attachid>228978</attachid>
    <who name="Mark Lam">mark.lam</who>
    <bug_when>2014-04-09 13:17:43 -0700</bug_when>
    <thetext>Created attachment 228978
revised patch.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>999267</commentid>
    <comment_count>7</comment_count>
      <attachid>228978</attachid>
    <who name="Mark Hahnenberg">mhahnenberg</who>
    <bug_when>2014-04-09 13:19:54 -0700</bug_when>
    <thetext>Comment on attachment 228978
revised patch.

r=me!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>999270</commentid>
    <comment_count>8</comment_count>
    <who name="Mark Lam">mark.lam</who>
    <bug_when>2014-04-09 13:24:39 -0700</bug_when>
    <thetext>Thanks.  Landed in r167031: &lt;http://trac.webkit.org/r167031&gt;.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>999277</commentid>
    <comment_count>9</comment_count>
    <who name="Geoffrey Garen">ggaren</who>
    <bug_when>2014-04-09 14:02:28 -0700</bug_when>
    <thetext>Does this fix a bug? If so, it should come with a description of the bug and a regression test.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>999280</commentid>
    <comment_count>10</comment_count>
    <who name="Mark Lam">mark.lam</who>
    <bug_when>2014-04-09 14:06:55 -0700</bug_when>
    <thetext>(In reply to comment #9)
&gt; Does this fix a bug? If so, it should come with a description of the bug and a regression test.

The bug only manifests on a big endian system and was caught as soon as Tomas ran the JSC stress tests.  I’ll try to create a more isolated test case and post that later.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>999291</commentid>
    <comment_count>11</comment_count>
    <who name="Geoffrey Garen">ggaren</who>
    <bug_when>2014-04-09 14:23:49 -0700</bug_when>
    <thetext>&gt; The bug only manifests on a big endian system and was caught as soon as Tomas ran the JSC stress tests.  I’ll try to create a more isolated test case and post that later.

If this patch fixes a bug that&apos;s already covered by existing regression test(s), then you don&apos;t need to write a new regression test, and instead you should mention that the existing regression test(s) in your ChangeLog.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>999292</commentid>
    <comment_count>12</comment_count>
    <who name="Mark Lam">mark.lam</who>
    <bug_when>2014-04-09 14:24:40 -0700</bug_when>
    <thetext>(In reply to comment #11)
&gt; &gt; The bug only manifests on a big endian system and was caught as soon as Tomas ran the JSC stress tests.  I’ll try to create a more isolated test case and post that later.
&gt; 
&gt; If this patch fixes a bug that&apos;s already covered by existing regression test(s), then you don&apos;t need to write a new regression test, and instead you should mention that the existing regression test(s) in your ChangeLog.

I will update the ChangeLog comment.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>999308</commentid>
    <comment_count>13</comment_count>
    <who name="Mark Lam">mark.lam</who>
    <bug_when>2014-04-09 14:53:45 -0700</bug_when>
    <thetext>ChangeLog comment updated in r167037: &lt;http://trac.webkit.org/changeset/167037&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>228975</attachid>
            <date>2014-04-09 12:44:05 -0700</date>
            <delta_ts>2014-04-09 13:17:43 -0700</delta_ts>
            <desc>the patch</desc>
            <filename>bug-131449.patch</filename>
            <type>text/plain</type>
            <size>3164</size>
            <attacher name="Mark Lam">mark.lam</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291
cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkocmV2aXNpb24gMTY3MDIxKQorKysgU291cmNl
L0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE5IEBA
CisyMDE0LTA0LTA5ICBNYXJrIExhbSAgPG1hcmsubGFtQGFwcGxlLmNvbT4KKworICAgICAgICBF
bnN1cmUgdGhhdCBMTElOVCBhY2Nlc3Npbmcgb2YgdGhlIFByb3RvQ2FsbEZyYW1lIGlzIGJpZyBl
bmRpYW4gZnJpZW5kbHkuCisgICAgICAgIDxodHRwczovL3dlYmtpdC5vcmcvYi8xMzE0NDk+CisK
KyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgUHJvdG9DYWxs
RnJhbWU6OnBhZGRlZEFyZ0NvdW50IGlzIGEgc2l6ZV90LiAgV2Ugc2hvdWxkIGJlIGxvYWRpbmcg
aXQgd2l0aCBhCisgICAgICAgIGxvYWRwIGluc3RlYWQgb2YgYSBsb2FkaS4KKworICAgICAgICBX
ZSBzaG91bGQgYWRkIHRoZSBQYXlMb2FkT2Zmc2V0IHRvIFByb3RvQ2FsbEZyYW1lOjphcmdDb3Vu
dEFuZENvZGVPcmlnaW5WYWx1ZQorICAgICAgICB3aGVuIGxvYWRpbmcgdGhlIGFyZ0NvdW50Lgor
CisgICAgICAgICogbGxpbnQvTG93TGV2ZWxJbnRlcnByZXRlcjMyXzY0LmFzbToKKyAgICAgICAg
KiBsbGludC9Mb3dMZXZlbEludGVycHJldGVyNjQuYXNtOgorCiAyMDE0LTA0LTA4ICBPbGl2ZXIg
SHVudCAgPG9saXZlckBhcHBsZS5jb20+CiAKICAgICAgICAgUmV3cml0ZSBGdW5jdGlvbi5iaW5k
IGFzIGEgYnVpbHRpbgpJbmRleDogU291cmNlL0phdmFTY3JpcHRDb3JlL2xsaW50L0xvd0xldmVs
SW50ZXJwcmV0ZXIzMl82NC5hc20KPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291cmNlL0phdmFTY3JpcHRDb3Jl
L2xsaW50L0xvd0xldmVsSW50ZXJwcmV0ZXIzMl82NC5hc20JKHJldmlzaW9uIDE2NzAyMCkKKysr
IFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9sbGludC9Mb3dMZXZlbEludGVycHJldGVyMzJfNjQuYXNt
CSh3b3JraW5nIGNvcHkpCkBAIC0yNDQsNyArMjQ0LDcgQEAgbWFjcm8gZG9DYWxsVG9KYXZhU2Ny
aXB0KG1ha2VDYWxsKQogICAgICAgICBsb2FkcCA0NFtzcCwgdGVtcDNdLCBwcm90b0NhbGxGcmFt
ZQogICAgIGVuZAogCi0gICAgbG9hZGkgUHJvdG9DYWxsRnJhbWU6OnBhZGRlZEFyZ0NvdW50W3By
b3RvQ2FsbEZyYW1lXSwgdGVtcDIKKyAgICBsb2FkcCBQcm90b0NhbGxGcmFtZTo6cGFkZGVkQXJn
Q291bnRbcHJvdG9DYWxsRnJhbWVdLCB0ZW1wMgogICAgIGFkZHAgQ2FsbEZyYW1lSGVhZGVyU2xv
dHMsIHRlbXAyLCB0ZW1wMgogICAgIGxzaGlmdHAgMywgdGVtcDIKICAgICBzdWJwIGNmciwgdGVt
cDIsIHRlbXAxCkBAIC0yOTEsOSArMjkxLDkgQEAgbWFjcm8gZG9DYWxsVG9KYXZhU2NyaXB0KG1h
a2VDYWxsKQogICAgIHN0b3JlaSB0ZW1wMywgUGF5bG9hZE9mZnNldCArIENvZGVCbG9ja1tzcCwg
dGVtcDEsIDhdCiAgICAgYnRpbnogdGVtcDEsIC5jb3B5SGVhZGVyTG9vcAogCi0gICAgbG9hZGkg
UHJvdG9DYWxsRnJhbWU6OmFyZ0NvdW50QW5kQ29kZU9yaWdpblZhbHVlW3Byb3RvQ2FsbEZyYW1l
XSwgdGVtcDIKKyAgICBsb2FkaSBQYXlsb2FkT2Zmc2V0ICsgUHJvdG9DYWxsRnJhbWU6OmFyZ0Nv
dW50QW5kQ29kZU9yaWdpblZhbHVlW3Byb3RvQ2FsbEZyYW1lXSwgdGVtcDIKICAgICBzdWJpIDEs
IHRlbXAyCi0gICAgbG9hZGkgUHJvdG9DYWxsRnJhbWU6OnBhZGRlZEFyZ0NvdW50W3Byb3RvQ2Fs
bEZyYW1lXSwgdGVtcDMKKyAgICBsb2FkcCBQcm90b0NhbGxGcmFtZTo6cGFkZGVkQXJnQ291bnRb
cHJvdG9DYWxsRnJhbWVdLCB0ZW1wMwogICAgIHN1YmkgMSwgdGVtcDMKIAogICAgIGJpZXEgdGVt
cDIsIHRlbXAzLCAuY29weUFyZ3MKSW5kZXg6IFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9sbGludC9M
b3dMZXZlbEludGVycHJldGVyNjQuYXNtCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9KYXZhU2NyaXB0
Q29yZS9sbGludC9Mb3dMZXZlbEludGVycHJldGVyNjQuYXNtCShyZXZpc2lvbiAxNjcwMjApCisr
KyBTb3VyY2UvSmF2YVNjcmlwdENvcmUvbGxpbnQvTG93TGV2ZWxJbnRlcnByZXRlcjY0LmFzbQko
d29ya2luZyBjb3B5KQpAQCAtMTQ5LDcgKzE0OSw3IEBAIG1hY3JvIGRvQ2FsbFRvSmF2YVNjcmlw
dChtYWtlQ2FsbCkKICAgICBzdG9yZXAgcHJldmlvdXNQQywgUmV0dXJuUENbY2ZyXQogICAgIHN0
b3JlcCBwcmV2aW91c0NGUiwgQ2FsbGVyRnJhbWVbY2ZyXQogCi0gICAgbG9hZGkgUHJvdG9DYWxs
RnJhbWU6OnBhZGRlZEFyZ0NvdW50W3Byb3RvQ2FsbEZyYW1lXSwgdGVtcDIKKyAgICBsb2FkcCBQ
cm90b0NhbGxGcmFtZTo6cGFkZGVkQXJnQ291bnRbcHJvdG9DYWxsRnJhbWVdLCB0ZW1wMgogICAg
IGFkZHAgQ2FsbEZyYW1lSGVhZGVyU2xvdHMsIHRlbXAyLCB0ZW1wMgogICAgIGxzaGlmdHAgMywg
dGVtcDIKICAgICBzdWJwIGNmciwgdGVtcDIsIHRlbXAxCkBAIC0xODksOSArMTg5LDkgQEAgbWFj
cm8gZG9DYWxsVG9KYXZhU2NyaXB0KG1ha2VDYWxsKQogICAgIHN0b3JlcSB0ZW1wMywgQ29kZUJs
b2NrW3NwLCB0ZW1wMSwgOF0KICAgICBidGlueiB0ZW1wMSwgLmNvcHlIZWFkZXJMb29wCiAKLSAg
ICBsb2FkaSBQcm90b0NhbGxGcmFtZTo6YXJnQ291bnRBbmRDb2RlT3JpZ2luVmFsdWVbcHJvdG9D
YWxsRnJhbWVdLCB0ZW1wMgorICAgIGxvYWRpIFBheWxvYWRPZmZzZXQgKyBQcm90b0NhbGxGcmFt
ZTo6YXJnQ291bnRBbmRDb2RlT3JpZ2luVmFsdWVbcHJvdG9DYWxsRnJhbWVdLCB0ZW1wMgogICAg
IHN1YmkgMSwgdGVtcDIKLSAgICBsb2FkaSBQcm90b0NhbGxGcmFtZTo6cGFkZGVkQXJnQ291bnRb
cHJvdG9DYWxsRnJhbWVdLCB0ZW1wMworICAgIGxvYWRwIFByb3RvQ2FsbEZyYW1lOjpwYWRkZWRB
cmdDb3VudFtwcm90b0NhbGxGcmFtZV0sIHRlbXAzCiAgICAgc3ViaSAxLCB0ZW1wMwogCiAgICAg
YmllcSB0ZW1wMiwgdGVtcDMsIC5jb3B5QXJncwo=
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>228978</attachid>
            <date>2014-04-09 13:17:43 -0700</date>
            <delta_ts>2014-04-09 13:19:54 -0700</delta_ts>
            <desc>revised patch.</desc>
            <filename>bug-131449.patch</filename>
            <type>text/plain</type>
            <size>3615</size>
            <attacher name="Mark Lam">mark.lam</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291
cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkocmV2aXNpb24gMTY3MDI5KQorKysgU291cmNl
L0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDIzIEBA
CisyMDE0LTA0LTA5ICBNYXJrIExhbSAgPG1hcmsubGFtQGFwcGxlLmNvbT4KKworICAgICAgICBF
bnN1cmUgdGhhdCBMTElOVCBhY2Nlc3Npbmcgb2YgdGhlIFByb3RvQ2FsbEZyYW1lIGlzIGJpZyBl
bmRpYW4gZnJpZW5kbHkuCisgICAgICAgIDxodHRwczovL3dlYmtpdC5vcmcvYi8xMzE0NDk+CisK
KyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgQ2hhbmdlIFBy
b3RvQ2FsbEZyYW1lOjpwYWRkZWRBcmdDb3VudCB0byBiZSBvZiB0eXBlIHVpbnQzMl90LiAgVGhl
IGFyZ0NvdW50CisgICAgICAgIHRoYXQgaXQgcGFkcyBpcyBvZiB0eXBlIGludCBhbnl3YXkuICBJ
dCBkb2Vzbid0IG5lZWQgdG8gYmUgNjQgYml0LiAgVGhpcworICAgICAgICBhbHNvIG1ha2VzIGl0
IHdvcmsgd2l0aCB0aGUgTExJTlQgd2hpY2ggaXMgbG9hZGluZyBpdCB3aXRoIGEgbG9hZGkKKyAg
ICAgICAgaW5zdHJ1Y3Rpb24uCisKKyAgICAgICAgV2Ugc2hvdWxkIGFkZCB0aGUgUGF5TG9hZE9m
ZnNldCB0byBQcm90b0NhbGxGcmFtZTo6YXJnQ291bnRBbmRDb2RlT3JpZ2luVmFsdWUKKyAgICAg
ICAgd2hlbiBsb2FkaW5nIHRoZSBhcmdDb3VudC4KKworICAgICAgICAqIGludGVycHJldGVyL1By
b3RvQ2FsbEZyYW1lLmg6CisgICAgICAgIChKU0M6OlByb3RvQ2FsbEZyYW1lOjpzZXRQYWRkZWRB
cmdDb3VudCk6CisgICAgICAgICogbGxpbnQvTG93TGV2ZWxJbnRlcnByZXRlcjMyXzY0LmFzbToK
KyAgICAgICAgKiBsbGludC9Mb3dMZXZlbEludGVycHJldGVyNjQuYXNtOgorCiAyMDE0LTA0LTA4
ICBPbGl2ZXIgSHVudCAgPG9saXZlckBhcHBsZS5jb20+CiAKICAgICAgICAgUmV3cml0ZSBGdW5j
dGlvbi5iaW5kIGFzIGEgYnVpbHRpbgpJbmRleDogU291cmNlL0phdmFTY3JpcHRDb3JlL2ludGVy
cHJldGVyL1Byb3RvQ2FsbEZyYW1lLmgKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291cmNlL0phdmFTY3JpcHRD
b3JlL2ludGVycHJldGVyL1Byb3RvQ2FsbEZyYW1lLmgJKHJldmlzaW9uIDE2NzAyMCkKKysrIFNv
dXJjZS9KYXZhU2NyaXB0Q29yZS9pbnRlcnByZXRlci9Qcm90b0NhbGxGcmFtZS5oCSh3b3JraW5n
IGNvcHkpCkBAIC0zNiw3ICszNiw3IEBAIHN0cnVjdCBQcm90b0NhbGxGcmFtZSB7CiAgICAgUmVn
aXN0ZXIgY2FsbGVlVmFsdWU7CiAgICAgUmVnaXN0ZXIgYXJnQ291bnRBbmRDb2RlT3JpZ2luVmFs
dWU7CiAgICAgUmVnaXN0ZXIgdGhpc0FyZzsKLSAgICBzaXplX3QgcGFkZGVkQXJnQ291bnQ7Cisg
ICAgdWludDMyX3QgcGFkZGVkQXJnQ291bnQ7CiAgICAgSlNWYWx1ZSAqYXJnczsKIAogICAgIHZv
aWQgaW5pdChDb2RlQmxvY2sqLCBKU1Njb3BlKiwgSlNPYmplY3QqLCBKU1ZhbHVlLCBpbnQsIEpT
VmFsdWUqIG90aGVyQXJncyA9IDApOwpAQCAtNTMsNyArNTMsNyBAQCBzdHJ1Y3QgUHJvdG9DYWxs
RnJhbWUgewogICAgIGludCBhcmd1bWVudENvdW50SW5jbHVkaW5nVGhpcygpIGNvbnN0IHsgcmV0
dXJuIGFyZ0NvdW50QW5kQ29kZU9yaWdpblZhbHVlLnBheWxvYWQoKTsgfQogICAgIGludCBhcmd1
bWVudENvdW50KCkgY29uc3QgeyByZXR1cm4gYXJndW1lbnRDb3VudEluY2x1ZGluZ1RoaXMoKSAt
IDE7IH0KICAgICB2b2lkIHNldEFyZ3VtZW50Q291bnRJbmNsdWRpbmdUaGlzKGludCBjb3VudCkg
eyBhcmdDb3VudEFuZENvZGVPcmlnaW5WYWx1ZS5wYXlsb2FkKCkgPSBjb3VudDsgfQotICAgIHZv
aWQgc2V0UGFkZGVkQXJnQ291bnQoc2l6ZV90IGFyZ0NvdW50KSB7IHBhZGRlZEFyZ0NvdW50ID0g
YXJnQ291bnQ7IH0KKyAgICB2b2lkIHNldFBhZGRlZEFyZ0NvdW50KHVpbnQzMl90IGFyZ0NvdW50
KSB7IHBhZGRlZEFyZ0NvdW50ID0gYXJnQ291bnQ7IH0KIAogICAgIHZvaWQgY2xlYXJDdXJyZW50
VlBDKCkgeyBhcmdDb3VudEFuZENvZGVPcmlnaW5WYWx1ZS50YWcoKSA9IDA7IH0KICAgICAKSW5k
ZXg6IFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9sbGludC9Mb3dMZXZlbEludGVycHJldGVyMzJfNjQu
YXNtCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9sbGludC9Mb3dMZXZlbElu
dGVycHJldGVyMzJfNjQuYXNtCShyZXZpc2lvbiAxNjcwMjApCisrKyBTb3VyY2UvSmF2YVNjcmlw
dENvcmUvbGxpbnQvTG93TGV2ZWxJbnRlcnByZXRlcjMyXzY0LmFzbQkod29ya2luZyBjb3B5KQpA
QCAtMjkxLDcgKzI5MSw3IEBAIG1hY3JvIGRvQ2FsbFRvSmF2YVNjcmlwdChtYWtlQ2FsbCkKICAg
ICBzdG9yZWkgdGVtcDMsIFBheWxvYWRPZmZzZXQgKyBDb2RlQmxvY2tbc3AsIHRlbXAxLCA4XQog
ICAgIGJ0aW56IHRlbXAxLCAuY29weUhlYWRlckxvb3AKIAotICAgIGxvYWRpIFByb3RvQ2FsbEZy
YW1lOjphcmdDb3VudEFuZENvZGVPcmlnaW5WYWx1ZVtwcm90b0NhbGxGcmFtZV0sIHRlbXAyCisg
ICAgbG9hZGkgUGF5bG9hZE9mZnNldCArIFByb3RvQ2FsbEZyYW1lOjphcmdDb3VudEFuZENvZGVP
cmlnaW5WYWx1ZVtwcm90b0NhbGxGcmFtZV0sIHRlbXAyCiAgICAgc3ViaSAxLCB0ZW1wMgogICAg
IGxvYWRpIFByb3RvQ2FsbEZyYW1lOjpwYWRkZWRBcmdDb3VudFtwcm90b0NhbGxGcmFtZV0sIHRl
bXAzCiAgICAgc3ViaSAxLCB0ZW1wMwpJbmRleDogU291cmNlL0phdmFTY3JpcHRDb3JlL2xsaW50
L0xvd0xldmVsSW50ZXJwcmV0ZXI2NC5hc20KPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291cmNlL0phdmFTY3Jp
cHRDb3JlL2xsaW50L0xvd0xldmVsSW50ZXJwcmV0ZXI2NC5hc20JKHJldmlzaW9uIDE2NzAyMCkK
KysrIFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9sbGludC9Mb3dMZXZlbEludGVycHJldGVyNjQuYXNt
CSh3b3JraW5nIGNvcHkpCkBAIC0xODksNyArMTg5LDcgQEAgbWFjcm8gZG9DYWxsVG9KYXZhU2Ny
aXB0KG1ha2VDYWxsKQogICAgIHN0b3JlcSB0ZW1wMywgQ29kZUJsb2NrW3NwLCB0ZW1wMSwgOF0K
ICAgICBidGlueiB0ZW1wMSwgLmNvcHlIZWFkZXJMb29wCiAKLSAgICBsb2FkaSBQcm90b0NhbGxG
cmFtZTo6YXJnQ291bnRBbmRDb2RlT3JpZ2luVmFsdWVbcHJvdG9DYWxsRnJhbWVdLCB0ZW1wMgor
ICAgIGxvYWRpIFBheWxvYWRPZmZzZXQgKyBQcm90b0NhbGxGcmFtZTo6YXJnQ291bnRBbmRDb2Rl
T3JpZ2luVmFsdWVbcHJvdG9DYWxsRnJhbWVdLCB0ZW1wMgogICAgIHN1YmkgMSwgdGVtcDIKICAg
ICBsb2FkaSBQcm90b0NhbGxGcmFtZTo6cGFkZGVkQXJnQ291bnRbcHJvdG9DYWxsRnJhbWVdLCB0
ZW1wMwogICAgIHN1YmkgMSwgdGVtcDMK
</data>
<flag name="review"
          id="253328"
          type_id="1"
          status="+"
          setter="mhahnenberg"
    />
          </attachment>
      

    </bug>

</bugzilla>