<?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>93214</bug_id>
          
          <creation_ts>2012-08-05 18:19:26 -0700</creation_ts>
          <short_desc>[EFL][DRT] http/tests/loading/307-after-303-after-post.html times out</short_desc>
          <delta_ts>2013-02-19 10:29:58 -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>WebKit EFL</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>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Joone Hur">joone</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>cdumez</cc>
    
    <cc>danw</cc>
    
    <cc>gustavo</cc>
    
    <cc>gyuyoung.kim</cc>
    
    <cc>laszlo.gombos</cc>
    
    <cc>lucas.de.marchi</cc>
    
    <cc>mrobinson</cc>
    
    <cc>rakuco</cc>
    
    <cc>svillar</cc>
    
    <cc>webkit.review.bot</cc>
    
    <cc>youennf</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>687691</commentid>
    <comment_count>0</comment_count>
    <who name="Joone Hur">joone</who>
    <bug_when>2012-08-05 18:19:26 -0700</bug_when>
    <thetext>http/tests/loading/307-after-303-after-post.html times out on EFL64bit-debug-WK1</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>727474</commentid>
    <comment_count>1</comment_count>
    <who name="Raphael Kubo da Costa (:rakuco)">rakuco</who>
    <bug_when>2012-09-25 03:42:34 -0700</bug_when>
    <thetext>For the record: as of r129481, it is failing on the WK2 bot with [ Failure Timeout Pass ].</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>825241</commentid>
    <comment_count>2</comment_count>
      <attachid>186569</attachid>
    <who name="youenn fablet">youennf</who>
    <bug_when>2013-02-05 00:57:37 -0800</bug_when>
    <thetext>Created attachment 186569
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>825448</commentid>
    <comment_count>3</comment_count>
      <attachid>186569</attachid>
    <who name="Dan Winship">danw</who>
    <bug_when>2013-02-05 06:45:43 -0800</bug_when>
    <thetext>Comment on attachment 186569
Patch

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

&gt; Source/WebCore/platform/network/soup/ResourceHandleSoup.cpp:425
&gt;      if (message-&gt;method == SOUP_METHOD_GET)
&gt; -        return false;
&gt; +        return request.httpMethod() != &quot;GET&quot;;

Why are message-&gt;method and request.httpMethod() different?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>828557</commentid>
    <comment_count>4</comment_count>
    <who name="youenn fablet">youennf</who>
    <bug_when>2013-02-08 07:31:56 -0800</bug_when>
    <thetext>request-&gt;httpMethod() is set from POST to GET after the 303 redirection in doRedirect.
But handle-&gt;firstRequest().httpMethod() is left unchanged.
On the next call to doRedirect, request-&gt;httpMethod() will be again set to POST.

message and message-&gt;method are persistent over the redirection response.
After the 303 redirection, message-&gt;method is set to GET and its value will remain as is.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>828659</commentid>
    <comment_count>5</comment_count>
    <who name="Dan Winship">danw</who>
    <bug_when>2013-02-08 10:12:46 -0800</bug_when>
    <thetext>ah... kov and I were talking about this at one point... There&apos;s a lot of code that&apos;s using firstRequest when it really means &quot;mostRecentRequest&quot;. I tried fixing this at one point but didn&apos;t get very far.

But anyway, that&apos;s the problem here; the logic in shouldRedirectAsGET() is correct, it&apos;s just that &quot;request&quot; was initialized with the wrong method. So I think the right fix is to add

    request.setHTTPMethod(message-&gt;method);

right after the request.setURL() line.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>828684</commentid>
    <comment_count>6</comment_count>
    <who name="Dan Winship">danw</who>
    <bug_when>2013-02-08 11:07:04 -0800</bug_when>
    <thetext>ok, actually that won&apos;t work because firstRequest will still have the request body too...

I guess your patch works, but the underlying problem is that we&apos;re using firstRequest to initialize the new request, and we really should be using the request that we sent last time instead.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>828698</commentid>
    <comment_count>7</comment_count>
    <who name="Martin Robinson">mrobinson</who>
    <bug_when>2013-02-08 11:35:35 -0800</bug_when>
    <thetext>(In reply to comment #6)
&gt; ok, actually that won&apos;t work because firstRequest will still have the request body too...
&gt; 
&gt; I guess your patch works, but the underlying problem is that we&apos;re using firstRequest to initialize the new request, and we really should be using the request that we sent last time instead.

Could we fix this problem by simply storing that request?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>829304</commentid>
    <comment_count>8</comment_count>
    <who name="youenn fablet">youennf</who>
    <bug_when>2013-02-10 09:51:36 -0800</bug_when>
    <thetext>Ah, I thought keeping firstRequest as is was part of the design.
I will look at this again with that in mind.
Also, would it make sense to try renaming &quot;firstRequest&quot; in the source code?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>829317</commentid>
    <comment_count>9</comment_count>
    <who name="Dan Winship">danw</who>
    <bug_when>2013-02-10 10:51:39 -0800</bug_when>
    <thetext>firstRequest is part of the platform-independent part of ResourceHandle, and there are places where it&apos;s used where they really do want the first request. (I think...)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>829766</commentid>
    <comment_count>10</comment_count>
      <attachid>187594</attachid>
    <who name="youenn fablet">youennf</who>
    <bug_when>2013-02-11 09:06:39 -0800</bug_when>
    <thetext>Created attachment 187594
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>829769</commentid>
    <comment_count>11</comment_count>
    <who name="youenn fablet">youennf</who>
    <bug_when>2013-02-11 09:09:26 -0800</bug_when>
    <thetext>After reading the code again, I would tend to stick with the current approach.
Storing a modified ResourceRequest is deviating from other port strategies.
It may also take more work to keep in sync things (referer, auth..) between the hops than go from firstRequest to the latest hop.

In the newly submitted patch, shouldRedirectAsGET is not modified.
The check is directly added in doRedirect.
The check is a bit looser than in the previous patch but I hope it is fine as is.
I also renamed request in newRequest, in the hope it makes the code clearer.

In the same vein, there is some code that looks a bit suspicious in resourceHandleSoup.cpp/handleUnignoredTLSErrors.
The host name that is used to retrieve certificates (that are not CA validated but granted by user) is coming from firstRequest.
To properly handle redirections, it may be better to get it directly from the soup layer.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>829820</commentid>
    <comment_count>12</comment_count>
      <attachid>187594</attachid>
    <who name="Martin Robinson">mrobinson</who>
    <bug_when>2013-02-11 10:01:56 -0800</bug_when>
    <thetext>Comment on attachment 187594
Patch

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

Perhaps it makes sense to unskip this test on GTK+ as well?

&gt; Source/WebCore/ChangeLog:5
&gt; +        [EFL][DRT] Ensured that GET verb is used for requests
&gt; +        triggered from a succession of 307 and other 3XX responses.
&gt; +        https://bugs.webkit.org/show_bug.cgi?id=93214

The first line should be the bug title and then the description should go underneath.

&gt; Source/WebCore/platform/network/soup/ResourceHandleSoup.cpp:457
&gt; -    ResourceRequest request = handle-&gt;firstRequest();
&gt; +    ResourceRequest newRequest = handle-&gt;firstRequest();

I like this change a lot!

&gt; Source/WebCore/platform/network/soup/ResourceHandleSoup.cpp:468
&gt; +    if (shouldRedirectAsGET(message, newURL, crossOrigin) || newRequest.httpMethod() != message-&gt;method) {
&gt; +        newRequest.setHTTPMethod(&quot;GET&quot;);
&gt; +        newRequest.setHTTPBody(0);
&gt; +        newRequest.clearHTTPContentType();
&gt;      }

Hrm. Shouldn&apos;t this restore the original method instead of just unconditionally using GET? In any case, why not roll this into shouldRedirectAsGET?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>829865</commentid>
    <comment_count>13</comment_count>
    <who name="youenn fablet">youennf</who>
    <bug_when>2013-02-11 10:37:15 -0800</bug_when>
    <thetext>&gt; Perhaps it makes sense to unskip this test on GTK+ as well?


On GTK+, there is a bug in the DRT that prevents printing the expected output 
(https://bugs.webkit.org/show_bug.cgi?id=35300)
Once this bug is solved, it should be fine to unskip it.

&gt; The first line should be the bug title and then the description should go underneath.

I will change this when reposting the patch.


&gt; Hrm. Shouldn&apos;t this restore the original method instead of just unconditionally using GET? In any case, why not roll this into shouldRedirectAsGET?

It&apos;s right that the check is a bit loose.
In theory, we should check equality of the methods only if the soup request is a GET. But in practice for HTTP, the methods will be different only if a redirect enforces the verb to go from value to GET.
Should it be tightened?

Concerning shouldRedirectAsGET, the test was included in the previous patch.
I received the feedback that it would be better to decouple the pure http logic (implemented in shouldRedirectAsGET) from this test which is related to the implementation strategy (always start from the original request).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>830897</commentid>
    <comment_count>14</comment_count>
    <who name="Dan Winship">danw</who>
    <bug_when>2013-02-12 06:44:24 -0800</bug_when>
    <thetext>*** Bug 109576 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>830932</commentid>
    <comment_count>15</comment_count>
    <who name="Sergio Villar Senin">svillar</who>
    <bug_when>2013-02-12 07:29:25 -0800</bug_when>
    <thetext>I worked on this same issue in bug 109576. I replaced the implementation of shouldRedirecAsGET() by a more simpler version, basically

+    if (((message-&gt;status_code == SOUP_STATUS_MOVED_PERMANENTLY)
+        || (message-&gt;status_code == SOUP_STATUS_FOUND)
+        || (message-&gt;status_code == SOUP_STATUS_SEE_OTHER))
+        &amp;&amp; (request.httpMethod() == &quot;POST&quot;))
         return true;

Do we really need all the checks we have right now?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>830949</commentid>
    <comment_count>16</comment_count>
    <who name="Dan Winship">danw</who>
    <bug_when>2013-02-12 07:57:20 -0800</bug_when>
    <thetext>(In reply to comment #15)
&gt; Do we really need all the checks we have right now?

Yes. WebKit&apos;s tests don&apos;t test all of the possibilities. Removing some of those cases would break web pages.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>833080</commentid>
    <comment_count>17</comment_count>
      <attachid>188352</attachid>
    <who name="youenn fablet">youennf</who>
    <bug_when>2013-02-14 07:55:23 -0800</bug_when>
    <thetext>Created attachment 188352
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>833086</commentid>
    <comment_count>18</comment_count>
    <who name="youenn fablet">youennf</who>
    <bug_when>2013-02-14 08:00:52 -0800</bug_when>
    <thetext>http://greenbytes.de/tech/tc/httpredirects/ and http://www.mnot.net/javascript/xmlhttprequest/#mr.HEAD.301 are good pointers for shouldRedirectAsGET logic.
I plan to work on adding some webkit tests based on that work. 
HEAD redirections could be improved for instance.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>833101</commentid>
    <comment_count>19</comment_count>
      <attachid>188352</attachid>
    <who name="Sergio Villar Senin">svillar</who>
    <bug_when>2013-02-14 08:30:35 -0800</bug_when>
    <thetext>Comment on attachment 188352
Patch

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

&gt; LayoutTests/platform/efl/TestExpectations:763
&gt; +webkit.org/b/93214 http/tests/loading/307-after-303-after-post.html [ Crash Pass Timeout ]

Remove it also from Gtk&apos;s TestExpectations</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>833759</commentid>
    <comment_count>20</comment_count>
    <who name="youenn fablet">youennf</who>
    <bug_when>2013-02-15 00:12:34 -0800</bug_when>
    <thetext>(In reply to comment #19)
&gt; (From update of attachment 188352 [details])
&gt; View in context: https://bugs.webkit.org/attachment.cgi?id=188352&amp;action=review
&gt; 
&gt; &gt; LayoutTests/platform/efl/TestExpectations:763
&gt; &gt; +webkit.org/b/93214 http/tests/loading/307-after-303-after-post.html [ Crash Pass Timeout ]
&gt; 
&gt; Remove it also from Gtk&apos;s TestExpectations

The test is failing with Gtk/wk1 (DRT is not printing the initial resource URL correctly, see also bug 35300).
For Gtk/wk2, the test is failing, but that may be my setup (additional favicon.ico lines are printed..).
Are you suggesting to create an expected test result file to pass the tests?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>835299</commentid>
    <comment_count>21</comment_count>
    <who name="Sergio Villar Senin">svillar</who>
    <bug_when>2013-02-18 03:19:03 -0800</bug_when>
    <thetext>(In reply to comment #20)
&gt; (In reply to comment #19)
&gt; &gt; (From update of attachment 188352 [details] [details])
&gt; &gt; View in context: https://bugs.webkit.org/attachment.cgi?id=188352&amp;action=review
&gt; &gt; 
&gt; &gt; &gt; LayoutTests/platform/efl/TestExpectations:763
&gt; &gt; &gt; +webkit.org/b/93214 http/tests/loading/307-after-303-after-post.html [ Crash Pass Timeout ]
&gt; &gt; 
&gt; &gt; Remove it also from Gtk&apos;s TestExpectations
&gt; 
&gt; The test is failing with Gtk/wk1 (DRT is not printing the initial resource URL correctly, see also bug 35300).
&gt; For Gtk/wk2, the test is failing, but that may be my setup (additional favicon.ico lines are printed..).
&gt; Are you suggesting to create an expected test result file to pass the tests?

Yeah you&apos;re right, let&apos;s forget about WK1. Let&apos;s try to land this ASAP</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>836523</commentid>
    <comment_count>22</comment_count>
      <attachid>188352</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2013-02-19 10:29:52 -0800</bug_when>
    <thetext>Comment on attachment 188352
Patch

Clearing flags on attachment: 188352

Committed r143352: &lt;http://trac.webkit.org/changeset/143352&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>836524</commentid>
    <comment_count>23</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2013-02-19 10:29:58 -0800</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>186569</attachid>
            <date>2013-02-05 00:57:37 -0800</date>
            <delta_ts>2013-02-11 09:06:31 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-93214-20130205095414.patch</filename>
            <type>text/plain</type>
            <size>3655</size>
            <attacher name="youenn fablet">youennf</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTQxODY3CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggOWNiYTk2M2JjODJmMjk0
ZDU1NTdhNmU2ZWZiYjEyZDBkMzczZjM1Zi4uODE5YjI5NjViNWQxMThmNTFjMjlmYjQ1MTAwZmQ2
YjQzZThiMWIzZCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE1IEBACisyMDEzLTAyLTA1ICBZb3Vl
bm4gRmFibGV0ICA8eW91ZW5uZkBnbWFpbC5jb20+CisKKyAgICAgICAgW0VGTF1bRFJUXSBFbnN1
cmVkIHRoYXQgR0VUIHZlcmIgaXMgdXNlZCBmb3IgcmVxdWVzdHMKKyAgICAgICAgdHJpZ2dlcmVk
IGZyb20gYSBzdWNjZXNzaW9uIG9mIDMwNyBhbmQgb3RoZXIgM1hYIHJlc3BvbnNlcy4KKyAgICAg
ICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTkzMjE0CisKKyAgICAg
ICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgKiBwbGF0Zm9ybS9uZXR3
b3JrL3NvdXAvUmVzb3VyY2VIYW5kbGVTb3VwLmNwcDoKKyAgICAgICAgKFdlYkNvcmU6OnNob3Vs
ZFJlZGlyZWN0QXNHRVQpOgorICAgICAgICAoV2ViQ29yZTo6ZG9SZWRpcmVjdCk6CisKIDIwMTMt
MDItMDUgIFByYXZlZW4gSmFkaGF2ICA8cHJhdmVlbi5qQHNhbXN1bmcuY29tPgogCiAgICAgICAg
IEZsb2F0aW5nIHBvaW50IHByZWNpc2lvbiBlcnJvciBpbiBBdWRpb1Bhbm5lck5vZGUuCmRpZmYg
LS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9uZXR3b3JrL3NvdXAvUmVzb3VyY2VIYW5k
bGVTb3VwLmNwcCBiL1NvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL25ldHdvcmsvc291cC9SZXNvdXJj
ZUhhbmRsZVNvdXAuY3BwCmluZGV4IDFmNmFmM2NjMTQzYmRjMmYyOTY1YmIzNjE2NDZiMTZjMzkw
OWI5ZGIuLmJhMWE4YjdiZDllZDU4NWZjZjVlMmRmNDBjY2RlYWY4YzIwYzI3ZjggMTAwNjQ0Ci0t
LSBhL1NvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL25ldHdvcmsvc291cC9SZXNvdXJjZUhhbmRsZVNv
dXAuY3BwCisrKyBiL1NvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL25ldHdvcmsvc291cC9SZXNvdXJj
ZUhhbmRsZVNvdXAuY3BwCkBAIC00MTksMTAgKzQxOSwxMCBAQCBzdGF0aWMgYm9vbCBzaG91bGRS
ZWRpcmVjdChSZXNvdXJjZUhhbmRsZSogaGFuZGxlKQogICAgIHJldHVybiB0cnVlOwogfQogCi1z
dGF0aWMgYm9vbCBzaG91bGRSZWRpcmVjdEFzR0VUKFNvdXBNZXNzYWdlKiBtZXNzYWdlLCBLVVJM
JiBuZXdVUkwsIGJvb2wgY3Jvc3NPcmlnaW4pCitzdGF0aWMgYm9vbCBzaG91bGRSZWRpcmVjdEFz
R0VUKFJlc291cmNlUmVxdWVzdCAmcmVxdWVzdCwgU291cE1lc3NhZ2UqIG1lc3NhZ2UsIEtVUkwm
IG5ld1VSTCwgYm9vbCBjcm9zc09yaWdpbikKIHsKICAgICBpZiAobWVzc2FnZS0+bWV0aG9kID09
IFNPVVBfTUVUSE9EX0dFVCkKLSAgICAgICAgcmV0dXJuIGZhbHNlOworICAgICAgICByZXR1cm4g
cmVxdWVzdC5odHRwTWV0aG9kKCkgIT0gIkdFVCI7CiAKICAgICBpZiAoIW5ld1VSTC5wcm90b2Nv
bElzSW5IVFRQRmFtaWx5KCkpCiAgICAgICAgIHJldHVybiB0cnVlOwpAQCAtNDYxLDcgKzQ2MSw3
IEBAIHN0YXRpYyB2b2lkIGRvUmVkaXJlY3QoUmVzb3VyY2VIYW5kbGUqIGhhbmRsZSkKICAgICBi
b29sIGNyb3NzT3JpZ2luID0gIXByb3RvY29sSG9zdEFuZFBvcnRBcmVFcXVhbChyZXF1ZXN0LnVy
bCgpLCBuZXdVUkwpOwogICAgIHJlcXVlc3Quc2V0VVJMKG5ld1VSTCk7CiAKLSAgICBpZiAoc2hv
dWxkUmVkaXJlY3RBc0dFVChtZXNzYWdlLCBuZXdVUkwsIGNyb3NzT3JpZ2luKSkgeworICAgIGlm
IChzaG91bGRSZWRpcmVjdEFzR0VUKHJlcXVlc3QsIG1lc3NhZ2UsIG5ld1VSTCwgY3Jvc3NPcmln
aW4pKSB7CiAgICAgICAgIHJlcXVlc3Quc2V0SFRUUE1ldGhvZCgiR0VUIik7CiAgICAgICAgIHJl
cXVlc3Quc2V0SFRUUEJvZHkoMCk7CiAgICAgICAgIHJlcXVlc3QuY2xlYXJIVFRQQ29udGVudFR5
cGUoKTsKZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL0NoYW5nZUxvZyBiL0xheW91dFRlc3RzL0No
YW5nZUxvZwppbmRleCAzODVhOGY4Y2NmNDAxMmNlODczNDc3ZmNhZjZmYTQxOTI5YTgwMGVjLi40
NWE4OGU4ZmYwMjE4MDlhMDQwYjRmNDkxMDI1ZjllMTJjNWNhYTc0IDEwMDY0NAotLS0gYS9MYXlv
dXRUZXN0cy9DaGFuZ2VMb2cKKysrIGIvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEs
MTMgQEAKKzIwMTMtMDItMDUgIFlvdWVubiBGYWJsZXQgIDx5b3Vlbm5mQGdtYWlsLmNvbT4KKwor
ICAgICAgICBbRUZMXVtEUlRdIFJlbW92ZWQgRmFpbHVyZSBmcm9tIHBvdGVudGlhbCBvdXRjb21l
IG9mIAorICAgICAgICBodHRwL3Rlc3RzL2xvYWRpbmcvMzA3LWFmdGVyLTMwMy1hZnRlci1wb3N0
Lmh0bWwgdGVzdAorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/
aWQ9OTMyMTQKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAg
ICAqIHBsYXRmb3JtL2VmbC9UZXN0RXhwZWN0YXRpb25zOgorCiAyMDEzLTAyLTA1ICBQcmF2ZWVu
IEphZGhhdiAgPHByYXZlZW4uakBzYW1zdW5nLmNvbT4KIAogICAgICAgICBGbG9hdGluZyBwb2lu
dCBwcmVjaXNpb24gZXJyb3IgaW4gQXVkaW9QYW5uZXJOb2RlLgpkaWZmIC0tZ2l0IGEvTGF5b3V0
VGVzdHMvcGxhdGZvcm0vZWZsL1Rlc3RFeHBlY3RhdGlvbnMgYi9MYXlvdXRUZXN0cy9wbGF0Zm9y
bS9lZmwvVGVzdEV4cGVjdGF0aW9ucwppbmRleCAwYjRkNWExOTQxOWVmNjMzODFiM2YxNDk3ZGZm
NmI4MTEzNWI4YmNkLi4zYzI2ZGI0OGUwNzI4OTEyODFkZjI5N2NlNDI3YTZiOWRmZDBlMzc0IDEw
MDY0NAotLS0gYS9MYXlvdXRUZXN0cy9wbGF0Zm9ybS9lZmwvVGVzdEV4cGVjdGF0aW9ucworKysg
Yi9MYXlvdXRUZXN0cy9wbGF0Zm9ybS9lZmwvVGVzdEV4cGVjdGF0aW9ucwpAQCAtNzYxLDcgKzc2
MSw3IEBAIHdlYmtpdC5vcmcvYi83NzYzMSBmYXN0L211bHRpY29sL3NwYW4vc3Bhbi1hcy1pbW1l
ZGlhdGUtY2hpbGQtcHJvcGVydHktcmVtb3ZhbC5oCiB3ZWJraXQub3JnL2IvNzc2MzEgZmFzdC9t
dWx0aWNvbC9zcGFuL3NwYW4tYXMtaW1tZWRpYXRlLWNvbHVtbnMtY2hpbGQtcmVtb3ZhbC5odG1s
IFsgRmFpbHVyZSBQYXNzIF0KIAogd2Via2l0Lm9yZy9iLzkzMjEzIGZhc3QvdGV4dC9pbnRlcm5h
dGlvbmFsL3RleHQtc3BsaWNlZC1mb250Lmh0bWwgWyBGYWlsdXJlIFBhc3MgXQotd2Via2l0Lm9y
Zy9iLzkzMjE0IGh0dHAvdGVzdHMvbG9hZGluZy8zMDctYWZ0ZXItMzAzLWFmdGVyLXBvc3QuaHRt
bCBbIENyYXNoIEZhaWx1cmUgUGFzcyBUaW1lb3V0IF0KK3dlYmtpdC5vcmcvYi85MzIxNCBodHRw
L3Rlc3RzL2xvYWRpbmcvMzA3LWFmdGVyLTMwMy1hZnRlci1wb3N0Lmh0bWwgWyBDcmFzaCBQYXNz
IFRpbWVvdXQgXQogd2Via2l0Lm9yZy9iLzkzMjEyIGh0dHAvdGVzdHMvaW5jcmVtZW50YWwvc2xv
dy11dGY4LXRleHQucGwgWyBQYXNzIFRpbWVvdXQgXQogCiB3ZWJraXQub3JnL2IvODUxODAgbWVk
aWEvYXVkaW8tZGVsZXRlLXdoaWxlLXN0ZXAtYnV0dG9uLWNsaWNrZWQuaHRtbCBbIEZhaWx1cmUg
UGFzcyBdCg==
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>187594</attachid>
            <date>2013-02-11 09:06:39 -0800</date>
            <delta_ts>2013-02-14 07:55:16 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-93214-20130211180308.patch</filename>
            <type>text/plain</type>
            <size>4814</size>
            <attacher name="youenn fablet">youennf</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTQyNDYzCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggYjJjNjc0MWUxNTg5NmY5
NDAwMTJlYzVmMGNkMDNiNTIxY2MzMjlhYi4uNjQ1YWY0NDMxOWIwOGU0YzU5MWZjOWM4MmJlNzZm
YjBjMTg0YTVlZCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE0IEBACisyMDEzLTAyLTExICBZb3Vl
bm4gRmFibGV0ICA8eW91ZW5uZkBnbWFpbC5jb20+CisKKyAgICAgICAgW0VGTF1bRFJUXSBFbnN1
cmVkIHRoYXQgR0VUIHZlcmIgaXMgdXNlZCBmb3IgcmVxdWVzdHMKKyAgICAgICAgdHJpZ2dlcmVk
IGZyb20gYSBzdWNjZXNzaW9uIG9mIDMwNyBhbmQgb3RoZXIgM1hYIHJlc3BvbnNlcy4KKyAgICAg
ICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTkzMjE0CisKKyAgICAg
ICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgKiBwbGF0Zm9ybS9uZXR3
b3JrL3NvdXAvUmVzb3VyY2VIYW5kbGVTb3VwLmNwcDoKKyAgICAgICAgKFdlYkNvcmU6OmRvUmVk
aXJlY3QpOgorCiAyMDEzLTAyLTExICBaYW4gRG9iZXJzZWsgIDx6ZG9iZXJzZWtAaWdhbGlhLmNv
bT4KIAogICAgICAgICBbR1RLXVtDbGFuZ10gQnVpbGQgZXJyb3JzIGluIExvY2FsaXplZFN0cmlu
Z3NHdGsuY3BwCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9uZXR3b3JrL3Nv
dXAvUmVzb3VyY2VIYW5kbGVTb3VwLmNwcCBiL1NvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL25ldHdv
cmsvc291cC9SZXNvdXJjZUhhbmRsZVNvdXAuY3BwCmluZGV4IGYzODdhZjUxMjkxNzRlNGE0NDQ3
ZjBiODFkOTBmYzFmOWNkZmU5MmEuLjlhMjQ5NzQxYzgzMWY4MDBlMWZkZDhmNmVjYjI4MDk3ZDJk
OTZkMDIgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL25ldHdvcmsvc291cC9S
ZXNvdXJjZUhhbmRsZVNvdXAuY3BwCisrKyBiL1NvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL25ldHdv
cmsvc291cC9SZXNvdXJjZUhhbmRsZVNvdXAuY3BwCkBAIC00NTQsNDggKzQ1NCw0OCBAQCBzdGF0
aWMgdm9pZCBkb1JlZGlyZWN0KFJlc291cmNlSGFuZGxlKiBoYW5kbGUpCiAgICAgICAgIHJldHVy
bjsKICAgICB9CiAKLSAgICBSZXNvdXJjZVJlcXVlc3QgcmVxdWVzdCA9IGhhbmRsZS0+Zmlyc3RS
ZXF1ZXN0KCk7CisgICAgUmVzb3VyY2VSZXF1ZXN0IG5ld1JlcXVlc3QgPSBoYW5kbGUtPmZpcnN0
UmVxdWVzdCgpOwogICAgIFNvdXBNZXNzYWdlKiBtZXNzYWdlID0gZC0+bV9zb3VwTWVzc2FnZS5n
ZXQoKTsKICAgICBjb25zdCBjaGFyKiBsb2NhdGlvbiA9IHNvdXBfbWVzc2FnZV9oZWFkZXJzX2dl
dF9vbmUobWVzc2FnZS0+cmVzcG9uc2VfaGVhZGVycywgIkxvY2F0aW9uIik7CiAgICAgS1VSTCBu
ZXdVUkwgPSBLVVJMKHNvdXBVUklUb0tVUkwoc291cF9tZXNzYWdlX2dldF91cmkobWVzc2FnZSkp
LCBsb2NhdGlvbik7Ci0gICAgYm9vbCBjcm9zc09yaWdpbiA9ICFwcm90b2NvbEhvc3RBbmRQb3J0
QXJlRXF1YWwocmVxdWVzdC51cmwoKSwgbmV3VVJMKTsKLSAgICByZXF1ZXN0LnNldFVSTChuZXdV
UkwpOworICAgIGJvb2wgY3Jvc3NPcmlnaW4gPSAhcHJvdG9jb2xIb3N0QW5kUG9ydEFyZUVxdWFs
KG5ld1JlcXVlc3QudXJsKCksIG5ld1VSTCk7CisgICAgbmV3UmVxdWVzdC5zZXRVUkwobmV3VVJM
KTsKIAotICAgIGlmIChzaG91bGRSZWRpcmVjdEFzR0VUKG1lc3NhZ2UsIG5ld1VSTCwgY3Jvc3NP
cmlnaW4pKSB7Ci0gICAgICAgIHJlcXVlc3Quc2V0SFRUUE1ldGhvZCgiR0VUIik7Ci0gICAgICAg
IHJlcXVlc3Quc2V0SFRUUEJvZHkoMCk7Ci0gICAgICAgIHJlcXVlc3QuY2xlYXJIVFRQQ29udGVu
dFR5cGUoKTsKKyAgICBpZiAoc2hvdWxkUmVkaXJlY3RBc0dFVChtZXNzYWdlLCBuZXdVUkwsIGNy
b3NzT3JpZ2luKSB8fCBuZXdSZXF1ZXN0Lmh0dHBNZXRob2QoKSAhPSBtZXNzYWdlLT5tZXRob2Qp
IHsKKyAgICAgICAgbmV3UmVxdWVzdC5zZXRIVFRQTWV0aG9kKCJHRVQiKTsKKyAgICAgICAgbmV3
UmVxdWVzdC5zZXRIVFRQQm9keSgwKTsKKyAgICAgICAgbmV3UmVxdWVzdC5jbGVhckhUVFBDb250
ZW50VHlwZSgpOwogICAgIH0KIAogICAgIC8vIFNob3VsZCBub3Qgc2V0IFJlZmVyZXIgYWZ0ZXIg
YSByZWRpcmVjdCBmcm9tIGEgc2VjdXJlIHJlc291cmNlIHRvIG5vbi1zZWN1cmUgb25lLgotICAg
IGlmICghbmV3VVJMLnByb3RvY29sSXMoImh0dHBzIikgJiYgcHJvdG9jb2xJcyhyZXF1ZXN0Lmh0
dHBSZWZlcnJlcigpLCAiaHR0cHMiKSAmJiBoYW5kbGUtPmNvbnRleHQoKS0+c2hvdWxkQ2xlYXJS
ZWZlcnJlck9uSFRUUFNUb0hUVFBSZWRpcmVjdCgpKQotICAgICAgICByZXF1ZXN0LmNsZWFySFRU
UFJlZmVycmVyKCk7CisgICAgaWYgKCFuZXdVUkwucHJvdG9jb2xJcygiaHR0cHMiKSAmJiBwcm90
b2NvbElzKG5ld1JlcXVlc3QuaHR0cFJlZmVycmVyKCksICJodHRwcyIpICYmIGhhbmRsZS0+Y29u
dGV4dCgpLT5zaG91bGRDbGVhclJlZmVycmVyT25IVFRQU1RvSFRUUFJlZGlyZWN0KCkpCisgICAg
ICAgIG5ld1JlcXVlc3QuY2xlYXJIVFRQUmVmZXJyZXIoKTsKIAogICAgIGQtPm1fdXNlciA9IG5l
d1VSTC51c2VyKCk7CiAgICAgZC0+bV9wYXNzID0gbmV3VVJMLnBhc3MoKTsKLSAgICByZXF1ZXN0
LnJlbW92ZUNyZWRlbnRpYWxzKCk7CisgICAgbmV3UmVxdWVzdC5yZW1vdmVDcmVkZW50aWFscygp
OwogCiAgICAgaWYgKGNyb3NzT3JpZ2luKSB7CiAgICAgICAgIC8vIElmIHRoZSBuZXR3b3JrIGxh
eWVyIGNhcnJpZXMgb3ZlciBhdXRoZW50aWNhdGlvbiBoZWFkZXJzIGZyb20gdGhlIG9yaWdpbmFs
IHJlcXVlc3QKICAgICAgICAgLy8gaW4gYSBjcm9zcy1vcmlnaW4gcmVkaXJlY3QsIHdlIHdhbnQg
dG8gY2xlYXIgdGhvc2UgaGVhZGVycyBoZXJlLiAKLSAgICAgICAgcmVxdWVzdC5jbGVhckhUVFBB
dXRob3JpemF0aW9uKCk7CisgICAgICAgIG5ld1JlcXVlc3QuY2xlYXJIVFRQQXV0aG9yaXphdGlv
bigpOwogCiAgICAgICAgIC8vIFRPRE86IFdlIGFyZSBsb3NpbmcgYW55IHVzZXJuYW1lIGFuZCBw
YXNzd29yZCBzcGVjaWZpZWQgaW4gdGhlIHJlZGlyZWN0IFVSTCwgYXMgdGhpcyBpcyB0aGUgCiAg
ICAgICAgIC8vIHNhbWUgYmVoYXZpb3IgYXMgdGhlIENGTmV0IHBvcnQuIFdlIHNob3VsZCBpbnZl
c3RpZ2F0ZSBpZiB0aGlzIGlzIHJlYWxseSB3aGF0IHdlIHdhbnQuCiAgICAgfSBlbHNlCi0gICAg
ICAgIGFwcGx5QXV0aGVudGljYXRpb25Ub1JlcXVlc3QoaGFuZGxlLCByZXF1ZXN0LCB0cnVlKTsK
KyAgICAgICAgYXBwbHlBdXRoZW50aWNhdGlvblRvUmVxdWVzdChoYW5kbGUsIG5ld1JlcXVlc3Qs
IHRydWUpOwogCiAgICAgY2xlYW51cFNvdXBSZXF1ZXN0T3BlcmF0aW9uKGhhbmRsZSk7Ci0gICAg
aWYgKCFjcmVhdGVTb3VwUmVxdWVzdEFuZE1lc3NhZ2VGb3JIYW5kbGUoaGFuZGxlLCByZXF1ZXN0
LCB0cnVlKSkgeworICAgIGlmICghY3JlYXRlU291cFJlcXVlc3RBbmRNZXNzYWdlRm9ySGFuZGxl
KGhhbmRsZSwgbmV3UmVxdWVzdCwgdHJ1ZSkpIHsKICAgICAgICAgZC0+Y2xpZW50KCktPmNhbm5v
dFNob3dVUkwoaGFuZGxlKTsKICAgICAgICAgcmV0dXJuOwogICAgIH0KIAogICAgIC8vIElmIHdl
IHNlbnQgY3JlZGVudGlhbHMgd2l0aCB0aGlzIHJlcXVlc3QncyBVUkwsIHdlIGRvbid0IHdhbnQg
dGhlIHJlc3BvbnNlIHRvIGNhcnJ5IHRoZW0gdG8KICAgICAvLyB0aGUgV2ViS2l0IGxheWVyLiBU
aGV5IHdlcmUgb25seSBwbGFjZWQgaW4gdGhlIFVSTCBmb3IgdGhlIGJlbmVmaXQgb2YgbGlic291
cC4KLSAgICByZXF1ZXN0LnJlbW92ZUNyZWRlbnRpYWxzKCk7CisgICAgbmV3UmVxdWVzdC5yZW1v
dmVDcmVkZW50aWFscygpOwogCi0gICAgZC0+Y2xpZW50KCktPndpbGxTZW5kUmVxdWVzdChoYW5k
bGUsIHJlcXVlc3QsIGQtPm1fcmVzcG9uc2UpOworICAgIGQtPmNsaWVudCgpLT53aWxsU2VuZFJl
cXVlc3QoaGFuZGxlLCBuZXdSZXF1ZXN0LCBkLT5tX3Jlc3BvbnNlKTsKICAgICBoYW5kbGUtPnNl
bmRQZW5kaW5nUmVxdWVzdCgpOwogfQogCmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9DaGFuZ2VM
b2cgYi9MYXlvdXRUZXN0cy9DaGFuZ2VMb2cKaW5kZXggYjEyNTQ2ZDkxODI5MDI3NTdjYzViOWQy
YWNlNjA2NzA1ZGQxMzhlNC4uZWI5MWRjODNkMDEyNGM3ZjNiYTIzZGIwNjJmNWYwMmY3MjU3NWNh
NSAxMDA2NDQKLS0tIGEvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCisrKyBiL0xheW91dFRlc3RzL0No
YW5nZUxvZwpAQCAtMSwzICsxLDEzIEBACisyMDEzLTAyLTExICBZb3Vlbm4gRmFibGV0ICA8eW91
ZW5uZkBnbWFpbC5jb20+CisKKyAgICAgICAgW0VGTF1bRFJUXSBSZW1vdmVkIEZhaWx1cmUgZnJv
bSBwb3RlbnRpYWwgb3V0Y29tZSBvZgorICAgICAgICBodHRwL3Rlc3RzL2xvYWRpbmcvMzA3LWFm
dGVyLTMwMy1hZnRlci1wb3N0Lmh0bWwgdGVzdC4KKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtp
dC5vcmcvc2hvd19idWcuY2dpP2lkPTkzMjE0CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZ
IChPT1BTISkuCisKKyAgICAgICAgKiBwbGF0Zm9ybS9lZmwvVGVzdEV4cGVjdGF0aW9uczoKKwog
MjAxMy0wMi0xMSAgQWxleGVpIEZpbGlwcG92ICA8YWxwaEBjaHJvbWl1bS5vcmc+CiAKICAgICAg
ICAgV2ViIEluc3BlY3RvcjogU3BsaXQgUHJvZmlsZXIgZG9tYWluIGluIHByb3RvY29sIGludG8g
UHJvZmlsZXIgYW5kIEhlYXBQcm9maWxlcgo=
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>188352</attachid>
            <date>2013-02-14 07:55:23 -0800</date>
            <delta_ts>2013-02-19 10:29:52 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-93214-20130214163434.patch</filename>
            <type>text/plain</type>
            <size>6147</size>
            <attacher name="youenn fablet">youennf</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTQyODc1CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggZDEzZGNjMjYzMjg4YzA3
ZmE4YjkyZDMyMTNmMDRhNjZlZGJhNmNiMC4uZDFmYTkwNjI4MGFkMWI5ZDJkZjQwODcxZjUwZmM1
OTI1NTQyYTBkYiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE2IEBACisyMDEzLTAyLTE0ICBZb3Vl
bm4gRmFibGV0ICA8eW91ZW5uZkBnbWFpbC5jb20+CisKKyAgICAgICAgW0VGTF1bRFJUXSBodHRw
L3Rlc3RzL2xvYWRpbmcvMzA3LWFmdGVyLTMwMy1hZnRlci1wb3N0Lmh0bWwgdGltZXMgb3V0Cisg
ICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD05MzIxNAorCisg
ICAgICAgIEVuc3VyZWQgdGhhdCBHRVQgdmVyYiBpcyBjb25zaXN0ZW50bHkgdXNlZCBmb3IgYW55
IHJlcXVlc3QKKyAgICAgICAgY29taW5nIGFmdGVyIGEgcmVkaXJlY3Rpb24gdGhhdCB0cmlnZ2Vy
cyBzd2l0Y2hpbmcgdG8gR0VULgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEp
LgorCisgICAgICAgICogcGxhdGZvcm0vbmV0d29yay9zb3VwL1Jlc291cmNlSGFuZGxlU291cC5j
cHA6CisgICAgICAgIChXZWJDb3JlOjpkb1JlZGlyZWN0KToKKwogMjAxMy0wMi0xNCAgWXVyeSBT
ZW1pa2hhdHNreSAgPHl1cnlzQGNocm9taXVtLm9yZz4KIAogICAgICAgICBXZWIgSW5zcGVjdG9y
OiBkb24ndCBjcmVhdGUgc3RhdGljIGxvY2FsIHN0cmluZyBmb3IgcHJvZ3JhbSBsaXRlcmFsIGlu
IEluc3BlY3RvclRpbWVsaW5lQWdlbnQKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL3BsYXRm
b3JtL25ldHdvcmsvc291cC9SZXNvdXJjZUhhbmRsZVNvdXAuY3BwIGIvU291cmNlL1dlYkNvcmUv
cGxhdGZvcm0vbmV0d29yay9zb3VwL1Jlc291cmNlSGFuZGxlU291cC5jcHAKaW5kZXggZjM4N2Fm
NTEyOTE3NGU0YTQ0NDdmMGI4MWQ5MGZjMWY5Y2RmZTkyYS4uYTRkYWNlM2NjZmNiMWNjNTgxZWQ1
MDViYzg1MGZjNzA1NDA3NzExYiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvcGxhdGZvcm0v
bmV0d29yay9zb3VwL1Jlc291cmNlSGFuZGxlU291cC5jcHAKKysrIGIvU291cmNlL1dlYkNvcmUv
cGxhdGZvcm0vbmV0d29yay9zb3VwL1Jlc291cmNlSGFuZGxlU291cC5jcHAKQEAgLTQ1NCw0OCAr
NDU0LDUyIEBAIHN0YXRpYyB2b2lkIGRvUmVkaXJlY3QoUmVzb3VyY2VIYW5kbGUqIGhhbmRsZSkK
ICAgICAgICAgcmV0dXJuOwogICAgIH0KIAotICAgIFJlc291cmNlUmVxdWVzdCByZXF1ZXN0ID0g
aGFuZGxlLT5maXJzdFJlcXVlc3QoKTsKKyAgICBSZXNvdXJjZVJlcXVlc3QgbmV3UmVxdWVzdCA9
IGhhbmRsZS0+Zmlyc3RSZXF1ZXN0KCk7CiAgICAgU291cE1lc3NhZ2UqIG1lc3NhZ2UgPSBkLT5t
X3NvdXBNZXNzYWdlLmdldCgpOwogICAgIGNvbnN0IGNoYXIqIGxvY2F0aW9uID0gc291cF9tZXNz
YWdlX2hlYWRlcnNfZ2V0X29uZShtZXNzYWdlLT5yZXNwb25zZV9oZWFkZXJzLCAiTG9jYXRpb24i
KTsKICAgICBLVVJMIG5ld1VSTCA9IEtVUkwoc291cFVSSVRvS1VSTChzb3VwX21lc3NhZ2VfZ2V0
X3VyaShtZXNzYWdlKSksIGxvY2F0aW9uKTsKLSAgICBib29sIGNyb3NzT3JpZ2luID0gIXByb3Rv
Y29sSG9zdEFuZFBvcnRBcmVFcXVhbChyZXF1ZXN0LnVybCgpLCBuZXdVUkwpOwotICAgIHJlcXVl
c3Quc2V0VVJMKG5ld1VSTCk7Ci0KLSAgICBpZiAoc2hvdWxkUmVkaXJlY3RBc0dFVChtZXNzYWdl
LCBuZXdVUkwsIGNyb3NzT3JpZ2luKSkgewotICAgICAgICByZXF1ZXN0LnNldEhUVFBNZXRob2Qo
IkdFVCIpOwotICAgICAgICByZXF1ZXN0LnNldEhUVFBCb2R5KDApOwotICAgICAgICByZXF1ZXN0
LmNsZWFySFRUUENvbnRlbnRUeXBlKCk7CisgICAgYm9vbCBjcm9zc09yaWdpbiA9ICFwcm90b2Nv
bEhvc3RBbmRQb3J0QXJlRXF1YWwoaGFuZGxlLT5maXJzdFJlcXVlc3QoKS51cmwoKSwgbmV3VVJM
KTsKKyAgICBuZXdSZXF1ZXN0LnNldFVSTChuZXdVUkwpOworCisgICAgaWYgKG5ld1JlcXVlc3Qu
aHR0cE1ldGhvZCgpICE9ICJHRVQiKSB7CisgICAgICAgIC8vIENoYW5nZSBuZXdSZXF1ZXN0IG1l
dGhvZCB0byBHRVQgaWYgY2hhbmdlIHdhcyBtYWRlIGR1cmluZyBhIHByZXZpb3VzIHJlZGlyZWN0
aW9uCisgICAgICAgIC8vIG9yIGlmIGN1cnJlbnQgcmVkaXJlY3Rpb24gc2F5cyBzbworICAgICAg
ICBpZiAobWVzc2FnZS0+bWV0aG9kID09IFNPVVBfTUVUSE9EX0dFVCB8fCBzaG91bGRSZWRpcmVj
dEFzR0VUKG1lc3NhZ2UsIG5ld1VSTCwgY3Jvc3NPcmlnaW4pKSB7CisgICAgICAgICAgICBuZXdS
ZXF1ZXN0LnNldEhUVFBNZXRob2QoIkdFVCIpOworICAgICAgICAgICAgbmV3UmVxdWVzdC5zZXRI
VFRQQm9keSgwKTsKKyAgICAgICAgICAgIG5ld1JlcXVlc3QuY2xlYXJIVFRQQ29udGVudFR5cGUo
KTsKKyAgICAgICAgfQogICAgIH0KIAogICAgIC8vIFNob3VsZCBub3Qgc2V0IFJlZmVyZXIgYWZ0
ZXIgYSByZWRpcmVjdCBmcm9tIGEgc2VjdXJlIHJlc291cmNlIHRvIG5vbi1zZWN1cmUgb25lLgot
ICAgIGlmICghbmV3VVJMLnByb3RvY29sSXMoImh0dHBzIikgJiYgcHJvdG9jb2xJcyhyZXF1ZXN0
Lmh0dHBSZWZlcnJlcigpLCAiaHR0cHMiKSAmJiBoYW5kbGUtPmNvbnRleHQoKS0+c2hvdWxkQ2xl
YXJSZWZlcnJlck9uSFRUUFNUb0hUVFBSZWRpcmVjdCgpKQotICAgICAgICByZXF1ZXN0LmNsZWFy
SFRUUFJlZmVycmVyKCk7CisgICAgaWYgKCFuZXdVUkwucHJvdG9jb2xJcygiaHR0cHMiKSAmJiBw
cm90b2NvbElzKG5ld1JlcXVlc3QuaHR0cFJlZmVycmVyKCksICJodHRwcyIpICYmIGhhbmRsZS0+
Y29udGV4dCgpLT5zaG91bGRDbGVhclJlZmVycmVyT25IVFRQU1RvSFRUUFJlZGlyZWN0KCkpCisg
ICAgICAgIG5ld1JlcXVlc3QuY2xlYXJIVFRQUmVmZXJyZXIoKTsKIAogICAgIGQtPm1fdXNlciA9
IG5ld1VSTC51c2VyKCk7CiAgICAgZC0+bV9wYXNzID0gbmV3VVJMLnBhc3MoKTsKLSAgICByZXF1
ZXN0LnJlbW92ZUNyZWRlbnRpYWxzKCk7CisgICAgbmV3UmVxdWVzdC5yZW1vdmVDcmVkZW50aWFs
cygpOwogCiAgICAgaWYgKGNyb3NzT3JpZ2luKSB7CiAgICAgICAgIC8vIElmIHRoZSBuZXR3b3Jr
IGxheWVyIGNhcnJpZXMgb3ZlciBhdXRoZW50aWNhdGlvbiBoZWFkZXJzIGZyb20gdGhlIG9yaWdp
bmFsIHJlcXVlc3QKICAgICAgICAgLy8gaW4gYSBjcm9zcy1vcmlnaW4gcmVkaXJlY3QsIHdlIHdh
bnQgdG8gY2xlYXIgdGhvc2UgaGVhZGVycyBoZXJlLiAKLSAgICAgICAgcmVxdWVzdC5jbGVhckhU
VFBBdXRob3JpemF0aW9uKCk7CisgICAgICAgIG5ld1JlcXVlc3QuY2xlYXJIVFRQQXV0aG9yaXph
dGlvbigpOwogCiAgICAgICAgIC8vIFRPRE86IFdlIGFyZSBsb3NpbmcgYW55IHVzZXJuYW1lIGFu
ZCBwYXNzd29yZCBzcGVjaWZpZWQgaW4gdGhlIHJlZGlyZWN0IFVSTCwgYXMgdGhpcyBpcyB0aGUg
CiAgICAgICAgIC8vIHNhbWUgYmVoYXZpb3IgYXMgdGhlIENGTmV0IHBvcnQuIFdlIHNob3VsZCBp
bnZlc3RpZ2F0ZSBpZiB0aGlzIGlzIHJlYWxseSB3aGF0IHdlIHdhbnQuCiAgICAgfSBlbHNlCi0g
ICAgICAgIGFwcGx5QXV0aGVudGljYXRpb25Ub1JlcXVlc3QoaGFuZGxlLCByZXF1ZXN0LCB0cnVl
KTsKKyAgICAgICAgYXBwbHlBdXRoZW50aWNhdGlvblRvUmVxdWVzdChoYW5kbGUsIG5ld1JlcXVl
c3QsIHRydWUpOwogCiAgICAgY2xlYW51cFNvdXBSZXF1ZXN0T3BlcmF0aW9uKGhhbmRsZSk7Ci0g
ICAgaWYgKCFjcmVhdGVTb3VwUmVxdWVzdEFuZE1lc3NhZ2VGb3JIYW5kbGUoaGFuZGxlLCByZXF1
ZXN0LCB0cnVlKSkgeworICAgIGlmICghY3JlYXRlU291cFJlcXVlc3RBbmRNZXNzYWdlRm9ySGFu
ZGxlKGhhbmRsZSwgbmV3UmVxdWVzdCwgdHJ1ZSkpIHsKICAgICAgICAgZC0+Y2xpZW50KCktPmNh
bm5vdFNob3dVUkwoaGFuZGxlKTsKICAgICAgICAgcmV0dXJuOwogICAgIH0KIAogICAgIC8vIElm
IHdlIHNlbnQgY3JlZGVudGlhbHMgd2l0aCB0aGlzIHJlcXVlc3QncyBVUkwsIHdlIGRvbid0IHdh
bnQgdGhlIHJlc3BvbnNlIHRvIGNhcnJ5IHRoZW0gdG8KICAgICAvLyB0aGUgV2ViS2l0IGxheWVy
LiBUaGV5IHdlcmUgb25seSBwbGFjZWQgaW4gdGhlIFVSTCBmb3IgdGhlIGJlbmVmaXQgb2YgbGli
c291cC4KLSAgICByZXF1ZXN0LnJlbW92ZUNyZWRlbnRpYWxzKCk7CisgICAgbmV3UmVxdWVzdC5y
ZW1vdmVDcmVkZW50aWFscygpOwogCi0gICAgZC0+Y2xpZW50KCktPndpbGxTZW5kUmVxdWVzdCho
YW5kbGUsIHJlcXVlc3QsIGQtPm1fcmVzcG9uc2UpOworICAgIGQtPmNsaWVudCgpLT53aWxsU2Vu
ZFJlcXVlc3QoaGFuZGxlLCBuZXdSZXF1ZXN0LCBkLT5tX3Jlc3BvbnNlKTsKICAgICBoYW5kbGUt
PnNlbmRQZW5kaW5nUmVxdWVzdCgpOwogfQogCmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9DaGFu
Z2VMb2cgYi9MYXlvdXRUZXN0cy9DaGFuZ2VMb2cKaW5kZXggNmE0N2EzMTM4YzdjMTVjNTEzZmIy
Zjc0ODcxYTNmM2QzM2Q4ZjQ2OC4uODNhMDIyYjYwNzhkMWE1ZGJiM2ZhMmVjMjRkM2I5NWMwYjc3
NjEzYiAxMDA2NDQKLS0tIGEvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCisrKyBiL0xheW91dFRlc3Rz
L0NoYW5nZUxvZwpAQCAtMSwzICsxLDE1IEBACisyMDEzLTAyLTExICBZb3Vlbm4gRmFibGV0ICA8
eW91ZW5uZkBnbWFpbC5jb20+CisKKyAgICAgICAgW0VGTF1bRFJUXSBodHRwL3Rlc3RzL2xvYWRp
bmcvMzA3LWFmdGVyLTMwMy1hZnRlci1wb3N0Lmh0bWwgdGltZXMgb3V0CisgICAgICAgIGh0dHBz
Oi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD05MzIxNAorCisgICAgICAgIFJlbW92
ZWQgRmFpbHVyZSBmcm9tIHBvdGVudGlhbCBvdXRjb21lIG9mCisgICAgICAgIGh0dHAvdGVzdHMv
bG9hZGluZy8zMDctYWZ0ZXItMzAzLWFmdGVyLXBvc3QuaHRtbCB0ZXN0LgorCisgICAgICAgIFJl
dmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgICogcGxhdGZvcm0vZWZsL1Rlc3RF
eHBlY3RhdGlvbnM6CisKIDIwMTMtMDItMTQgIMOBZMOhbSBLYWxsYWkgIDxrYWRhbUBpbmYudS1z
emVnZWQuaHU+CiAKICAgICAgICAgW1F0XSBVbnJldmlld2VkIGdhcmRlbmluZy4gU2tpcCBmbGFr
eSB0ZXN0IGFmdGVyIHIxNDA2ODkuCmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9wbGF0Zm9ybS9l
ZmwvVGVzdEV4cGVjdGF0aW9ucyBiL0xheW91dFRlc3RzL3BsYXRmb3JtL2VmbC9UZXN0RXhwZWN0
YXRpb25zCmluZGV4IDZjZjM1NzZiMGUyNDc3MjkzZDlmNTUwNzQwYWI0YjJhOTJhOTMyZDEuLmU2
NmNiODExMTc5OThkOGQ1OGExZDJjN2FmZWQ3NzQ3NDJjODc3NzIgMTAwNjQ0Ci0tLSBhL0xheW91
dFRlc3RzL3BsYXRmb3JtL2VmbC9UZXN0RXhwZWN0YXRpb25zCisrKyBiL0xheW91dFRlc3RzL3Bs
YXRmb3JtL2VmbC9UZXN0RXhwZWN0YXRpb25zCkBAIC03NjAsNyArNzYwLDcgQEAgd2Via2l0Lm9y
Zy9iLzc3NjMxIGZhc3QvbXVsdGljb2wvc3Bhbi9zcGFuLWFzLWltbWVkaWF0ZS1jaGlsZC1wcm9w
ZXJ0eS1yZW1vdmFsLmgKIHdlYmtpdC5vcmcvYi83NzYzMSBmYXN0L211bHRpY29sL3NwYW4vc3Bh
bi1hcy1pbW1lZGlhdGUtY29sdW1ucy1jaGlsZC1yZW1vdmFsLmh0bWwgWyBGYWlsdXJlIFBhc3Mg
XQogCiB3ZWJraXQub3JnL2IvOTMyMTMgZmFzdC90ZXh0L2ludGVybmF0aW9uYWwvdGV4dC1zcGxp
Y2VkLWZvbnQuaHRtbCBbIEZhaWx1cmUgUGFzcyBdCi13ZWJraXQub3JnL2IvOTMyMTQgaHR0cC90
ZXN0cy9sb2FkaW5nLzMwNy1hZnRlci0zMDMtYWZ0ZXItcG9zdC5odG1sIFsgQ3Jhc2ggRmFpbHVy
ZSBQYXNzIFRpbWVvdXQgXQord2Via2l0Lm9yZy9iLzkzMjE0IGh0dHAvdGVzdHMvbG9hZGluZy8z
MDctYWZ0ZXItMzAzLWFmdGVyLXBvc3QuaHRtbCBbIENyYXNoIFBhc3MgVGltZW91dCBdCiB3ZWJr
aXQub3JnL2IvOTMyMTIgaHR0cC90ZXN0cy9pbmNyZW1lbnRhbC9zbG93LXV0ZjgtdGV4dC5wbCBb
IFBhc3MgVGltZW91dCBdCiAKIHdlYmtpdC5vcmcvYi84NTE4MCBtZWRpYS9hdWRpby1kZWxldGUt
d2hpbGUtc3RlcC1idXR0b24tY2xpY2tlZC5odG1sIFsgRmFpbHVyZSBQYXNzIF0K
</data>

          </attachment>
      

    </bug>

</bugzilla>