<?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>159419</bug_id>
          
          <creation_ts>2016-07-05 07:37:54 -0700</creation_ts>
          <short_desc>[ARMv7] ASSERTION FAILED: (cond == Zero) || (cond == NonZero)</short_desc>
          <delta_ts>2016-07-07 01:33:32 -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>Other</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          <see_also>https://bugs.webkit.org/show_bug.cgi?id=155066</see_also>
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P1</priority>
          <bug_severity>Critical</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Csaba Osztrogonác">ossy</reporter>
          <assigned_to name="Csaba Osztrogonác">ossy</assigned_to>
          <cc>benjamin</cc>
    
    <cc>commit-queue</cc>
    
    <cc>fpizlo</cc>
    
    <cc>keith_miller</cc>
    
    <cc>mark.lam</cc>
    
    <cc>msaboff</cc>
    
    <cc>ossy</cc>
    
    <cc>saam</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1207824</commentid>
    <comment_count>0</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2016-07-05 07:37:54 -0700</bug_when>
    <thetext>https://trac.webkit.org/changeset/197655 modified the code in compileArithMul()
to check negative zero:
-            speculationCheck(NegativeZero, JSValueRegs(), 0, m_jit.branch32(MacroAssembler::LessThan, reg1, TrustedImm32(0)));
-            speculationCheck(NegativeZero, JSValueRegs(), 0, m_jit.branch32(MacroAssembler::LessThan, reg2, TrustedImm32(0)));
+            speculationCheck(NegativeZero, JSValueRegs(), 0, m_jit.branchTest32(MacroAssembler::Signed, reg1));
+            speculationCheck(NegativeZero, JSValueRegs(), 0, m_jit.branchTest32(MacroAssembler::Signed, reg2));

Using branchTest32 with signed conditional flag is incorrect, because the underlying 
tst ARM assembly instruction doesn&apos;t affect the V (signed/overflow) flag at all.

The MacroAssemblerARM.h asserts in this case properly.

ASSERTION FAILED: (cond == Zero) || (cond == NonZero)
../../Source/JavaScriptCore/assembler/MacroAssemblerARM.h(714) : JSC::AbstractMacroAssembler&lt;JSC::ARMAssembler, JSC::MacroAssemblerARM&gt;::Jump JSC::MacroAssemblerARM::branchTest32(JSC::MacroAssemblerARM::ResultCondition, JSC::AbstractMacroAssembler&lt;JSC::ARMAssembler, JSC::MacroAssemblerARM&gt;::RegisterID, JSC::AbstractMacroAssembler&lt;JSC::ARMAssembler, JSC::MacroAssemblerARM&gt;::TrustedImm32)
1   0x41af4060 WTFCrash
2   0x40fa0a20 JSC::MacroAssemblerARM::branchTest32(JSC::MacroAssemblerARM::ResultCondition, JSC::ARMRegisters::RegisterID, JSC::AbstractMacroAssembler&lt;JSC::ARMAssembler, JSC::MacroAssemblerARM&gt;::TrustedImm32)
3   0x413ab1e0 JSC::DFG::SpeculativeJIT::compileArithMul(JSC::DFG::Node*)
4   0x41410fec JSC::DFG::SpeculativeJIT::compile(JSC::DFG::Node*)
5   0x4139f5ec JSC::DFG::SpeculativeJIT::compileCurrentBlock()
6   0x4139fcec JSC::DFG::SpeculativeJIT::compile()
7   0x4127220c JSC::DFG::JITCompiler::compileBody()
8   0x41273e04 JSC::DFG::JITCompiler::compileFunction()
9   0x4134ed18 JSC::DFG::Plan::compileInThreadImpl(JSC::DFG::LongLivedState&amp;)
10  0x4134e078 JSC::DFG::Plan::compileInThread(JSC::DFG::LongLivedState&amp;, JSC::DFG::ThreadData*)
11  0x411f2e88
12  0x411f2fec JSC::DFG::compile(JSC::VM&amp;, JSC::CodeBlock*, JSC::CodeBlock*, JSC::DFG::CompilationMode, unsigned int, JSC::Operands&lt;JSC::JSValue, JSC::OperandValueTraits&lt;JSC::JSValue&gt; &gt; const&amp;, WTF::PassRefPtr&lt;JSC::DeferredCompilationCallback&gt;)
13  0x41600004
Illegal instruction
ERROR: Unexpected exit code: 132</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1207844</commentid>
    <comment_count>1</comment_count>
      <attachid>282794</attachid>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2016-07-05 08:29:17 -0700</bug_when>
    <thetext>Created attachment 282794
Patch

WIP patch, it fixes only the traditional ARM instruction set, will investigate the Thumb2 too.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1208126</commentid>
    <comment_count>2</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2016-07-06 01:49:57 -0700</bug_when>
    <thetext>(In reply to comment #0)
&gt; Using branchTest32 with signed conditional flag is incorrect, 
&gt; because the underlying tst ARM assembly instruction doesn&apos;t 
&gt; affect the V (signed/overflow) flag at all.

Ah, I was wrong. Not the V, but N flag is responsible for 
signed conditional which is really updated by the tst instruction.
So it&apos;s not a regression, but an incomplete assertion. Fix is coming.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1208127</commentid>
    <comment_count>3</comment_count>
      <attachid>282863</attachid>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2016-07-06 01:50:51 -0700</bug_when>
    <thetext>Created attachment 282863
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1208536</commentid>
    <comment_count>4</comment_count>
      <attachid>282863</attachid>
    <who name="Benjamin Poulain">benjamin</who>
    <bug_when>2016-07-06 22:08:44 -0700</bug_when>
    <thetext>Comment on attachment 282863
Patch

LGTM but you don&apos;t need all those parenthesis, that&apos;s excessive.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1208565</commentid>
    <comment_count>5</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2016-07-07 01:03:57 -0700</bug_when>
    <thetext>(In reply to comment #4)
&gt; Comment on attachment 282863 [details]
&gt; Patch
&gt; 
&gt; LGTM but you don&apos;t need all those parenthesis, that&apos;s excessive.

Thanks, I&apos;ll remove the extra parenthesis before landing.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1208566</commentid>
    <comment_count>6</comment_count>
      <attachid>282996</attachid>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2016-07-07 01:04:36 -0700</bug_when>
    <thetext>Created attachment 282996
Patch for landing</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1208569</commentid>
    <comment_count>7</comment_count>
      <attachid>282996</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2016-07-07 01:33:27 -0700</bug_when>
    <thetext>Comment on attachment 282996
Patch for landing

Clearing flags on attachment: 282996

Committed r202899: &lt;http://trac.webkit.org/changeset/202899&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1208570</commentid>
    <comment_count>8</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2016-07-07 01:33:32 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>282794</attachid>
            <date>2016-07-05 08:29:17 -0700</date>
            <delta_ts>2016-07-06 01:50:45 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-159419-20160705082846.patch</filename>
            <type>text/plain</type>
            <size>1559</size>
            <attacher name="Csaba Osztrogonác">ossy</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjAyODE3CmRpZmYgLS1naXQgYS9Tb3VyY2UvSmF2YVNjcmlw
dENvcmUvQ2hhbmdlTG9nIGIvU291cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwppbmRleCBk
M2VmNGQ0ZDUwOWYyY2VhMmY2YzIzNzQ1NTQ3YTI2MjU5YjQzOWRhLi5hYWMzODRkMWQwNGQ5Mzc5
MDZmNWU3MjY0NGM5ZTdkNzIyNTRiNTAwIDEwMDY0NAotLS0gYS9Tb3VyY2UvSmF2YVNjcmlwdENv
cmUvQ2hhbmdlTG9nCisrKyBiL1NvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKQEAgLTEs
MyArMSwxMyBAQAorMjAxNi0wNy0wNSAgQ3NhYmEgT3N6dHJvZ29uw6FjICA8b3NzeUB3ZWJraXQu
b3JnPgorCisgICAgICAgIFtBUk12N10gUkVHUkVTU0lPTihyMTk3NjU1KTogQVNTRVJUSU9OIEZB
SUxFRDogKGNvbmQgPT0gWmVybykgfHwgKGNvbmQgPT0gTm9uWmVybykKKyAgICAgICAgaHR0cHM6
Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTE1OTQxOQorCisgICAgICAgIFJldmll
d2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgICogYXNzZW1ibGVyL01hY3JvQXNzZW1i
bGVyQVJNLmg6CisgICAgICAgIChKU0M6Ok1hY3JvQXNzZW1ibGVyQVJNOjpicmFuY2hUZXN0MzIp
OgorCiAyMDE2LTA3LTAzICBQZXIgQXJuZSBWb2xsYW4gIDxwdm9sbGFuQGFwcGxlLmNvbT4KIAog
ICAgICAgICBbV2luXSBETExzIGFyZSBtaXNzaW5nIHZlcnNpb24gaW5mb3JtYXRpb24uCmRpZmYg
LS1naXQgYS9Tb3VyY2UvSmF2YVNjcmlwdENvcmUvYXNzZW1ibGVyL01hY3JvQXNzZW1ibGVyQVJN
LmggYi9Tb3VyY2UvSmF2YVNjcmlwdENvcmUvYXNzZW1ibGVyL01hY3JvQXNzZW1ibGVyQVJNLmgK
aW5kZXggZmNiMjBhYzg2Njk3NmY3MTQ2YjdhMjRiNDlkMmRhMjE0Mjk4OWEzMy4uYWQ0Zjk1Y2M2
MjJhYWQ3YTIyMmRlZmQ5ZWI2M2VhYTdmNmE1Y2ZkYSAxMDA2NDQKLS0tIGEvU291cmNlL0phdmFT
Y3JpcHRDb3JlL2Fzc2VtYmxlci9NYWNyb0Fzc2VtYmxlckFSTS5oCisrKyBiL1NvdXJjZS9KYXZh
U2NyaXB0Q29yZS9hc3NlbWJsZXIvTWFjcm9Bc3NlbWJsZXJBUk0uaApAQCAtNzExLDYgKzcxMSwx
MSBAQCBwdWJsaWM6CiAKICAgICBKdW1wIGJyYW5jaFRlc3QzMihSZXN1bHRDb25kaXRpb24gY29u
ZCwgUmVnaXN0ZXJJRCByZWcsIFRydXN0ZWRJbW0zMiBtYXNrID0gVHJ1c3RlZEltbTMyKC0xKSkK
ICAgICB7CisgICAgICAgIGlmIChjb25kID09IFNpZ25lZCAmJiBtYXNrLm1fdmFsdWUgPT0gLTEp
IHsKKyAgICAgICAgICAgIG1fYXNzZW1ibGVyLnRlcShyZWcsIHJlZyk7CisgICAgICAgICAgICBy
ZXR1cm4gSnVtcChtX2Fzc2VtYmxlci5qbXAoQVJNQ29uZGl0aW9uKGNvbmQpKSk7CisgICAgICAg
IH0KKwogICAgICAgICBBU1NFUlQoKGNvbmQgPT0gWmVybykgfHwgKGNvbmQgPT0gTm9uWmVybykp
OwogICAgICAgICBBUk1Xb3JkIHcgPSBtX2Fzc2VtYmxlci5nZXRJbW0obWFzay5tX3ZhbHVlLCBB
Uk1SZWdpc3RlcnM6OlMwLCB0cnVlKTsKICAgICAgICAgaWYgKHcgJiBBUk1Bc3NlbWJsZXI6Ok9w
MkludmVydGVkSW1tZWRpYXRlKQo=
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>282863</attachid>
            <date>2016-07-06 01:50:51 -0700</date>
            <delta_ts>2016-07-07 01:04:29 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-159419-20160706015019.patch</filename>
            <type>text/plain</type>
            <size>3120</size>
            <attacher name="Csaba Osztrogonác">ossy</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjAyODE3CmRpZmYgLS1naXQgYS9Tb3VyY2UvSmF2YVNjcmlw
dENvcmUvQ2hhbmdlTG9nIGIvU291cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwppbmRleCBk
M2VmNGQ0ZDUwOWYyY2VhMmY2YzIzNzQ1NTQ3YTI2MjU5YjQzOWRhLi41MTkxNWM2ZDY1OTI1YWVl
MWExOTEwZjQ3Mjc5MjdiZmYzYTczNmNhIDEwMDY0NAotLS0gYS9Tb3VyY2UvSmF2YVNjcmlwdENv
cmUvQ2hhbmdlTG9nCisrKyBiL1NvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKQEAgLTEs
MyArMSwxNyBAQAorMjAxNi0wNy0wNiAgQ3NhYmEgT3N6dHJvZ29uw6FjICA8b3NzeUB3ZWJraXQu
b3JnPgorCisgICAgICAgIFtBUk12N10gUkVHUkVTU0lPTihyMTk3NjU1KTogQVNTRVJUSU9OIEZB
SUxFRDogKGNvbmQgPT0gWmVybykgfHwgKGNvbmQgPT0gTm9uWmVybykKKyAgICAgICAgaHR0cHM6
Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTE1OTQxOQorCisgICAgICAgIFJldmll
d2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIEFsbG93IFNpZ25lZCBhbmQgUG9zaXRp
dmVPclplcm8gY29uZGl0aW9ucyB0b28gYmVjYXVzZSB0c3QgaW5zdHJ1Y3Rpb24gdXBkYXRlcyBO
IGFuZCBaIGZsYWdzLgorCisgICAgICAgICogYXNzZW1ibGVyL01hY3JvQXNzZW1ibGVyQVJNLmg6
CisgICAgICAgIChKU0M6Ok1hY3JvQXNzZW1ibGVyQVJNOjpicmFuY2hUZXN0MzIpOgorICAgICAg
ICAqIGFzc2VtYmxlci9NYWNyb0Fzc2VtYmxlckFSTXY3Lmg6CisgICAgICAgIChKU0M6Ok1hY3Jv
QXNzZW1ibGVyQVJNdjc6OmJyYW5jaFRlc3QzMik6IEFkZCBhc3NlcnRpb25zIHRvIGF2b2lkIHBv
c3NpYmxlIGJ1Z3MgaW4gdGhlIGZ1dHVyZS4KKwogMjAxNi0wNy0wMyAgUGVyIEFybmUgVm9sbGFu
ICA8cHZvbGxhbkBhcHBsZS5jb20+CiAKICAgICAgICAgW1dpbl0gRExMcyBhcmUgbWlzc2luZyB2
ZXJzaW9uIGluZm9ybWF0aW9uLgpkaWZmIC0tZ2l0IGEvU291cmNlL0phdmFTY3JpcHRDb3JlL2Fz
c2VtYmxlci9NYWNyb0Fzc2VtYmxlckFSTS5oIGIvU291cmNlL0phdmFTY3JpcHRDb3JlL2Fzc2Vt
Ymxlci9NYWNyb0Fzc2VtYmxlckFSTS5oCmluZGV4IGZjYjIwYWM4NjY5NzZmNzE0NmI3YTI0YjQ5
ZDJkYTIxNDI5ODlhMzMuLjE1ZjY1ZDAzODVkMDllODQyNzA3ZjVhOGM3MDhiNzE4MWQ0NGFiNTgg
MTAwNjQ0Ci0tLSBhL1NvdXJjZS9KYXZhU2NyaXB0Q29yZS9hc3NlbWJsZXIvTWFjcm9Bc3NlbWJs
ZXJBUk0uaAorKysgYi9Tb3VyY2UvSmF2YVNjcmlwdENvcmUvYXNzZW1ibGVyL01hY3JvQXNzZW1i
bGVyQVJNLmgKQEAgLTcwNCwxNCArNzA0LDE0IEBAIHB1YmxpYzoKIAogICAgIEp1bXAgYnJhbmNo
VGVzdDMyKFJlc3VsdENvbmRpdGlvbiBjb25kLCBSZWdpc3RlcklEIHJlZywgUmVnaXN0ZXJJRCBt
YXNrKQogICAgIHsKLSAgICAgICAgQVNTRVJUKChjb25kID09IFplcm8pIHx8IChjb25kID09IE5v
blplcm8pKTsKKyAgICAgICAgQVNTRVJUKChjb25kID09IFplcm8pIHx8IChjb25kID09IE5vblpl
cm8pIHx8IChjb25kID09IFNpZ25lZCkgfHwgKGNvbmQgPT0gUG9zaXRpdmVPclplcm8pKTsKICAg
ICAgICAgbV9hc3NlbWJsZXIudHN0KHJlZywgbWFzayk7CiAgICAgICAgIHJldHVybiBKdW1wKG1f
YXNzZW1ibGVyLmptcChBUk1Db25kaXRpb24oY29uZCkpKTsKICAgICB9CiAKICAgICBKdW1wIGJy
YW5jaFRlc3QzMihSZXN1bHRDb25kaXRpb24gY29uZCwgUmVnaXN0ZXJJRCByZWcsIFRydXN0ZWRJ
bW0zMiBtYXNrID0gVHJ1c3RlZEltbTMyKC0xKSkKICAgICB7Ci0gICAgICAgIEFTU0VSVCgoY29u
ZCA9PSBaZXJvKSB8fCAoY29uZCA9PSBOb25aZXJvKSk7CisgICAgICAgIEFTU0VSVCgoY29uZCA9
PSBaZXJvKSB8fCAoY29uZCA9PSBOb25aZXJvKSB8fCAoY29uZCA9PSBTaWduZWQpIHx8IChjb25k
ID09IFBvc2l0aXZlT3JaZXJvKSk7CiAgICAgICAgIEFSTVdvcmQgdyA9IG1fYXNzZW1ibGVyLmdl
dEltbShtYXNrLm1fdmFsdWUsIEFSTVJlZ2lzdGVyczo6UzAsIHRydWUpOwogICAgICAgICBpZiAo
dyAmIEFSTUFzc2VtYmxlcjo6T3AySW52ZXJ0ZWRJbW1lZGlhdGUpCiAgICAgICAgICAgICBtX2Fz
c2VtYmxlci5iaWNzKEFSTVJlZ2lzdGVyczo6UzAsIHJlZywgdyAmIH5BUk1Bc3NlbWJsZXI6Ok9w
MkludmVydGVkSW1tZWRpYXRlKTsKZGlmZiAtLWdpdCBhL1NvdXJjZS9KYXZhU2NyaXB0Q29yZS9h
c3NlbWJsZXIvTWFjcm9Bc3NlbWJsZXJBUk12Ny5oIGIvU291cmNlL0phdmFTY3JpcHRDb3JlL2Fz
c2VtYmxlci9NYWNyb0Fzc2VtYmxlckFSTXY3LmgKaW5kZXggZWY2YTU0MjM3ZWJmNjYxYzhlOWU2
NDlhNzI0MWViZWRhY2ZlNjQ1Zi4uMmEyZTk0ZTgwZjIxMmJjMWRhMjIxZjc5MTU1MDRkM2MyOGNi
ZWNhNyAxMDA2NDQKLS0tIGEvU291cmNlL0phdmFTY3JpcHRDb3JlL2Fzc2VtYmxlci9NYWNyb0Fz
c2VtYmxlckFSTXY3LmgKKysrIGIvU291cmNlL0phdmFTY3JpcHRDb3JlL2Fzc2VtYmxlci9NYWNy
b0Fzc2VtYmxlckFSTXY3LmgKQEAgLTE1MDYsMTIgKzE1MDYsMTQgQEAgcHVibGljOgogICAgIAog
ICAgIEp1bXAgYnJhbmNoVGVzdDMyKFJlc3VsdENvbmRpdGlvbiBjb25kLCBSZWdpc3RlcklEIHJl
ZywgUmVnaXN0ZXJJRCBtYXNrKQogICAgIHsKKyAgICAgICAgQVNTRVJUKChjb25kID09IFplcm8p
IHx8IChjb25kID09IE5vblplcm8pIHx8IChjb25kID09IFNpZ25lZCkgfHwgKGNvbmQgPT0gUG9z
aXRpdmVPclplcm8pKTsKICAgICAgICAgbV9hc3NlbWJsZXIudHN0KHJlZywgbWFzayk7CiAgICAg
ICAgIHJldHVybiBKdW1wKG1ha2VCcmFuY2goY29uZCkpOwogICAgIH0KIAogICAgIEp1bXAgYnJh
bmNoVGVzdDMyKFJlc3VsdENvbmRpdGlvbiBjb25kLCBSZWdpc3RlcklEIHJlZywgVHJ1c3RlZElt
bTMyIG1hc2sgPSBUcnVzdGVkSW1tMzIoLTEpKQogICAgIHsKKyAgICAgICAgQVNTRVJUKChjb25k
ID09IFplcm8pIHx8IChjb25kID09IE5vblplcm8pIHx8IChjb25kID09IFNpZ25lZCkgfHwgKGNv
bmQgPT0gUG9zaXRpdmVPclplcm8pKTsKICAgICAgICAgdGVzdDMyKHJlZywgbWFzayk7CiAgICAg
ICAgIHJldHVybiBKdW1wKG1ha2VCcmFuY2goY29uZCkpOwogICAgIH0K
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>282996</attachid>
            <date>2016-07-07 01:04:36 -0700</date>
            <delta_ts>2016-07-07 01:33:27 -0700</delta_ts>
            <desc>Patch for landing</desc>
            <filename>bug-159419-20160707010402.patch</filename>
            <type>text/plain</type>
            <size>3101</size>
            <attacher name="Csaba Osztrogonác">ossy</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjAyODk3CmRpZmYgLS1naXQgYS9Tb3VyY2UvSmF2YVNjcmlw
dENvcmUvQ2hhbmdlTG9nIGIvU291cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwppbmRleCBk
YWRlY2U2YWYzY2QzZjkwYjQ1ODhmYTBiZTVhNzBhYTJlYThkMGFhLi45YjExN2E2MTJhYjA3NDZj
ZmFiNGUzZmE1YjEwNTZmNTBhZTg3ZjZjIDEwMDY0NAotLS0gYS9Tb3VyY2UvSmF2YVNjcmlwdENv
cmUvQ2hhbmdlTG9nCisrKyBiL1NvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKQEAgLTEs
MyArMSwxNyBAQAorMjAxNi0wNy0wNyAgQ3NhYmEgT3N6dHJvZ29uw6FjICA8b3NzeUB3ZWJraXQu
b3JnPgorCisgICAgICAgIFtBUk12N10gUkVHUkVTU0lPTihyMTk3NjU1KTogQVNTRVJUSU9OIEZB
SUxFRDogKGNvbmQgPT0gWmVybykgfHwgKGNvbmQgPT0gTm9uWmVybykKKyAgICAgICAgaHR0cHM6
Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTE1OTQxOQorCisgICAgICAgIFJldmll
d2VkIGJ5IEJlbmphbWluIFBvdWxhaW4uCisKKyAgICAgICAgQWxsb3cgU2lnbmVkIGFuZCBQb3Np
dGl2ZU9yWmVybyBjb25kaXRpb25zIHRvbyBiZWNhdXNlIHRzdCBpbnN0cnVjdGlvbiB1cGRhdGVz
IE4gYW5kIFogZmxhZ3MuCisKKyAgICAgICAgKiBhc3NlbWJsZXIvTWFjcm9Bc3NlbWJsZXJBUk0u
aDoKKyAgICAgICAgKEpTQzo6TWFjcm9Bc3NlbWJsZXJBUk06OmJyYW5jaFRlc3QzMik6CisgICAg
ICAgICogYXNzZW1ibGVyL01hY3JvQXNzZW1ibGVyQVJNdjcuaDoKKyAgICAgICAgKEpTQzo6TWFj
cm9Bc3NlbWJsZXJBUk12Nzo6YnJhbmNoVGVzdDMyKTogQWRkIGFzc2VydGlvbnMgdG8gYXZvaWQg
cG9zc2libGUgYnVncyBpbiB0aGUgZnV0dXJlLgorCiAyMDE2LTA3LTA2ICBZb3Vlbm4gRmFibGV0
ICA8eW91ZW5uQGFwcGxlLmNvbT4KIAogICAgICAgICBCdWlsdGluIGdlbmVyYXRvciBzaG91bGQg
dXNlIHByYWdtYSBvbmNlIGZvciBoZWFkZXIgZmlsZXMKZGlmZiAtLWdpdCBhL1NvdXJjZS9KYXZh
U2NyaXB0Q29yZS9hc3NlbWJsZXIvTWFjcm9Bc3NlbWJsZXJBUk0uaCBiL1NvdXJjZS9KYXZhU2Ny
aXB0Q29yZS9hc3NlbWJsZXIvTWFjcm9Bc3NlbWJsZXJBUk0uaAppbmRleCBmY2IyMGFjODY2OTc2
ZjcxNDZiN2EyNGI0OWQyZGEyMTQyOTg5YTMzLi43OTgyMTFmMDYyZmE0MGI2NzNiOGVhNDk1ZDk4
YjhmYjU0NTkxMzhhIDEwMDY0NAotLS0gYS9Tb3VyY2UvSmF2YVNjcmlwdENvcmUvYXNzZW1ibGVy
L01hY3JvQXNzZW1ibGVyQVJNLmgKKysrIGIvU291cmNlL0phdmFTY3JpcHRDb3JlL2Fzc2VtYmxl
ci9NYWNyb0Fzc2VtYmxlckFSTS5oCkBAIC03MDQsMTQgKzcwNCwxNCBAQCBwdWJsaWM6CiAKICAg
ICBKdW1wIGJyYW5jaFRlc3QzMihSZXN1bHRDb25kaXRpb24gY29uZCwgUmVnaXN0ZXJJRCByZWcs
IFJlZ2lzdGVySUQgbWFzaykKICAgICB7Ci0gICAgICAgIEFTU0VSVCgoY29uZCA9PSBaZXJvKSB8
fCAoY29uZCA9PSBOb25aZXJvKSk7CisgICAgICAgIEFTU0VSVChjb25kID09IFplcm8gfHwgY29u
ZCA9PSBOb25aZXJvIHx8IGNvbmQgPT0gU2lnbmVkIHx8IGNvbmQgPT0gUG9zaXRpdmVPclplcm8p
OwogICAgICAgICBtX2Fzc2VtYmxlci50c3QocmVnLCBtYXNrKTsKICAgICAgICAgcmV0dXJuIEp1
bXAobV9hc3NlbWJsZXIuam1wKEFSTUNvbmRpdGlvbihjb25kKSkpOwogICAgIH0KIAogICAgIEp1
bXAgYnJhbmNoVGVzdDMyKFJlc3VsdENvbmRpdGlvbiBjb25kLCBSZWdpc3RlcklEIHJlZywgVHJ1
c3RlZEltbTMyIG1hc2sgPSBUcnVzdGVkSW1tMzIoLTEpKQogICAgIHsKLSAgICAgICAgQVNTRVJU
KChjb25kID09IFplcm8pIHx8IChjb25kID09IE5vblplcm8pKTsKKyAgICAgICAgQVNTRVJUKGNv
bmQgPT0gWmVybyB8fCBjb25kID09IE5vblplcm8gfHwgY29uZCA9PSBTaWduZWQgfHwgY29uZCA9
PSBQb3NpdGl2ZU9yWmVybyk7CiAgICAgICAgIEFSTVdvcmQgdyA9IG1fYXNzZW1ibGVyLmdldElt
bShtYXNrLm1fdmFsdWUsIEFSTVJlZ2lzdGVyczo6UzAsIHRydWUpOwogICAgICAgICBpZiAodyAm
IEFSTUFzc2VtYmxlcjo6T3AySW52ZXJ0ZWRJbW1lZGlhdGUpCiAgICAgICAgICAgICBtX2Fzc2Vt
Ymxlci5iaWNzKEFSTVJlZ2lzdGVyczo6UzAsIHJlZywgdyAmIH5BUk1Bc3NlbWJsZXI6Ok9wMklu
dmVydGVkSW1tZWRpYXRlKTsKZGlmZiAtLWdpdCBhL1NvdXJjZS9KYXZhU2NyaXB0Q29yZS9hc3Nl
bWJsZXIvTWFjcm9Bc3NlbWJsZXJBUk12Ny5oIGIvU291cmNlL0phdmFTY3JpcHRDb3JlL2Fzc2Vt
Ymxlci9NYWNyb0Fzc2VtYmxlckFSTXY3LmgKaW5kZXggZWY2YTU0MjM3ZWJmNjYxYzhlOWU2NDlh
NzI0MWViZWRhY2ZlNjQ1Zi4uMjIzMzZkMDJhYjYzOGFiMmUzNmM5NjcxNDdhOGI2MzE3NWM0Mzc2
YyAxMDA2NDQKLS0tIGEvU291cmNlL0phdmFTY3JpcHRDb3JlL2Fzc2VtYmxlci9NYWNyb0Fzc2Vt
YmxlckFSTXY3LmgKKysrIGIvU291cmNlL0phdmFTY3JpcHRDb3JlL2Fzc2VtYmxlci9NYWNyb0Fz
c2VtYmxlckFSTXY3LmgKQEAgLTE1MDYsMTIgKzE1MDYsMTQgQEAgcHVibGljOgogICAgIAogICAg
IEp1bXAgYnJhbmNoVGVzdDMyKFJlc3VsdENvbmRpdGlvbiBjb25kLCBSZWdpc3RlcklEIHJlZywg
UmVnaXN0ZXJJRCBtYXNrKQogICAgIHsKKyAgICAgICAgQVNTRVJUKGNvbmQgPT0gWmVybyB8fCBj
b25kID09IE5vblplcm8gfHwgY29uZCA9PSBTaWduZWQgfHwgY29uZCA9PSBQb3NpdGl2ZU9yWmVy
byk7CiAgICAgICAgIG1fYXNzZW1ibGVyLnRzdChyZWcsIG1hc2spOwogICAgICAgICByZXR1cm4g
SnVtcChtYWtlQnJhbmNoKGNvbmQpKTsKICAgICB9CiAKICAgICBKdW1wIGJyYW5jaFRlc3QzMihS
ZXN1bHRDb25kaXRpb24gY29uZCwgUmVnaXN0ZXJJRCByZWcsIFRydXN0ZWRJbW0zMiBtYXNrID0g
VHJ1c3RlZEltbTMyKC0xKSkKICAgICB7CisgICAgICAgIEFTU0VSVChjb25kID09IFplcm8gfHwg
Y29uZCA9PSBOb25aZXJvIHx8IGNvbmQgPT0gU2lnbmVkIHx8IGNvbmQgPT0gUG9zaXRpdmVPclpl
cm8pOwogICAgICAgICB0ZXN0MzIocmVnLCBtYXNrKTsKICAgICAgICAgcmV0dXJuIEp1bXAobWFr
ZUJyYW5jaChjb25kKSk7CiAgICAgfQo=
</data>

          </attachment>
      

    </bug>

</bugzilla>