<?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>216437</bug_id>
          
          <creation_ts>2020-09-12 09:03:30 -0700</creation_ts>
          <short_desc>Coerce computed property before adding to |excludedList|</short_desc>
          <delta_ts>2020-09-22 15:23:14 -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>Safari 13</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>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="HyeockJin Kim">kherootz</reporter>
          <assigned_to name="HyeockJin Kim">kherootz</assigned_to>
          <cc>ashvayka</cc>
    
    <cc>darin</cc>
    
    <cc>ews-watchlist</cc>
    
    <cc>keith_miller</cc>
    
    <cc>mark.lam</cc>
    
    <cc>msaboff</cc>
    
    <cc>saam</cc>
    
    <cc>tzagallo</cc>
    
    <cc>webkit-bug-importer</cc>
    
    <cc>ysuzuki</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1687927</commentid>
    <comment_count>0</comment_count>
    <who name="HyeockJin Kim">kherootz</who>
    <bug_when>2020-09-12 09:03:30 -0700</bug_when>
    <thetext>version: 13.1.2(15609.3.5.1.3)

chrome:
&gt; ({[1]: c, ...b} = {1: 4})
{1: 4}
&gt; b
{}
 
Firefox:
&gt; ({[1]: c, ...b} = {1: 4})
Object { 1: 4 }

&gt; b
Object {  }

Safari:
&gt; ({1: c, ...b} = {1: 4})
&lt; {1: 4}
&gt; b
&lt; {}

&gt; ({[1]: c, ...b} = {1: 4})
&lt; {1: 4}
&gt; b
&lt; {1: 4}

When using the bracket key in destructuring assignment, rest ignores the bracket key.

If the issue hasn&apos;t been fixed yet, can I try this issue?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1687932</commentid>
    <comment_count>1</comment_count>
    <who name="Alexey Shvayka">ashvayka</who>
    <bug_when>2020-09-12 09:33:52 -0700</bug_when>
    <thetext>(In reply to HyeockJin Kim from comment #0)
&gt; When using the bracket key in destructuring assignment, rest ignores the
&gt; bracket key.
&gt; 
&gt; If the issue hasn&apos;t been fixed yet, can I try this issue?

This is a great find, thank you!
The bug is still there as of r266924, a patch is very welcome.
In case you need assistance, there is a #jsc channel on Slack: https://webkit.org/getting-started/#staying-in-touch.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1689909</commentid>
    <comment_count>2</comment_count>
      <attachid>409160</attachid>
    <who name="HyeockJin Kim">kherootz</who>
    <bug_when>2020-09-18 12:20:48 -0700</bug_when>
    <thetext>Created attachment 409160
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1689910</commentid>
    <comment_count>3</comment_count>
    <who name="HyeockJin Kim">kherootz</who>
    <bug_when>2020-09-18 12:22:18 -0700</bug_when>
    <thetext>If the computed property is not a string value, the property in the excludedList was not recognized because the type was different.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1689927</commentid>
    <comment_count>4</comment_count>
      <attachid>409160</attachid>
    <who name="Alexey Shvayka">ashvayka</who>
    <bug_when>2020-09-18 12:51:27 -0700</bug_when>
    <thetext>Comment on attachment 409160
Patch

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

&gt; Source/JavaScriptCore/bytecompiler/NodesCodegen.cpp:5237
&gt; +                    generator.emitToString(propertyString.get(), propertyName.get());

I don&apos;t think we can call ToString here as the `propertyName` might have a symbol, rising a TypeError.
https://tc39.es/ecma262/#sec-object-initializer-runtime-semantics-evaluation for ComputedPropertyName calls ToPropertyKey.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1689933</commentid>
    <comment_count>5</comment_count>
    <who name="Alexey Shvayka">ashvayka</who>
    <bug_when>2020-09-18 13:05:04 -0700</bug_when>
    <thetext>Also, could you please run `Tools/Scripts/test262-runner --release --save`? There might be new test(s) passing.
If there are none, it would be great to submit a PR to https://github.com/tc39/test262 so other runtimes would get fixed if needed.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1689976</commentid>
    <comment_count>6</comment_count>
      <attachid>409160</attachid>
    <who name="Yusuke Suzuki">ysuzuki</who>
    <bug_when>2020-09-18 15:46:23 -0700</bug_when>
    <thetext>Comment on attachment 409160
Patch

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

&gt;&gt; Source/JavaScriptCore/bytecompiler/NodesCodegen.cpp:5237
&gt;&gt; +                    generator.emitToString(propertyString.get(), propertyName.get());
&gt; 
&gt; I don&apos;t think we can call ToString here as the `propertyName` might have a symbol, rising a TypeError.
&gt; https://tc39.es/ecma262/#sec-object-initializer-runtime-semantics-evaluation for ComputedPropertyName calls ToPropertyKey.

We should avoid emitting this if we know that this is not necessary. For example, if target.propertyExpression is nullptr, we know what is specified in propertyName.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1689984</commentid>
    <comment_count>7</comment_count>
      <attachid>409160</attachid>
    <who name="Yusuke Suzuki">ysuzuki</who>
    <bug_when>2020-09-18 16:14:39 -0700</bug_when>
    <thetext>Comment on attachment 409160
Patch

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

&gt;&gt;&gt; Source/JavaScriptCore/bytecompiler/NodesCodegen.cpp:5237
&gt;&gt;&gt; +                    generator.emitToString(propertyString.get(), propertyName.get());
&gt;&gt; 
&gt;&gt; I don&apos;t think we can call ToString here as the `propertyName` might have a symbol, rising a TypeError.
&gt;&gt; https://tc39.es/ecma262/#sec-object-initializer-runtime-semantics-evaluation for ComputedPropertyName calls ToPropertyKey.
&gt; 
&gt; We should avoid emitting this if we know that this is not necessary. For example, if target.propertyExpression is nullptr, we know what is specified in propertyName.

Yeah, so let&apos;s call `emitToPropertyKey`.
And avoid emitting this if propertyName is known as value that does not need `ToPropertyKey`.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1689985</commentid>
    <comment_count>8</comment_count>
    <who name="Yusuke Suzuki">ysuzuki</who>
    <bug_when>2020-09-18 16:14:58 -0700</bug_when>
    <thetext>I think this is related. https://github.com/tc39/test262/issues/2089</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1690055</commentid>
    <comment_count>9</comment_count>
      <attachid>409198</attachid>
    <who name="HyeockJin Kim">kherootz</who>
    <bug_when>2020-09-19 01:45:56 -0700</bug_when>
    <thetext>Created attachment 409198
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1690056</commentid>
    <comment_count>10</comment_count>
    <who name="HyeockJin Kim">kherootz</who>
    <bug_when>2020-09-19 01:56:33 -0700</bug_when>
    <thetext>(In reply to Alexey Shvayka from comment #5)
&gt; Also, could you please run `Tools/Scripts/test262-runner --release --save`?
&gt; There might be new test(s) passing.
&gt; If there are none, it would be great to submit a PR to
&gt; https://github.com/tc39/test262 so other runtimes would get fixed if needed.


JSTests/test262/expectations.yaml has been modified by executing the command.
Is this correct?

If not, is there any reference for me on how to send a PR to https://github.com/tc39/test262?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1690057</commentid>
    <comment_count>11</comment_count>
    <who name="Alexey Shvayka">ashvayka</who>
    <bug_when>2020-09-19 02:07:14 -0700</bug_when>
    <thetext>(In reply to HyeockJin Kim from comment #10)
&gt; JSTests/test262/expectations.yaml has been modified by executing the command.
&gt; Is this correct?

--save caused the modification, yet intl402/Collator failures are unrelated.
Please revert JSTests/test262/expectations.yaml change.

&gt; If not, is there any reference for me on how to send a PR to
&gt; https://github.com/tc39/test262?

You need to clone tc39/test262/src/dstr-assignment/obj-rest-computed-property.case (it is a template), make relevant modifications (see https://github.com/tc39/test262/issues/2089 for examples), and re-generate .js tests (like tc39/test262/test/language/statements/for-of/dstr/obj-rest-computed-property.js) via Python script (see https://github.com/tc39/test262/blob/main/CONTRIBUTING.md#procedurally-generated-tests).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1690058</commentid>
    <comment_count>12</comment_count>
      <attachid>409198</attachid>
    <who name="Alexey Shvayka">ashvayka</who>
    <bug_when>2020-09-19 02:10:57 -0700</bug_when>
    <thetext>Comment on attachment 409198
Patch

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

&gt; Source/JavaScriptCore/ChangeLog:3
&gt; +        Add the property to be excluded from the rest operation of the object as string

This is a bit hard to follow IMO.
What about &quot;Coerce computed property before adding to |excludedList|&quot;?

&gt; Source/JavaScriptCore/bytecompiler/NodesCodegen.cpp:5233
&gt;                      if (!target.propertyExpression)

Maybe we should flip this negation?

&gt; JSTests/stress/object-rest-deconstruct.js:205
&gt; +// Destructuring non-string computed property

Could you please add a Symbol() test (that would fail with previous patch)?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1690075</commentid>
    <comment_count>13</comment_count>
      <attachid>409198</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2020-09-19 07:54:53 -0700</bug_when>
    <thetext>Comment on attachment 409198
Patch

Need to upload a new patch that applies cleanly, ideally also with the changes that Alexey requested.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1690083</commentid>
    <comment_count>14</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2020-09-19 09:04:18 -0700</bug_when>
    <thetext>&lt;rdar://problem/69215687&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1690106</commentid>
    <comment_count>15</comment_count>
      <attachid>409198</attachid>
    <who name="Yusuke Suzuki">ysuzuki</who>
    <bug_when>2020-09-19 12:22:02 -0700</bug_when>
    <thetext>Comment on attachment 409198
Patch

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

&gt; Source/JavaScriptCore/bytecompiler/NodesCodegen.cpp:5234
&gt;                          propertyName = generator.emitLoad(nullptr, target.propertyName);

Let&apos;s check what the target.propertyExpression is. If it is number constant, then we need to generate JSString from that here.
If it is not JSString, then we need to have generator.emitToPropertyKey. But if it is JSString, we do not need it.
And I think this special handling is very important since in major cases, it is JSString or Number. So we can simply avoid ToPropertyName call for most cases.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1690107</commentid>
    <comment_count>16</comment_count>
      <attachid>409198</attachid>
    <who name="Yusuke Suzuki">ysuzuki</who>
    <bug_when>2020-09-19 12:23:32 -0700</bug_when>
    <thetext>Comment on attachment 409198
Patch

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

&gt;&gt; Source/JavaScriptCore/bytecompiler/NodesCodegen.cpp:5234
&gt;&gt;                          propertyName = generator.emitLoad(nullptr, target.propertyName);
&gt; 
&gt; Let&apos;s check what the target.propertyExpression is. If it is number constant, then we need to generate JSString from that here.
&gt; If it is not JSString, then we need to have generator.emitToPropertyKey. But if it is JSString, we do not need it.
&gt; And I think this special handling is very important since in major cases, it is JSString or Number. So we can simply avoid ToPropertyName call for most cases.

Ah, no. The above should work since target.propertyName is `const Identifer&amp;`. So it is always producing JSStirng. Ignore my comment above.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1690116</commentid>
    <comment_count>17</comment_count>
    <who name="HyeockJin Kim">kherootz</who>
    <bug_when>2020-09-19 13:32:49 -0700</bug_when>
    <thetext>(In reply to Alexey Shvayka from comment #12)
&gt; &gt; Source/JavaScriptCore/ChangeLog:3
&gt; &gt; +        Add the property to be excluded from the rest operation of the object as string
&gt; 
&gt; This is a bit hard to follow IMO.
&gt; What about &quot;Coerce computed property before adding to |excludedList|&quot;?
&gt; 

I change title.

&gt; &gt; Source/JavaScriptCore/bytecompiler/NodesCodegen.cpp:5233
&gt; &gt;                      if (!target.propertyExpression)
&gt; 
&gt; Maybe we should flip this negation?
&gt; 

Segmentation fault occurs when do negation this condition.
&gt;&gt;&gt; a = 1
1
&gt;&gt;&gt; ({[a]:b, ...c} = {[a]: 2, 2:3})
[1]    73132 segmentation fault  ./Tools/Scripts/run-jsc --jsc-only


&gt; &gt; JSTests/stress/object-rest-deconstruct.js:205
&gt; &gt; +// Destructuring non-string computed property
&gt; 
&gt; Could you please add a Symbol() test (that would fail with previous patch)?

I didn&apos;t understand what the `Symbol() test` means.
I&apos;m sorry, can you explain a little more?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1690117</commentid>
    <comment_count>18</comment_count>
    <who name="HyeockJin Kim">kherootz</who>
    <bug_when>2020-09-19 13:36:54 -0700</bug_when>
    <thetext>(In reply to Alexey Shvayka from comment #11)
&gt; (In reply to HyeockJin Kim from comment #10)

&gt; You need to clone
&gt; tc39/test262/src/dstr-assignment/obj-rest-computed-property.case (it is a
&gt; template), make relevant modifications (see
&gt; https://github.com/tc39/test262/issues/2089 for examples), and re-generate
&gt; .js tests (like
&gt; tc39/test262/test/language/statements/for-of/dstr/obj-rest-computed-property.
&gt; js) via Python script (see
&gt; https://github.com/tc39/test262/blob/main/CONTRIBUTING.md#procedurally-
&gt; generated-tests).

I did PR, is it correct?
https://github.com/tc39/test262/pull/2799</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1690155</commentid>
    <comment_count>19</comment_count>
      <attachid>409198</attachid>
    <who name="Yusuke Suzuki">ysuzuki</who>
    <bug_when>2020-09-20 02:23:05 -0700</bug_when>
    <thetext>Comment on attachment 409198
Patch

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

Can you rebase the patch to make it applied to the tip of tree? :) The other part looks good.

&gt;&gt; Source/JavaScriptCore/bytecompiler/NodesCodegen.cpp:5233
&gt;&gt;                      if (!target.propertyExpression)
&gt; 
&gt; Maybe we should flip this negation?

I think &quot;negation&quot; here means,

if (target.propertyExpression)
    generator.emitToPropertyKey(propertyName.get(), propertyName.get());
else
    propertyName = generator.emitLoad(nullptr, target.propertyName);

&gt; JSTests/test262/expectations.yaml:1557
&gt; +test/intl402/Collator/ignore-invalid-unicode-ext-values.js:
&gt; +  default: &apos;Test262Error: Locale ko is affected by key co; value standard. Expected SameValue(Â«koÂ», Â«ko-KRÂ») to be true&apos;
&gt; +  strict mode: &apos;Test262Error: Locale ko is affected by key co; value standard. Expected SameValue(Â«koÂ», Â«ko-KRÂ») to be true&apos;
&gt;  test/intl402/Collator/missing-unicode-ext-value-defaults-to-true.js:
&gt; -  default: &quot;Test262Error: \&quot;kn-true\&quot; is returned in locale, but shouldn&apos;t be. Expected SameValue(Â«7Â», Â«-1Â») to be true&quot;
&gt; -  strict mode: &quot;Test262Error: \&quot;kn-true\&quot; is returned in locale, but shouldn&apos;t be. Expected SameValue(Â«7Â», Â«-1Â») to be true&quot;
&gt; +  default: &quot;Test262Error: \&quot;kn-true\&quot; is returned in locale, but shouldn&apos;t be. Expected SameValue(Â«4Â», Â«-1Â») to be true&quot;
&gt; +  strict mode: &quot;Test262Error: \&quot;kn-true\&quot; is returned in locale, but shouldn&apos;t be. Expected SameValue(Â«4Â», Â«-1Â») to be true&quot;
&gt;  test/intl402/DateTimeFormat/constructor-options-order-fractionalSecondDigits.js:

I think this is due to ICU version. So let&apos;s exclude this. I think test262 does not cover this case.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1690158</commentid>
    <comment_count>20</comment_count>
      <attachid>409229</attachid>
    <who name="HyeockJin Kim">kherootz</who>
    <bug_when>2020-09-20 02:57:39 -0700</bug_when>
    <thetext>Created attachment 409229
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1690159</commentid>
    <comment_count>21</comment_count>
      <attachid>409230</attachid>
    <who name="HyeockJin Kim">kherootz</who>
    <bug_when>2020-09-20 03:10:31 -0700</bug_when>
    <thetext>Created attachment 409230
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1690164</commentid>
    <comment_count>22</comment_count>
    <who name="HyeockJin Kim">kherootz</who>
    <bug_when>2020-09-20 06:06:43 -0700</bug_when>
    <thetext>(In reply to Yusuke Suzuki from comment #19)
&gt; Comment on attachment 409198 [details]
&gt; Patch
&gt; 
&gt; View in context:
&gt; https://bugs.webkit.org/attachment.cgi?id=409198&amp;action=review
&gt; 
&gt; Can you rebase the patch to make it applied to the tip of tree? :) The other
&gt; part looks good.

I think it&apos;s all fixed. :)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1690698</commentid>
    <comment_count>23</comment_count>
    <who name="HyeockJin Kim">kherootz</who>
    <bug_when>2020-09-22 08:20:19 -0700</bug_when>
    <thetext>(In reply to Alexey Shvayka from comment #11)
&gt; You need to clone
&gt; tc39/test262/src/dstr-assignment/obj-rest-computed-property.case (it is a
&gt; template), make relevant modifications (see
&gt; https://github.com/tc39/test262/issues/2089 for examples), and re-generate
&gt; .js tests (like
&gt; tc39/test262/test/language/statements/for-of/dstr/obj-rest-computed-property.
&gt; js) via Python script (see
&gt; https://github.com/tc39/test262/blob/main/CONTRIBUTING.md#procedurally-
&gt; generated-tests).

I sent a PR and it was merged. :)
https://github.com/tc39/test262/pull/2799</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1690878</commentid>
    <comment_count>24</comment_count>
      <attachid>409230</attachid>
    <who name="Yusuke Suzuki">ysuzuki</who>
    <bug_when>2020-09-22 15:20:20 -0700</bug_when>
    <thetext>Comment on attachment 409230
Patch

r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1690882</commentid>
    <comment_count>25</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2020-09-22 15:23:13 -0700</bug_when>
    <thetext>Committed r267440: &lt;https://trac.webkit.org/changeset/267440&gt;

All reviewed patches have been landed. Closing bug and clearing flags on attachment 409230.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>409160</attachid>
            <date>2020-09-18 12:20:48 -0700</date>
            <delta_ts>2020-09-19 01:45:53 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-216437-20200919042047.patch</filename>
            <type>text/plain</type>
            <size>3332</size>
            <attacher name="HyeockJin Kim">kherootz</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjY2MzIyCmRpZmYgLS1naXQgYS9Tb3VyY2UvSmF2YVNjcmlw
dENvcmUvQ2hhbmdlTG9nIGIvU291cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwppbmRleCBk
NDM3MDhiZjZmNWZjYmRjNDczOWM4OTdjZmZmZDFmMDA4MGJmNWVmLi4yZTkxZWQ2ODkxNGMzM2I5
ODMyNmZjNThlYzBmZWI4YTJiMGNkYmU4IDEwMDY0NAotLS0gYS9Tb3VyY2UvSmF2YVNjcmlwdENv
cmUvQ2hhbmdlTG9nCisrKyBiL1NvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKQEAgLTEs
MyArMSwxMyBAQAorMjAyMC0wOS0xOCAgSHllb2NrSmluIEtpbSAgPGtoZXJvb3R6QGdtYWlsLmNv
bT4KKworICAgICAgICBBZGQgdGhlIHByb3BlcnR5IHRvIGJlIGV4Y2x1ZGVkIGZyb20gdGhlIHJl
c3Qgb3BlcmF0aW9uIG9mIHRoZSBvYmplY3QgYXMgc3RyaW5nCisgICAgICAgIGh0dHBzOi8vYnVn
cy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0yMTY0MzcKKworICAgICAgICBSZXZpZXdlZCBi
eSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICAqIGJ5dGVjb21waWxlci9Ob2Rlc0NvZGVnZW4u
Y3BwOgorICAgICAgICAoSlNDOjpPYmplY3RQYXR0ZXJuTm9kZTo6YmluZFZhbHVlIGNvbnN0KToK
KwogMjAyMC0wOC0yOCAgWXVzdWtlIFN1enVraSAgPHlzdXp1a2lAYXBwbGUuY29tPgogCiAgICAg
ICAgIFtKU0NdIHN1cGVyIHByb3BlcnR5IHdpdGggbmV3IHNob3VsZCBiZSBhY2NlcHRlZApkaWZm
IC0tZ2l0IGEvU291cmNlL0phdmFTY3JpcHRDb3JlL2J5dGVjb21waWxlci9Ob2Rlc0NvZGVnZW4u
Y3BwIGIvU291cmNlL0phdmFTY3JpcHRDb3JlL2J5dGVjb21waWxlci9Ob2Rlc0NvZGVnZW4uY3Bw
CmluZGV4IDkzZjYyY2RlNmM3MGQzZjZiOTk0Y2Y5ZGI0ZGQzMWQ0ZTcyNGVlODIuLjdlZWY0OWMw
ZmQ3ZGI4YWQ5NTBiYzFhYjI2YmM0N2VmMDg2MGMxYjYgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9KYXZh
U2NyaXB0Q29yZS9ieXRlY29tcGlsZXIvTm9kZXNDb2RlZ2VuLmNwcAorKysgYi9Tb3VyY2UvSmF2
YVNjcmlwdENvcmUvYnl0ZWNvbXBpbGVyL05vZGVzQ29kZWdlbi5jcHAKQEAgLTUyMzMsOSArNTIz
MywxMiBAQCB2b2lkIE9iamVjdFBhdHRlcm5Ob2RlOjpiaW5kVmFsdWUoQnl0ZWNvZGVHZW5lcmF0
b3ImIGdlbmVyYXRvciwgUmVnaXN0ZXJJRCogcmhzKQogICAgICAgICAgICAgICAgICAgICBpZiAo
IXRhcmdldC5wcm9wZXJ0eUV4cHJlc3Npb24pCiAgICAgICAgICAgICAgICAgICAgICAgICBwcm9w
ZXJ0eU5hbWUgPSBnZW5lcmF0b3IuZW1pdExvYWQobnVsbHB0ciwgdGFyZ2V0LnByb3BlcnR5TmFt
ZSk7CiAKKyAgICAgICAgICAgICAgICAgICAgUmVmUHRyPFJlZ2lzdGVySUQ+IHByb3BlcnR5U3Ry
aW5nID0gZ2VuZXJhdG9yLm5ld1RlbXBvcmFyeSgpOworICAgICAgICAgICAgICAgICAgICBnZW5l
cmF0b3IuZW1pdFRvU3RyaW5nKHByb3BlcnR5U3RyaW5nLmdldCgpLCBwcm9wZXJ0eU5hbWUuZ2V0
KCkpOworCiAgICAgICAgICAgICAgICAgICAgIENhbGxBcmd1bWVudHMgYXJncyhnZW5lcmF0b3Is
IG51bGxwdHIsIDEpOwogICAgICAgICAgICAgICAgICAgICBnZW5lcmF0b3IubW92ZShhcmdzLnRo
aXNSZWdpc3RlcigpLCBleGNsdWRlZExpc3QuZ2V0KCkpOwotICAgICAgICAgICAgICAgICAgICBn
ZW5lcmF0b3IubW92ZShhcmdzLmFyZ3VtZW50UmVnaXN0ZXIoMCksIHByb3BlcnR5TmFtZS5nZXQo
KSk7CisgICAgICAgICAgICAgICAgICAgIGdlbmVyYXRvci5tb3ZlKGFyZ3MuYXJndW1lbnRSZWdp
c3RlcigwKSwgcHJvcGVydHlTdHJpbmcuZ2V0KCkpOwogICAgICAgICAgICAgICAgICAgICBnZW5l
cmF0b3IuZW1pdENhbGwoZ2VuZXJhdG9yLm5ld1RlbXBvcmFyeSgpLCBhZGRNZXRob2QuZ2V0KCks
IE5vRXhwZWN0ZWRGdW5jdGlvbiwgYXJncywgZGl2b3QoKSwgZGl2b3RTdGFydCgpLCBkaXZvdEVu
ZCgpLCBEZWJ1Z2dhYmxlQ2FsbDo6Tm8pOwogICAgICAgICAgICAgICAgIH0gZWxzZQogICAgICAg
ICAgICAgICAgICAgICBleGNsdWRlZFNldC5hZGQodGFyZ2V0LnByb3BlcnR5TmFtZS5pbXBsKCkp
OwpkaWZmIC0tZ2l0IGEvSlNUZXN0cy9DaGFuZ2VMb2cgYi9KU1Rlc3RzL0NoYW5nZUxvZwppbmRl
eCA4ZGI1MzdmY2M0NTkxNTUxNzEyMzZlYjYyMzFhNjVlOWM2OTRhNzc5Li4yMjI1ZTZhMGRlMThi
NmJmNzU2NWRmMGUyYjA5NDA3ZmFhZjQ0YTgyIDEwMDY0NAotLS0gYS9KU1Rlc3RzL0NoYW5nZUxv
ZworKysgYi9KU1Rlc3RzL0NoYW5nZUxvZwpAQCAtMSwzICsxLDEzIEBACisyMDIwLTA5LTE4ICBI
eWVvY2tKaW4gS2ltICA8a2hlcm9vdHpAZ21haWwuY29tPgorCisgICAgICAgIEFkZCB0aGUgcHJv
cGVydHkgdG8gYmUgZXhjbHVkZWQgZnJvbSB0aGUgcmVzdCBvcGVyYXRpb24gb2YgdGhlIG9iamVj
dCBhcyBzdHJpbmcKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dp
P2lkPTIxNjQzNworCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAg
ICAgICogc3RyZXNzL29iamVjdC1yZXN0LWRlY29uc3RydWN0LmpzOgorICAgICAgICAoZ2V0IDMp
OgorCiAyMDIwLTA4LTI4ICBZdXN1a2UgU3V6dWtpICA8eXN1enVraUBhcHBsZS5jb20+CiAKICAg
ICAgICAgW0pTQ10gc3VwZXIgcHJvcGVydHkgd2l0aCBuZXcgc2hvdWxkIGJlIGFjY2VwdGVkCmRp
ZmYgLS1naXQgYS9KU1Rlc3RzL3N0cmVzcy9vYmplY3QtcmVzdC1kZWNvbnN0cnVjdC5qcyBiL0pT
VGVzdHMvc3RyZXNzL29iamVjdC1yZXN0LWRlY29uc3RydWN0LmpzCmluZGV4IGM5ODRlNzE0MGI1
Njc1MWFhNmJjNDZlMTgyZjg1MGU3NDcxMTRiMGEuLmE5MmNkNTg0NjMxMjY4YmNmZTlkODU1ZGQ5
YWI0YzNjOGRhMjhkM2MgMTAwNjQ0Ci0tLSBhL0pTVGVzdHMvc3RyZXNzL29iamVjdC1yZXN0LWRl
Y29uc3RydWN0LmpzCisrKyBiL0pTVGVzdHMvc3RyZXNzL29iamVjdC1yZXN0LWRlY29uc3RydWN0
LmpzCkBAIC0yMDIsNiArMjAyLDE2IEBAIGxldCBhc3NlcnRQcm9wRGVzY3JpcHRvciA9IChyZXN0
T2JqLCBwcm9wKSA9PiB7CiAgICAgYXNzZXJ0KHIuYmF6ID09PSAzKTsKIH0pKCk7CiAKKy8vIERl
c3RydWN0dXJpbmcgbm9uLXN0cmluZyBjb21wdXRlZCBwcm9wZXJ0eQorKCgpID0+IHsKKyAgICB2
YXIgYSA9IDE7CisgICAgdmFyIHtbYV06IGIsIC4uLnJ9ID0ge1thXTogMSwgYjogMiwgYzogM307
CisgICAgYXNzZXJ0KGIgPT09IDEpOworICAgIGFzc2VydChyWzFdID09PSB1bmRlZmluZWQpOwor
ICAgIGFzc2VydChyLmIgPT09IDIpOworICAgIGFzc2VydChyLmMgPT09IDMpOworfSkoKTsKKwog
Ly8gQ2F0Y2ggY2FzZQogCiAoKCkgPT4gewo=
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>409198</attachid>
            <date>2020-09-19 01:45:56 -0700</date>
            <delta_ts>2020-09-20 02:57:37 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-216437-20200919174555.patch</filename>
            <type>text/plain</type>
            <size>5188</size>
            <attacher name="HyeockJin Kim">kherootz</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjY2MzIyCmRpZmYgLS1naXQgYS9Tb3VyY2UvSmF2YVNjcmlw
dENvcmUvQ2hhbmdlTG9nIGIvU291cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwppbmRleCBk
NDM3MDhiZjZmNWZjYmRjNDczOWM4OTdjZmZmZDFmMDA4MGJmNWVmLi44MThjOGIyNTdjZDA4MWMx
M2Y1ZjQ3NTBiNWUxODg0YzBkZGJkYjMzIDEwMDY0NAotLS0gYS9Tb3VyY2UvSmF2YVNjcmlwdENv
cmUvQ2hhbmdlTG9nCisrKyBiL1NvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKQEAgLTEs
MyArMSwxMyBAQAorMjAyMC0wOS0xOSAgSHllb2NrSmluIEtpbSAgPGtoZXJvb3R6QGdtYWlsLmNv
bT4KKworICAgICAgICBBZGQgdGhlIHByb3BlcnR5IHRvIGJlIGV4Y2x1ZGVkIGZyb20gdGhlIHJl
c3Qgb3BlcmF0aW9uIG9mIHRoZSBvYmplY3QgYXMgc3RyaW5nCisgICAgICAgIGh0dHBzOi8vYnVn
cy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0yMTY0MzcKKworICAgICAgICBSZXZpZXdlZCBi
eSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICAqIGJ5dGVjb21waWxlci9Ob2Rlc0NvZGVnZW4u
Y3BwOgorICAgICAgICAoSlNDOjpPYmplY3RQYXR0ZXJuTm9kZTo6YmluZFZhbHVlIGNvbnN0KToK
KwogMjAyMC0wOC0yOCAgWXVzdWtlIFN1enVraSAgPHlzdXp1a2lAYXBwbGUuY29tPgogCiAgICAg
ICAgIFtKU0NdIHN1cGVyIHByb3BlcnR5IHdpdGggbmV3IHNob3VsZCBiZSBhY2NlcHRlZApkaWZm
IC0tZ2l0IGEvU291cmNlL0phdmFTY3JpcHRDb3JlL2J5dGVjb21waWxlci9Ob2Rlc0NvZGVnZW4u
Y3BwIGIvU291cmNlL0phdmFTY3JpcHRDb3JlL2J5dGVjb21waWxlci9Ob2Rlc0NvZGVnZW4uY3Bw
CmluZGV4IDkzZjYyY2RlNmM3MGQzZjZiOTk0Y2Y5ZGI0ZGQzMWQ0ZTcyNGVlODIuLjMzZDdlYjY2
ZDQ1MGE0OGRiMmYzZGNlZjgyMGI5OTE2NzdjMzQzN2IgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9KYXZh
U2NyaXB0Q29yZS9ieXRlY29tcGlsZXIvTm9kZXNDb2RlZ2VuLmNwcAorKysgYi9Tb3VyY2UvSmF2
YVNjcmlwdENvcmUvYnl0ZWNvbXBpbGVyL05vZGVzQ29kZWdlbi5jcHAKQEAgLTUyMzIsNiArNTIz
Miw4IEBAIHZvaWQgT2JqZWN0UGF0dGVybk5vZGU6OmJpbmRWYWx1ZShCeXRlY29kZUdlbmVyYXRv
ciYgZ2VuZXJhdG9yLCBSZWdpc3RlcklEKiByaHMpCiAgICAgICAgICAgICAgICAgaWYgKG1fY29u
dGFpbnNDb21wdXRlZFByb3BlcnR5KSB7CiAgICAgICAgICAgICAgICAgICAgIGlmICghdGFyZ2V0
LnByb3BlcnR5RXhwcmVzc2lvbikKICAgICAgICAgICAgICAgICAgICAgICAgIHByb3BlcnR5TmFt
ZSA9IGdlbmVyYXRvci5lbWl0TG9hZChudWxscHRyLCB0YXJnZXQucHJvcGVydHlOYW1lKTsKKyAg
ICAgICAgICAgICAgICAgICAgZWxzZQorICAgICAgICAgICAgICAgICAgICAgICAgZ2VuZXJhdG9y
LmVtaXRUb1Byb3BlcnR5S2V5KHByb3BlcnR5TmFtZS5nZXQoKSwgcHJvcGVydHlOYW1lLmdldCgp
KTsKIAogICAgICAgICAgICAgICAgICAgICBDYWxsQXJndW1lbnRzIGFyZ3MoZ2VuZXJhdG9yLCBu
dWxscHRyLCAxKTsKICAgICAgICAgICAgICAgICAgICAgZ2VuZXJhdG9yLm1vdmUoYXJncy50aGlz
UmVnaXN0ZXIoKSwgZXhjbHVkZWRMaXN0LmdldCgpKTsKZGlmZiAtLWdpdCBhL0pTVGVzdHMvQ2hh
bmdlTG9nIGIvSlNUZXN0cy9DaGFuZ2VMb2cKaW5kZXggOGRiNTM3ZmNjNDU5MTU1MTcxMjM2ZWI2
MjMxYTY1ZTljNjk0YTc3OS4uZGQ0Njk1NjMyZDcxZmYwNTM5MmU1NzIxMDM4NmFiZmQ2MWJhNDNl
MyAxMDA2NDQKLS0tIGEvSlNUZXN0cy9DaGFuZ2VMb2cKKysrIGIvSlNUZXN0cy9DaGFuZ2VMb2cK
QEAgLTEsMyArMSwxNCBAQAorMjAyMC0wOS0xOSAgSHllb2NrSmluIEtpbSAgPGtoZXJvb3R6QGdt
YWlsLmNvbT4KKworICAgICAgICBBZGQgdGhlIHByb3BlcnR5IHRvIGJlIGV4Y2x1ZGVkIGZyb20g
dGhlIHJlc3Qgb3BlcmF0aW9uIG9mIHRoZSBvYmplY3QgYXMgc3RyaW5nCisgICAgICAgIGh0dHBz
Oi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0yMTY0MzcKKworICAgICAgICBSZXZp
ZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICAqIHN0cmVzcy9vYmplY3QtcmVzdC1k
ZWNvbnN0cnVjdC5qczoKKyAgICAgICAgKGdldCAzKToKKyAgICAgICAgKiB0ZXN0MjYyL2V4cGVj
dGF0aW9ucy55YW1sOgorCiAyMDIwLTA4LTI4ICBZdXN1a2UgU3V6dWtpICA8eXN1enVraUBhcHBs
ZS5jb20+CiAKICAgICAgICAgW0pTQ10gc3VwZXIgcHJvcGVydHkgd2l0aCBuZXcgc2hvdWxkIGJl
IGFjY2VwdGVkCmRpZmYgLS1naXQgYS9KU1Rlc3RzL3N0cmVzcy9vYmplY3QtcmVzdC1kZWNvbnN0
cnVjdC5qcyBiL0pTVGVzdHMvc3RyZXNzL29iamVjdC1yZXN0LWRlY29uc3RydWN0LmpzCmluZGV4
IGM5ODRlNzE0MGI1Njc1MWFhNmJjNDZlMTgyZjg1MGU3NDcxMTRiMGEuLmE5MmNkNTg0NjMxMjY4
YmNmZTlkODU1ZGQ5YWI0YzNjOGRhMjhkM2MgMTAwNjQ0Ci0tLSBhL0pTVGVzdHMvc3RyZXNzL29i
amVjdC1yZXN0LWRlY29uc3RydWN0LmpzCisrKyBiL0pTVGVzdHMvc3RyZXNzL29iamVjdC1yZXN0
LWRlY29uc3RydWN0LmpzCkBAIC0yMDIsNiArMjAyLDE2IEBAIGxldCBhc3NlcnRQcm9wRGVzY3Jp
cHRvciA9IChyZXN0T2JqLCBwcm9wKSA9PiB7CiAgICAgYXNzZXJ0KHIuYmF6ID09PSAzKTsKIH0p
KCk7CiAKKy8vIERlc3RydWN0dXJpbmcgbm9uLXN0cmluZyBjb21wdXRlZCBwcm9wZXJ0eQorKCgp
ID0+IHsKKyAgICB2YXIgYSA9IDE7CisgICAgdmFyIHtbYV06IGIsIC4uLnJ9ID0ge1thXTogMSwg
YjogMiwgYzogM307CisgICAgYXNzZXJ0KGIgPT09IDEpOworICAgIGFzc2VydChyWzFdID09PSB1
bmRlZmluZWQpOworICAgIGFzc2VydChyLmIgPT09IDIpOworICAgIGFzc2VydChyLmMgPT09IDMp
OworfSkoKTsKKwogLy8gQ2F0Y2ggY2FzZQogCiAoKCkgPT4gewpkaWZmIC0tZ2l0IGEvSlNUZXN0
cy90ZXN0MjYyL2V4cGVjdGF0aW9ucy55YW1sIGIvSlNUZXN0cy90ZXN0MjYyL2V4cGVjdGF0aW9u
cy55YW1sCmluZGV4IGNhZjRiYTMyNzRkMzlhODhhMWE4YThhYjBiODU0OTQ0YzQxNWE0N2UuLjUy
ZTI5NjhmNzMyOTMzNGYyZmU5NjM1ZTEwNmE0ZjRmNDA5ZDI2OTcgMTAwNjQ0Ci0tLSBhL0pTVGVz
dHMvdGVzdDI2Mi9leHBlY3RhdGlvbnMueWFtbAorKysgYi9KU1Rlc3RzL3Rlc3QyNjIvZXhwZWN0
YXRpb25zLnlhbWwKQEAgLTE1NDgsOSArMTU0OCwxMiBAQCB0ZXN0L2J1aWx0LWlucy9UeXBlZEFy
cmF5Q29uc3RydWN0b3JzL29mL2N1c3RvbS1jdG9yLmpzOgogdGVzdC9idWlsdC1pbnMvVHlwZWRB
cnJheUNvbnN0cnVjdG9ycy9vZi9uZXctaW5zdGFuY2UtdXNpbmctY3VzdG9tLWN0b3IuanM6CiAg
IGRlZmF1bHQ6ICdUeXBlRXJyb3I6IFR5cGVkQXJyYXkub2YgcmVxdWlyZXMgaXRzIHRoaXMgYXJn
dW1lbnQgdG8gc3ViY2xhc3MgYSBUeXBlZEFycmF5IGNvbnN0cnVjdG9yIChUZXN0aW5nIHdpdGgg
RmxvYXQ2NEFycmF5LiknCiAgIHN0cmljdCBtb2RlOiAnVHlwZUVycm9yOiBUeXBlZEFycmF5Lm9m
IHJlcXVpcmVzIGl0cyB0aGlzIGFyZ3VtZW50IHRvIHN1YmNsYXNzIGEgVHlwZWRBcnJheSBjb25z
dHJ1Y3RvciAoVGVzdGluZyB3aXRoIEZsb2F0NjRBcnJheS4pJwordGVzdC9pbnRsNDAyL0NvbGxh
dG9yL2lnbm9yZS1pbnZhbGlkLXVuaWNvZGUtZXh0LXZhbHVlcy5qczoKKyAgZGVmYXVsdDogJ1Rl
c3QyNjJFcnJvcjogTG9jYWxlIGtvIGlzIGFmZmVjdGVkIGJ5IGtleSBjbzsgdmFsdWUgc3RhbmRh
cmQuIEV4cGVjdGVkIFNhbWVWYWx1ZSjCq2tvwrssIMKra28tS1LCuykgdG8gYmUgdHJ1ZScKKyAg
c3RyaWN0IG1vZGU6ICdUZXN0MjYyRXJyb3I6IExvY2FsZSBrbyBpcyBhZmZlY3RlZCBieSBrZXkg
Y287IHZhbHVlIHN0YW5kYXJkLiBFeHBlY3RlZCBTYW1lVmFsdWUowqtrb8K7LCDCq2tvLUtSwrsp
IHRvIGJlIHRydWUnCiB0ZXN0L2ludGw0MDIvQ29sbGF0b3IvbWlzc2luZy11bmljb2RlLWV4dC12
YWx1ZS1kZWZhdWx0cy10by10cnVlLmpzOgotICBkZWZhdWx0OiAiVGVzdDI2MkVycm9yOiBcImtu
LXRydWVcIiBpcyByZXR1cm5lZCBpbiBsb2NhbGUsIGJ1dCBzaG91bGRuJ3QgYmUuIEV4cGVjdGVk
IFNhbWVWYWx1ZSjCqzfCuywgwqstMcK7KSB0byBiZSB0cnVlIgotICBzdHJpY3QgbW9kZTogIlRl
c3QyNjJFcnJvcjogXCJrbi10cnVlXCIgaXMgcmV0dXJuZWQgaW4gbG9jYWxlLCBidXQgc2hvdWxk
bid0IGJlLiBFeHBlY3RlZCBTYW1lVmFsdWUowqs3wrssIMKrLTHCuykgdG8gYmUgdHJ1ZSIKKyAg
ZGVmYXVsdDogIlRlc3QyNjJFcnJvcjogXCJrbi10cnVlXCIgaXMgcmV0dXJuZWQgaW4gbG9jYWxl
LCBidXQgc2hvdWxkbid0IGJlLiBFeHBlY3RlZCBTYW1lVmFsdWUowqs0wrssIMKrLTHCuykgdG8g
YmUgdHJ1ZSIKKyAgc3RyaWN0IG1vZGU6ICJUZXN0MjYyRXJyb3I6IFwia24tdHJ1ZVwiIGlzIHJl
dHVybmVkIGluIGxvY2FsZSwgYnV0IHNob3VsZG4ndCBiZS4gRXhwZWN0ZWQgU2FtZVZhbHVlKMKr
NMK7LCDCqy0xwrspIHRvIGJlIHRydWUiCiB0ZXN0L2ludGw0MDIvRGF0ZVRpbWVGb3JtYXQvY29u
c3RydWN0b3Itb3B0aW9ucy1vcmRlci1mcmFjdGlvbmFsU2Vjb25kRGlnaXRzLmpzOgogICBkZWZh
dWx0OiAnVGVzdDI2MkVycm9yOiBFeHBlY3RlZCBbc2Vjb25kLCBmcmFjdGlvbmFsU2Vjb25kRGln
aXRzLCBsb2NhbGVNYXRjaGVyLCBzZWNvbmQsIGZyYWN0aW9uYWxTZWNvbmREaWdpdHMsIHRpbWVa
b25lTmFtZSwgZm9ybWF0TWF0Y2hlcl0gYW5kIFtzZWNvbmQsIGZyYWN0aW9uYWxTZWNvbmREaWdp
dHMsIGxvY2FsZU1hdGNoZXIsIHNlY29uZCwgdGltZVpvbmVOYW1lLCBmcmFjdGlvbmFsU2Vjb25k
RGlnaXRzLCBmb3JtYXRNYXRjaGVyXSB0byBoYXZlIHRoZSBzYW1lIGNvbnRlbnRzLiAnCiAgIHN0
cmljdCBtb2RlOiAnVGVzdDI2MkVycm9yOiBFeHBlY3RlZCBbc2Vjb25kLCBmcmFjdGlvbmFsU2Vj
b25kRGlnaXRzLCBsb2NhbGVNYXRjaGVyLCBzZWNvbmQsIGZyYWN0aW9uYWxTZWNvbmREaWdpdHMs
IHRpbWVab25lTmFtZSwgZm9ybWF0TWF0Y2hlcl0gYW5kIFtzZWNvbmQsIGZyYWN0aW9uYWxTZWNv
bmREaWdpdHMsIGxvY2FsZU1hdGNoZXIsIHNlY29uZCwgdGltZVpvbmVOYW1lLCBmcmFjdGlvbmFs
U2Vjb25kRGlnaXRzLCBmb3JtYXRNYXRjaGVyXSB0byBoYXZlIHRoZSBzYW1lIGNvbnRlbnRzLiAn
Cg==
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>409229</attachid>
            <date>2020-09-20 02:57:39 -0700</date>
            <delta_ts>2020-09-20 03:10:29 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-216437-20200920185738.patch</filename>
            <type>text/plain</type>
            <size>3126</size>
            <attacher name="HyeockJin Kim">kherootz</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjY2MzIyCmRpZmYgLS1naXQgYS9Tb3VyY2UvSmF2YVNjcmlw
dENvcmUvQ2hhbmdlTG9nIGIvU291cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwppbmRleCBk
NDM3MDhiZjZmNWZjYmRjNDczOWM4OTdjZmZmZDFmMDA4MGJmNWVmLi43ODA2MDQ4MzZmNTdiNmY0
ZTkyOWM5YzczZDJkMjE0ZTE1MTkzMmNkIDEwMDY0NAotLS0gYS9Tb3VyY2UvSmF2YVNjcmlwdENv
cmUvQ2hhbmdlTG9nCisrKyBiL1NvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKQEAgLTEs
MyArMSwxMyBAQAorMjAyMC0wOS0yMCAgSHllb2NrSmluIEtpbSAgPGtoZXJvb3R6QGdtYWlsLmNv
bT4KKworICAgICAgICBDb2VyY2UgY29tcHV0ZWQgcHJvcGVydHkgYmVmb3JlIGFkZGluZyB0byB8
ZXhjbHVkZWRMaXN0fAorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5j
Z2k/aWQ9MjE2NDM3CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAg
ICAgICAgKiBieXRlY29tcGlsZXIvTm9kZXNDb2RlZ2VuLmNwcDoKKyAgICAgICAgKEpTQzo6T2Jq
ZWN0UGF0dGVybk5vZGU6OmJpbmRWYWx1ZSBjb25zdCk6CisKIDIwMjAtMDgtMjggIFl1c3VrZSBT
dXp1a2kgIDx5c3V6dWtpQGFwcGxlLmNvbT4KIAogICAgICAgICBbSlNDXSBzdXBlciBwcm9wZXJ0
eSB3aXRoIG5ldyBzaG91bGQgYmUgYWNjZXB0ZWQKZGlmZiAtLWdpdCBhL1NvdXJjZS9KYXZhU2Ny
aXB0Q29yZS9ieXRlY29tcGlsZXIvTm9kZXNDb2RlZ2VuLmNwcCBiL1NvdXJjZS9KYXZhU2NyaXB0
Q29yZS9ieXRlY29tcGlsZXIvTm9kZXNDb2RlZ2VuLmNwcAppbmRleCA5M2Y2MmNkZTZjNzBkM2Y2
Yjk5NGNmOWRiNGRkMzFkNGU3MjRlZTgyLi4zM2Q3ZWI2NmQ0NTBhNDhkYjJmM2RjZWY4MjBiOTkx
Njc3YzM0MzdiIDEwMDY0NAotLS0gYS9Tb3VyY2UvSmF2YVNjcmlwdENvcmUvYnl0ZWNvbXBpbGVy
L05vZGVzQ29kZWdlbi5jcHAKKysrIGIvU291cmNlL0phdmFTY3JpcHRDb3JlL2J5dGVjb21waWxl
ci9Ob2Rlc0NvZGVnZW4uY3BwCkBAIC01MjMyLDYgKzUyMzIsOCBAQCB2b2lkIE9iamVjdFBhdHRl
cm5Ob2RlOjpiaW5kVmFsdWUoQnl0ZWNvZGVHZW5lcmF0b3ImIGdlbmVyYXRvciwgUmVnaXN0ZXJJ
RCogcmhzKQogICAgICAgICAgICAgICAgIGlmIChtX2NvbnRhaW5zQ29tcHV0ZWRQcm9wZXJ0eSkg
ewogICAgICAgICAgICAgICAgICAgICBpZiAoIXRhcmdldC5wcm9wZXJ0eUV4cHJlc3Npb24pCiAg
ICAgICAgICAgICAgICAgICAgICAgICBwcm9wZXJ0eU5hbWUgPSBnZW5lcmF0b3IuZW1pdExvYWQo
bnVsbHB0ciwgdGFyZ2V0LnByb3BlcnR5TmFtZSk7CisgICAgICAgICAgICAgICAgICAgIGVsc2UK
KyAgICAgICAgICAgICAgICAgICAgICAgIGdlbmVyYXRvci5lbWl0VG9Qcm9wZXJ0eUtleShwcm9w
ZXJ0eU5hbWUuZ2V0KCksIHByb3BlcnR5TmFtZS5nZXQoKSk7CiAKICAgICAgICAgICAgICAgICAg
ICAgQ2FsbEFyZ3VtZW50cyBhcmdzKGdlbmVyYXRvciwgbnVsbHB0ciwgMSk7CiAgICAgICAgICAg
ICAgICAgICAgIGdlbmVyYXRvci5tb3ZlKGFyZ3MudGhpc1JlZ2lzdGVyKCksIGV4Y2x1ZGVkTGlz
dC5nZXQoKSk7CmRpZmYgLS1naXQgYS9KU1Rlc3RzL0NoYW5nZUxvZyBiL0pTVGVzdHMvQ2hhbmdl
TG9nCmluZGV4IDhkYjUzN2ZjYzQ1OTE1NTE3MTIzNmViNjIzMWE2NWU5YzY5NGE3NzkuLjZmZjA2
OGVjYjUxNjM4Nzg0NmIwYzRiMTU1NjdkNmI1ZjU5ZTI5MmQgMTAwNjQ0Ci0tLSBhL0pTVGVzdHMv
Q2hhbmdlTG9nCisrKyBiL0pTVGVzdHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTMgQEAKKzIwMjAt
MDktMjAgIEh5ZW9ja0ppbiBLaW0gIDxraGVyb290ekBnbWFpbC5jb20+CisKKyAgICAgICAgQ29l
cmNlIGNvbXB1dGVkIHByb3BlcnR5IGJlZm9yZSBhZGRpbmcgdG8gfGV4Y2x1ZGVkTGlzdHwKKyAg
ICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTIxNjQzNworCisg
ICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgICogc3RyZXNzL29i
amVjdC1yZXN0LWRlY29uc3RydWN0LmpzOgorICAgICAgICAoZ2V0IDMpOgorCiAyMDIwLTA4LTI4
ICBZdXN1a2UgU3V6dWtpICA8eXN1enVraUBhcHBsZS5jb20+CiAKICAgICAgICAgW0pTQ10gc3Vw
ZXIgcHJvcGVydHkgd2l0aCBuZXcgc2hvdWxkIGJlIGFjY2VwdGVkCmRpZmYgLS1naXQgYS9KU1Rl
c3RzL3N0cmVzcy9vYmplY3QtcmVzdC1kZWNvbnN0cnVjdC5qcyBiL0pTVGVzdHMvc3RyZXNzL29i
amVjdC1yZXN0LWRlY29uc3RydWN0LmpzCmluZGV4IGM5ODRlNzE0MGI1Njc1MWFhNmJjNDZlMTgy
Zjg1MGU3NDcxMTRiMGEuLjk0MzNkMjlhNzAyZTJiNDAzODZlZDYzN2Y3Y2M4OGE2YjU1NzJmMjAg
MTAwNjQ0Ci0tLSBhL0pTVGVzdHMvc3RyZXNzL29iamVjdC1yZXN0LWRlY29uc3RydWN0LmpzCisr
KyBiL0pTVGVzdHMvc3RyZXNzL29iamVjdC1yZXN0LWRlY29uc3RydWN0LmpzCkBAIC0yMDIsNiAr
MjAyLDI3IEBAIGxldCBhc3NlcnRQcm9wRGVzY3JpcHRvciA9IChyZXN0T2JqLCBwcm9wKSA9PiB7
CiAgICAgYXNzZXJ0KHIuYmF6ID09PSAzKTsKIH0pKCk7CiAKKy8vIERlc3RydWN0dXJpbmcgbm9u
LXN0cmluZyBjb21wdXRlZCBwcm9wZXJ0eQorKCgpID0+IHsKKyAgICB2YXIgYSA9IDE7CisgICAg
dmFyIHtbYV06IGIsIC4uLnJ9ID0ge1thXTogMSwgYjogMiwgYzogM307CisgICAgYXNzZXJ0KGIg
PT09IDEpOworICAgIGFzc2VydChyWzFdID09PSB1bmRlZmluZWQpOworICAgIGFzc2VydChyLmIg
PT09IDIpOworICAgIGFzc2VydChyLmMgPT09IDMpOworfSkoKTsKKworLy8gRGVzdHJ1Y3R1cmlu
ZyBTeW1ib2wgY29tcHV0ZWQgcHJvcGVydHkKKygoKSA9PiB7CisgICAgdmFyIGEgPSBTeW1ib2wo
J2EnKTsKKyAgICB2YXIgYiA9IFN5bWJvbCgnYScpOworICAgIHZhciB7W2FdOiBjLCAuLi5yfSA9
IHtbYl06IDEsIGI6IDIsIGM6IDN9OworICAgIGFzc2VydChjID09PSB1bmRlZmluZWQpOworICAg
IGFzc2VydChyW2JdID09PSAxKTsKKyAgICBhc3NlcnQoci5iID09PSAyKTsKKyAgICBhc3NlcnQo
ci5jID09PSAzKTsKK30pKCk7CisKIC8vIENhdGNoIGNhc2UKIAogKCgpID0+IHsK
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>409230</attachid>
            <date>2020-09-20 03:10:31 -0700</date>
            <delta_ts>2020-09-22 15:23:14 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-216437-20200920191030.patch</filename>
            <type>text/plain</type>
            <size>3202</size>
            <attacher name="HyeockJin Kim">kherootz</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjY3MzI0CmRpZmYgLS1naXQgYS9Tb3VyY2UvSmF2YVNjcmlw
dENvcmUvQ2hhbmdlTG9nIGIvU291cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwppbmRleCAy
OGZjZmMzMTU4YTljMmFmMTZkNDBmMmMyYTE3ZjQ2OGNlOTVkNDk1Li5lMjBmMDAxZWQ2N2Y5NDQ3
ZmExNjA3NDkwNWNiMTNmZmIzOGEzZjc3IDEwMDY0NAotLS0gYS9Tb3VyY2UvSmF2YVNjcmlwdENv
cmUvQ2hhbmdlTG9nCisrKyBiL1NvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKQEAgLTEs
MyArMSwxMyBAQAorMjAyMC0wOS0yMCAgSHllb2NrSmluIEtpbSAgPGtoZXJvb3R6QGdtYWlsLmNv
bT4KKworICAgICAgICBDb2VyY2UgY29tcHV0ZWQgcHJvcGVydHkgYmVmb3JlIGFkZGluZyB0byB8
ZXhjbHVkZWRMaXN0fAorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5j
Z2k/aWQ9MjE2NDM3CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAg
ICAgICAgKiBieXRlY29tcGlsZXIvTm9kZXNDb2RlZ2VuLmNwcDoKKyAgICAgICAgKEpTQzo6T2Jq
ZWN0UGF0dGVybk5vZGU6OmJpbmRWYWx1ZSBjb25zdCk6CisKIDIwMjAtMDktMjAgIFl1c3VrZSBT
dXp1a2kgIDx5c3V6dWtpQGFwcGxlLmNvbT4KIAogICAgICAgICBVbnJldmlld2VkLCB1c2UgUkVM
RUFTRV9BTkRfUkVUVVJOIHRvIHN1cHByZXNzIGV4Y2VwdGlvbiB2ZXJpZmljYXRpb24gZmFpbHVy
ZQpkaWZmIC0tZ2l0IGEvU291cmNlL0phdmFTY3JpcHRDb3JlL2J5dGVjb21waWxlci9Ob2Rlc0Nv
ZGVnZW4uY3BwIGIvU291cmNlL0phdmFTY3JpcHRDb3JlL2J5dGVjb21waWxlci9Ob2Rlc0NvZGVn
ZW4uY3BwCmluZGV4IGI3ODM5MmM4ZWU4YjQwMTQwZDNhZWJhMDRiNzY3NmQxM2NlNWNhOTYuLjIy
YTU1NWRhMDQ4NDFlMTcwZWRmNzM5NjFmYTVlNzZjOWI5MjNlN2UgMTAwNjQ0Ci0tLSBhL1NvdXJj
ZS9KYXZhU2NyaXB0Q29yZS9ieXRlY29tcGlsZXIvTm9kZXNDb2RlZ2VuLmNwcAorKysgYi9Tb3Vy
Y2UvSmF2YVNjcmlwdENvcmUvYnl0ZWNvbXBpbGVyL05vZGVzQ29kZWdlbi5jcHAKQEAgLTUyMzMs
NyArNTIzMyw5IEBAIHZvaWQgT2JqZWN0UGF0dGVybk5vZGU6OmJpbmRWYWx1ZShCeXRlY29kZUdl
bmVyYXRvciYgZ2VuZXJhdG9yLCBSZWdpc3RlcklEKiByaHMpCiAKICAgICAgICAgICAgIGlmICht
X2NvbnRhaW5zUmVzdEVsZW1lbnQpIHsKICAgICAgICAgICAgICAgICBpZiAobV9jb250YWluc0Nv
bXB1dGVkUHJvcGVydHkpIHsKLSAgICAgICAgICAgICAgICAgICAgaWYgKCF0YXJnZXQucHJvcGVy
dHlFeHByZXNzaW9uKQorICAgICAgICAgICAgICAgICAgICBpZiAodGFyZ2V0LnByb3BlcnR5RXhw
cmVzc2lvbikKKyAgICAgICAgICAgICAgICAgICAgICAgIGdlbmVyYXRvci5lbWl0VG9Qcm9wZXJ0
eUtleShwcm9wZXJ0eU5hbWUuZ2V0KCksIHByb3BlcnR5TmFtZS5nZXQoKSk7CisgICAgICAgICAg
ICAgICAgICAgIGVsc2UKICAgICAgICAgICAgICAgICAgICAgICAgIHByb3BlcnR5TmFtZSA9IGdl
bmVyYXRvci5lbWl0TG9hZChudWxscHRyLCB0YXJnZXQucHJvcGVydHlOYW1lKTsKIAogICAgICAg
ICAgICAgICAgICAgICBDYWxsQXJndW1lbnRzIGFyZ3MoZ2VuZXJhdG9yLCBudWxscHRyLCAxKTsK
ZGlmZiAtLWdpdCBhL0pTVGVzdHMvQ2hhbmdlTG9nIGIvSlNUZXN0cy9DaGFuZ2VMb2cKaW5kZXgg
MzljM2YzZGY0YjVkZTVkYWMzYzc2MjQ5NDM5ZDg5NDc0MTlhYTVhNC4uMzRmY2EzMWE1Nzc3ZDA2
ODc1OTdhY2Y2NTUxMTIwZmU1NjNkYTk2NiAxMDA2NDQKLS0tIGEvSlNUZXN0cy9DaGFuZ2VMb2cK
KysrIGIvSlNUZXN0cy9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxMyBAQAorMjAyMC0wOS0yMCAgSHll
b2NrSmluIEtpbSAgPGtoZXJvb3R6QGdtYWlsLmNvbT4KKworICAgICAgICBDb2VyY2UgY29tcHV0
ZWQgcHJvcGVydHkgYmVmb3JlIGFkZGluZyB0byB8ZXhjbHVkZWRMaXN0fAorICAgICAgICBodHRw
czovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MjE2NDM3CisKKyAgICAgICAgUmV2
aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgKiBzdHJlc3Mvb2JqZWN0LXJlc3Qt
ZGVjb25zdHJ1Y3QuanM6CisgICAgICAgIChnZXQgMyk6CisKIDIwMjAtMDktMTggIFl1c3VrZSBT
dXp1a2kgIDx5c3V6dWtpQGFwcGxlLmNvbT4KIAogICAgICAgICBbSlNDXSBHZW5lcmF0b3IgZGVj
bGFyYXRpb24gc2hvdWxkIG5vdCBiZSBhbGxvd2VkIGluIHNpbmdsZSBzdGF0ZW1lbnQgY29udGV4
dApkaWZmIC0tZ2l0IGEvSlNUZXN0cy9zdHJlc3Mvb2JqZWN0LXJlc3QtZGVjb25zdHJ1Y3QuanMg
Yi9KU1Rlc3RzL3N0cmVzcy9vYmplY3QtcmVzdC1kZWNvbnN0cnVjdC5qcwppbmRleCBjOTg0ZTcx
NDBiNTY3NTFhYTZiYzQ2ZTE4MmY4NTBlNzQ3MTE0YjBhLi45NDMzZDI5YTcwMmUyYjQwMzg2ZWQ2
MzdmN2NjODhhNmI1NTcyZjIwIDEwMDY0NAotLS0gYS9KU1Rlc3RzL3N0cmVzcy9vYmplY3QtcmVz
dC1kZWNvbnN0cnVjdC5qcworKysgYi9KU1Rlc3RzL3N0cmVzcy9vYmplY3QtcmVzdC1kZWNvbnN0
cnVjdC5qcwpAQCAtMjAyLDYgKzIwMiwyNyBAQCBsZXQgYXNzZXJ0UHJvcERlc2NyaXB0b3IgPSAo
cmVzdE9iaiwgcHJvcCkgPT4gewogICAgIGFzc2VydChyLmJheiA9PT0gMyk7CiB9KSgpOwogCisv
LyBEZXN0cnVjdHVyaW5nIG5vbi1zdHJpbmcgY29tcHV0ZWQgcHJvcGVydHkKKygoKSA9PiB7Cisg
ICAgdmFyIGEgPSAxOworICAgIHZhciB7W2FdOiBiLCAuLi5yfSA9IHtbYV06IDEsIGI6IDIsIGM6
IDN9OworICAgIGFzc2VydChiID09PSAxKTsKKyAgICBhc3NlcnQoclsxXSA9PT0gdW5kZWZpbmVk
KTsKKyAgICBhc3NlcnQoci5iID09PSAyKTsKKyAgICBhc3NlcnQoci5jID09PSAzKTsKK30pKCk7
CisKKy8vIERlc3RydWN0dXJpbmcgU3ltYm9sIGNvbXB1dGVkIHByb3BlcnR5CisoKCkgPT4gewor
ICAgIHZhciBhID0gU3ltYm9sKCdhJyk7CisgICAgdmFyIGIgPSBTeW1ib2woJ2EnKTsKKyAgICB2
YXIge1thXTogYywgLi4ucn0gPSB7W2JdOiAxLCBiOiAyLCBjOiAzfTsKKyAgICBhc3NlcnQoYyA9
PT0gdW5kZWZpbmVkKTsKKyAgICBhc3NlcnQocltiXSA9PT0gMSk7CisgICAgYXNzZXJ0KHIuYiA9
PT0gMik7CisgICAgYXNzZXJ0KHIuYyA9PT0gMyk7Cit9KSgpOworCiAvLyBDYXRjaCBjYXNlCiAK
ICgoKSA9PiB7Cg==
</data>

          </attachment>
      

    </bug>

</bugzilla>