<?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>59470</bug_id>
          
          <creation_ts>2011-04-26 11:03:38 -0700</creation_ts>
          <short_desc>Prohibit the use of OwnPtr&lt;T*&gt;</short_desc>
          <delta_ts>2011-06-20 09:41:40 -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>New Bugs</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Other</rep_platform>
          <op_sys>OS X 10.5</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>WONTFIX</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>
          <dependson>59469</dependson>
    
    <dependson>59489</dependson>
    
    <dependson>59559</dependson>
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Adrienne Walker">enne</reporter>
          <assigned_to name="Adrienne Walker">enne</assigned_to>
          <cc>abarth</cc>
    
    <cc>buildbot</cc>
    
    <cc>dglazkov</cc>
    
    <cc>enne</cc>
    
    <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>392570</commentid>
    <comment_count>0</comment_count>
    <who name="Adrienne Walker">enne</who>
    <bug_when>2011-04-26 11:03:38 -0700</bug_when>
    <thetext>Prohibit the use of OwnPtr&lt;T*&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>392573</commentid>
    <comment_count>1</comment_count>
      <attachid>91129</attachid>
    <who name="Adrienne Walker">enne</who>
    <bug_when>2011-04-26 11:04:47 -0700</bug_when>
    <thetext>Created attachment 91129
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>392574</commentid>
    <comment_count>2</comment_count>
    <who name="Adrienne Walker">enne</who>
    <bug_when>2011-04-26 11:09:51 -0700</bug_when>
    <thetext>There may be some better way to implement this, but I think it would be an excellent idea to prevent somebody accidentally using OwnPtr&lt;T*&gt; when they intended OwnPtr&lt;T&gt;.  I can&apos;t think of any usage for the former.

(Also, I&apos;m a little unsure of who to CC on this bug, so please add others.)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>392586</commentid>
    <comment_count>3</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2011-04-26 11:26:49 -0700</bug_when>
    <thetext>Attachment 91129 did not build on chromium:
Build output: http://queues.webkit.org/results/8509701</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>392625</commentid>
    <comment_count>4</comment_count>
    <who name="Build Bot">buildbot</who>
    <bug_when>2011-04-26 12:07:53 -0700</bug_when>
    <thetext>Attachment 91129 did not build on win:
Build output: http://queues.webkit.org/results/8508728</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>392629</commentid>
    <comment_count>5</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2011-04-26 12:11:43 -0700</bug_when>
    <thetext>Attachment 91129 did not build on mac:
Build output: http://queues.webkit.org/results/8505948</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>392630</commentid>
    <comment_count>6</comment_count>
      <attachid>91144</attachid>
    <who name="Adrienne Walker">enne</who>
    <bug_when>2011-04-26 12:13:51 -0700</bug_when>
    <thetext>Created attachment 91144
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>392631</commentid>
    <comment_count>7</comment_count>
    <who name="Adrienne Walker">enne</who>
    <bug_when>2011-04-26 12:14:30 -0700</bug_when>
    <thetext>Let&apos;s try this again now that the blocking bugfix has landed.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>392642</commentid>
    <comment_count>8</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2011-04-26 12:57:47 -0700</bug_when>
    <thetext>Attachment 91144 did not build on chromium:
Build output: http://queues.webkit.org/results/8509721</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>392657</commentid>
    <comment_count>9</comment_count>
      <attachid>91144</attachid>
    <who name="Adrienne Walker">enne</who>
    <bug_when>2011-04-26 13:23:57 -0700</bug_when>
    <thetext>Comment on attachment 91144
Patch

Oops.  Need to touch up DRT as well.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>392668</commentid>
    <comment_count>10</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2011-04-26 13:54:35 -0700</bug_when>
    <thetext>Attachment 91129 did not build on chromium:
Build output: http://queues.webkit.org/results/8510701</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>392677</commentid>
    <comment_count>11</comment_count>
    <who name="Build Bot">buildbot</who>
    <bug_when>2011-04-26 14:01:35 -0700</bug_when>
    <thetext>Attachment 91144 did not build on win:
Build output: http://queues.webkit.org/results/8509734</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>392832</commentid>
    <comment_count>12</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2011-04-26 15:34:52 -0700</bug_when>
    <thetext>Attachment 91144 did not build on mac:
Build output: http://queues.webkit.org/results/8508780</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>392988</commentid>
    <comment_count>13</comment_count>
      <attachid>91177</attachid>
    <who name="Adrienne Walker">enne</who>
    <bug_when>2011-04-26 16:23:33 -0700</bug_when>
    <thetext>Created attachment 91177
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>392991</commentid>
    <comment_count>14</comment_count>
    <who name="Adrienne Walker">enne</who>
    <bug_when>2011-04-26 16:23:59 -0700</bug_when>
    <thetext>Third time&apos;s a charm for fixing all the build errors.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>393104</commentid>
    <comment_count>15</comment_count>
      <attachid>91177</attachid>
    <who name="Geoffrey Garen">ggaren</who>
    <bug_when>2011-04-26 16:59:51 -0700</bug_when>
    <thetext>Comment on attachment 91177
Patch

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

r=me

&gt; Source/JavaScriptCore/wtf/OwnPtr.h:179
&gt; +    template&lt;typename T&gt; class OwnPtr&lt;T*&gt; {
&gt; +        // Prevent usage as a real OwnPtr by not defining any functions.
&gt; +    };
&gt; +#endif

I believe you can do even better here by just using a declaration with no definition.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>393122</commentid>
    <comment_count>16</comment_count>
    <who name="Adrienne Walker">enne</who>
    <bug_when>2011-04-26 17:03:50 -0700</bug_when>
    <thetext>(In reply to comment #15)
&gt; (From update of attachment 91177 [details])
&gt; View in context: https://bugs.webkit.org/attachment.cgi?id=91177&amp;action=review
&gt; 
&gt; r=me

Thanks for all the reviews today.  :)

&gt; &gt; Source/JavaScriptCore/wtf/OwnPtr.h:179
&gt; &gt; +    template&lt;typename T&gt; class OwnPtr&lt;T*&gt; {
&gt; &gt; +        // Prevent usage as a real OwnPtr by not defining any functions.
&gt; &gt; +    };
&gt; &gt; +#endif
&gt; 
&gt; I believe you can do even better here by just using a declaration with no definition.

That would be cleaner.  I&apos;ll do that before landing.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>393132</commentid>
    <comment_count>17</comment_count>
    <who name="Build Bot">buildbot</who>
    <bug_when>2011-04-26 17:05:00 -0700</bug_when>
    <thetext>Attachment 91177 did not build on win:
Build output: http://queues.webkit.org/results/8507775</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>393174</commentid>
    <comment_count>18</comment_count>
    <who name="Adrienne Walker">enne</who>
    <bug_when>2011-04-26 17:25:08 -0700</bug_when>
    <thetext>(In reply to comment #16)
&gt; (In reply to comment #15)

&gt; &gt; &gt; Source/JavaScriptCore/wtf/OwnPtr.h:179
&gt; &gt; &gt; +    template&lt;typename T&gt; class OwnPtr&lt;T*&gt; {
&gt; &gt; &gt; +        // Prevent usage as a real OwnPtr by not defining any functions.
&gt; &gt; &gt; +    };
&gt; &gt; &gt; +#endif
&gt; &gt; 
&gt; &gt; I believe you can do even better here by just using a declaration with no definition.
&gt; 
&gt; That would be cleaner.  I&apos;ll do that before landing.

After some study, I think an empty class rather than an incomplete declaration is better.

If the class is incomplete, then it&apos;s harder to #ifdef out this error because every class that is-a or has-a class with an OwnPtr&lt;T*&gt; needs the ALLOW_RAW_POINTER_IN_OWNPTR define.  Hopefully that should go away soon, but in the short term MediaPlayer.h is used in a number of places.

Also, the error message points to OwnPtr.h in the former case where somebody might see the comment, but just says the slightly more obscure &quot;incomplete declaration&quot; in the latter.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>393221</commentid>
    <comment_count>19</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2011-04-26 18:22:01 -0700</bug_when>
    <thetext>Attachment 91144 did not build on chromium:
Build output: http://queues.webkit.org/results/8510781</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>393666</commentid>
    <comment_count>20</comment_count>
    <who name="Adrienne Walker">enne</who>
    <bug_when>2011-04-27 11:11:10 -0700</bug_when>
    <thetext>Oh, yuck.  It looks like the Windows port is set up to handle OwnPtr&lt;HBITMAP&gt; (which is a void*), but has a deleteOwnPtr(HBITMAP) function which will properly clean up its resources.

So, I can&apos;t disallow all OwnPtr&lt;T*&gt;, just all except any except the 10 or so port-specific pointer types listed in OwnPtrCommon.h.

I&apos;ll have to take another look at this patch to account for that while still disallowing the general case.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>423191</commentid>
    <comment_count>21</comment_count>
      <attachid>91177</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2011-06-18 12:46:21 -0700</bug_when>
    <thetext>Comment on attachment 91177
Patch

Clearing review flag because we need to rework this for Windows’ sake. Best possibility would probably be to change Windows to use to something other than OwnPtr. It’s not safe to have it using OwnPtr with void*!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>423712</commentid>
    <comment_count>22</comment_count>
    <who name="Adrienne Walker">enne</who>
    <bug_when>2011-06-20 09:41:40 -0700</bug_when>
    <thetext>(In reply to comment #21)
&gt; (From update of attachment 91177 [details])
&gt; Clearing review flag because we need to rework this for Windows’ sake. Best possibility would probably be to change Windows to use to something other than OwnPtr. It’s not safe to have it using OwnPtr with void*!

Ack.  I thought I had closed this bug.

It turns out that I was just mistaken.  All the Windows-specific versions of OwnPtr like OwnPtr&lt;HBITMAP&gt; have their own specialized delete function which gets called.

And thanks to template magic, for the general case of T, it turns out that OwnPtr&lt;T*&gt; is identical to OwnPtr&lt;T&gt; in terms of what gets stored in the OwnPtr and which delete function gets called.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>91129</attachid>
            <date>2011-04-26 11:04:47 -0700</date>
            <delta_ts>2011-04-26 12:13:48 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-59470-20110426110446.patch</filename>
            <type>text/plain</type>
            <size>2706</size>
            <attacher name="Adrienne Walker">enne</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogODQ5MTYKZGlmZiAtLWdpdCBhL1NvdXJjZS9KYXZhU2NyaXB0
Q29yZS9DaGFuZ2VMb2cgYi9Tb3VyY2UvSmF2YVNjcmlwdENvcmUvQ2hhbmdlTG9nCmluZGV4IDhh
Nzk1NGMzNGZkM2FiMmU3OTc4YzlhYTQ0ZmQ3NmRhYWZiZGY4NDguLjI4ZGU2ZWNiYjJkZTY5ZjM0
ZWVkMGVmNWM1MmZlYzc1YzYyYjJkYmMgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9KYXZhU2NyaXB0Q29y
ZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwpAQCAtMSwz
ICsxLDEyIEBACisyMDExLTA0LTI2ICBBZHJpZW5uZSBXYWxrZXIgIDxlbm5lQGdvb2dsZS5jb20+
CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgUHJvaGli
aXQgdGhlIHVzZSBvZiBPd25QdHI8VCo+CisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3Jn
L3Nob3dfYnVnLmNnaT9pZD01OTQ3MAorCisgICAgICAgICogd3RmL093blB0ci5oOgorCiAyMDEx
LTA0LTI2ICBHYWJvciBMb2tpICA8bG9raUB3ZWJraXQub3JnPgogCiAgICAgICAgIFJldmlld2Vk
IGJ5IENzYWJhIE9zenRyb2dvbsOhYy4KZGlmZiAtLWdpdCBhL1NvdXJjZS9KYXZhU2NyaXB0Q29y
ZS93dGYvT3duUHRyLmggYi9Tb3VyY2UvSmF2YVNjcmlwdENvcmUvd3RmL093blB0ci5oCmluZGV4
IGZiNTk0MzIxYjVhMDU0Mzg3MGY4NjgyZWI4YmQ1MzFjYjUwODM2M2IuLmI5YzQ3YTVlYWM0ZWIw
NTNjMDVlZDRlMzEwZjcwZTNiYmI2ODIzNDQgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9KYXZhU2NyaXB0
Q29yZS93dGYvT3duUHRyLmgKKysrIGIvU291cmNlL0phdmFTY3JpcHRDb3JlL3d0Zi9Pd25QdHIu
aApAQCAtMTcwLDYgKzE3MCwxNCBAQCBuYW1lc3BhY2UgV1RGIHsKICAgICAgICAgcmV0dXJuIHAu
Z2V0KCk7CiAgICAgfQogCisjaWZuZGVmIEFMTE9XX1JBV19QT0lOVEVSX0lOX09XTlBUUgorICAg
IC8vIFBhcnRpYWwgdGVtcGxhdGUgc3BlY2lhbGl6YXRpb24gdG8gcHJvaGliaXQgcmF3IHBvaW50
ZXJzIGJlaW5nIHN0b3JlZAorICAgIC8vIGluIGFuIE93blB0ci4gVXNlIE93bkFycmF5UHRyIGZv
ciBhcnJheXMuCisgICAgdGVtcGxhdGU8dHlwZW5hbWUgVD4gY2xhc3MgT3duUHRyPFQqPiB7Cisg
ICAgICAgIC8vIFByZXZlbnQgdXNhZ2UgYXMgYSByZWFsIE93blB0ciBieSBub3QgZGVmaW5pbmcg
YW55IGZ1bmN0aW9ucy4KKyAgICB9OworI2VuZGlmCisKIH0gLy8gbmFtZXNwYWNlIFdURgogCiB1
c2luZyBXVEY6Ok93blB0cjsKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyBi
L1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwppbmRleCAwNzRjYzk4ZTM4OWJlNWUxODM5NTY0Yzhk
MjhlNWFkNjdjNTMwZjUzLi4wNjM5ZTg0Yzg3OGExOGViNzFlNDliMWY3YTM5MmE1MzFhN2IwNmEw
IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkNv
cmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTQgQEAKKzIwMTEtMDQtMjYgIEFkcmllbm5lIFdhbGtl
ciAgPGVubmVAZ29vZ2xlLmNvbT4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMh
KS4KKworICAgICAgICBQcm9oaWJpdCB0aGUgdXNlIG9mIE93blB0cjxUKj4KKyAgICAgICAgaHR0
cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTU5NDcwCisKKyAgICAgICAgQWxs
b3cgdGhpcyB0ZW1wb3JhcmlseSBpbiBNZWRpYVBsYXllciBiZWNhdXNlIGl0IG5lZWRzIHRvIGJl
IGZpeGVkLgorCisgICAgICAgICogcGxhdGZvcm0vZ3JhcGhpY3MvTWVkaWFQbGF5ZXIuaDoKKwog
MjAxMS0wNC0yNiAgR2Fib3IgTG9raSAgPGxva2lAd2Via2l0Lm9yZz4KIAogICAgICAgICBSZXZp
ZXdlZCBieSBDc2FiYSBPc3p0cm9nb27DoWMuCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9w
bGF0Zm9ybS9ncmFwaGljcy9NZWRpYVBsYXllci5oIGIvU291cmNlL1dlYkNvcmUvcGxhdGZvcm0v
Z3JhcGhpY3MvTWVkaWFQbGF5ZXIuaAppbmRleCAzNDgxYjBlMWQ2MzhiZDI3YWE4ZTE4ZDVjYzA2
OGNlNjIxNTBhMmVhLi41NDQyOGVhMzA5ZTlkYTFlNTdmYjc5N2FjNjE3ODE1YjBjN2JlNTBkIDEw
MDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy9NZWRpYVBsYXllci5o
CisrKyBiL1NvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL01lZGlhUGxheWVyLmgKQEAg
LTI4LDYgKzI4LDkgQEAKIAogI2lmIEVOQUJMRShWSURFTykKIAorLy8gRklYTUU6IFJlbW92ZSB0
aGlzIGFmdGVyIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD01OTQ2Ngor
I2RlZmluZSBBTExPV19SQVdfUE9JTlRFUl9JTl9PV05QVFIKKwogI2lmIEVOQUJMRShQTFVHSU5f
UFJPWFlfRk9SX1ZJREVPKQogI2luY2x1ZGUgIk1lZGlhUGxheWVyUHJveHkuaCIKICNlbmRpZgpA
QCAtMzQzLDYgKzM0Niw5IEBAIHR5cGVkZWYgdm9pZCAoKk1lZGlhRW5naW5lUmVnaXN0cmFyKShD
cmVhdGVNZWRpYUVuZ2luZVBsYXllciwgTWVkaWFFbmdpbmVTdXBwb3J0CiAKIH0KIAorLy8gRklY
TUU6IFJlbW92ZSB0aGlzIGFmdGVyIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNn
aT9pZD01OTQ2NgorI3VuZGVmIEFMTE9XX1JBV19QT0lOVEVSX0lOX09XTlBUUgorCiAjZW5kaWYg
Ly8gRU5BQkxFKFZJREVPKQogCiAjZW5kaWYK
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>91144</attachid>
            <date>2011-04-26 12:13:51 -0700</date>
            <delta_ts>2011-04-26 16:23:30 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-59470-20110426121350.patch</filename>
            <type>text/plain</type>
            <size>2623</size>
            <attacher name="Adrienne Walker">enne</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogODQ5NDAKZGlmZiAtLWdpdCBhL1NvdXJjZS9KYXZhU2NyaXB0
Q29yZS9DaGFuZ2VMb2cgYi9Tb3VyY2UvSmF2YVNjcmlwdENvcmUvQ2hhbmdlTG9nCmluZGV4IGE1
ZDA0MzhlN2JlMWU2YjFkOTg4MjlkNTY2NDYyNzlmNDE2MGExZDEuLjk4ZWE1ODBkYmI0ODVlNTQz
MTM4ODdjNzc2MWM3ZWJhMTY4YmI3ODEgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9KYXZhU2NyaXB0Q29y
ZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwpAQCAtMSwz
ICsxLDEyIEBACisyMDExLTA0LTI2ICBBZHJpZW5uZSBXYWxrZXIgIDxlbm5lQGdvb2dsZS5jb20+
CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgUHJvaGli
aXQgdGhlIHVzZSBvZiBPd25QdHI8VCo+CisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3Jn
L3Nob3dfYnVnLmNnaT9pZD01OTQ3MAorCisgICAgICAgICogd3RmL093blB0ci5oOgorCiAyMDEx
LTA0LTI1ICBHZW9mZnJleSBHYXJlbiAgPGdnYXJlbkBhcHBsZS5jb20+CiAKICAgICAgICAgUmV2
aWV3ZWQgYnkgT2xpdmVyIEh1bnQuCmRpZmYgLS1naXQgYS9Tb3VyY2UvSmF2YVNjcmlwdENvcmUv
d3RmL093blB0ci5oIGIvU291cmNlL0phdmFTY3JpcHRDb3JlL3d0Zi9Pd25QdHIuaAppbmRleCBm
YjU5NDMyMWI1YTA1NDM4NzBmODY4MmViOGJkNTMxY2I1MDgzNjNiLi5iOWM0N2E1ZWFjNGViMDUz
YzA1ZWQ0ZTMxMGY3MGUzYmJiNjgyMzQ0IDEwMDY0NAotLS0gYS9Tb3VyY2UvSmF2YVNjcmlwdENv
cmUvd3RmL093blB0ci5oCisrKyBiL1NvdXJjZS9KYXZhU2NyaXB0Q29yZS93dGYvT3duUHRyLmgK
QEAgLTE3MCw2ICsxNzAsMTQgQEAgbmFtZXNwYWNlIFdURiB7CiAgICAgICAgIHJldHVybiBwLmdl
dCgpOwogICAgIH0KIAorI2lmbmRlZiBBTExPV19SQVdfUE9JTlRFUl9JTl9PV05QVFIKKyAgICAv
LyBQYXJ0aWFsIHRlbXBsYXRlIHNwZWNpYWxpemF0aW9uIHRvIHByb2hpYml0IHJhdyBwb2ludGVy
cyBiZWluZyBzdG9yZWQKKyAgICAvLyBpbiBhbiBPd25QdHIuIFVzZSBPd25BcnJheVB0ciBmb3Ig
YXJyYXlzLgorICAgIHRlbXBsYXRlPHR5cGVuYW1lIFQ+IGNsYXNzIE93blB0cjxUKj4geworICAg
ICAgICAvLyBQcmV2ZW50IHVzYWdlIGFzIGEgcmVhbCBPd25QdHIgYnkgbm90IGRlZmluaW5nIGFu
eSBmdW5jdGlvbnMuCisgICAgfTsKKyNlbmRpZgorCiB9IC8vIG5hbWVzcGFjZSBXVEYKIAogdXNp
bmcgV1RGOjpPd25QdHI7CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cgYi9T
b3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggNjVlYjU2MjdhODU3YmU4MmNjOWY4OTY1NTQ5
YWQwZDdiNTg3OGM0NC4uNWEwNDRiZDA3MjU1YmY5ZmQyMzRiY2EwOGIyMTg1OWU0YmJjMTgzOSAx
MDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1NvdXJjZS9XZWJDb3Jl
L0NoYW5nZUxvZwpAQCAtMSw1ICsxLDE2IEBACiAyMDExLTA0LTI2ICBBZHJpZW5uZSBXYWxrZXIg
IDxlbm5lQGdvb2dsZS5jb20+CiAKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISku
CisKKyAgICAgICAgUHJvaGliaXQgdGhlIHVzZSBvZiBPd25QdHI8VCo+CisgICAgICAgIGh0dHBz
Oi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD01OTQ3MAorCisgICAgICAgIEFsbG93
IHRoaXMgdGVtcG9yYXJpbHkgaW4gTWVkaWFQbGF5ZXIgYmVjYXVzZSBpdCBuZWVkcyB0byBiZSBm
aXhlZC4KKworICAgICAgICAqIHBsYXRmb3JtL2dyYXBoaWNzL01lZGlhUGxheWVyLmg6CisKKzIw
MTEtMDQtMjYgIEFkcmllbm5lIFdhbGtlciAgPGVubmVAZ29vZ2xlLmNvbT4KKwogICAgICAgICBS
ZXZpZXdlZCBieSBHZW9mZnJleSBHYXJlbi4KIAogICAgICAgICBVc2UgT3duQXJyYXlQdHI8VD4g
aW5zdGVhZCBvZiBPd25QdHI8VCo+IGluIFNjb3BlVHJhY2VyCmRpZmYgLS1naXQgYS9Tb3VyY2Uv
V2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy9NZWRpYVBsYXllci5oIGIvU291cmNlL1dlYkNvcmUv
cGxhdGZvcm0vZ3JhcGhpY3MvTWVkaWFQbGF5ZXIuaAppbmRleCAzNDgxYjBlMWQ2MzhiZDI3YWE4
ZTE4ZDVjYzA2OGNlNjIxNTBhMmVhLi43YmI4MGYxMzZhYzIzZDNmZDg0MmFjMDAyOWI0MTRkYmVm
NGY4Njc1IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy9NZWRp
YVBsYXllci5oCisrKyBiL1NvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL01lZGlhUGxh
eWVyLmgKQEAgLTI4LDYgKzI4LDEzIEBACiAKICNpZiBFTkFCTEUoVklERU8pCiAKKy8vIEZJWE1F
OiBSZW1vdmUgdGhpcyBhZnRlciBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/
aWQ9NTk0NjYKKyNpZm5kZWYgQUxMT1dfUkFXX1BPSU5URVJfSU5fT1dOUFRSCisjZGVmaW5lIEFM
TE9XX1JBV19QT0lOVEVSX0lOX09XTlBUUgorI2luY2x1ZGUgPHd0Zi9Pd25QdHIuaD4KKyN1bmRl
ZiBBTExPV19SQVdfUE9JTlRFUl9JTl9PV05QVFIKKyNlbmRpZgorCiAjaWYgRU5BQkxFKFBMVUdJ
Tl9QUk9YWV9GT1JfVklERU8pCiAjaW5jbHVkZSAiTWVkaWFQbGF5ZXJQcm94eS5oIgogI2VuZGlm
Cg==
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>91177</attachid>
            <date>2011-04-26 16:23:33 -0700</date>
            <delta_ts>2011-06-18 12:46:21 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-59470-20110426162332.patch</filename>
            <type>text/plain</type>
            <size>2623</size>
            <attacher name="Adrienne Walker">enne</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogODQ5NDAKZGlmZiAtLWdpdCBhL1NvdXJjZS9KYXZhU2NyaXB0
Q29yZS9DaGFuZ2VMb2cgYi9Tb3VyY2UvSmF2YVNjcmlwdENvcmUvQ2hhbmdlTG9nCmluZGV4IGE1
ZDA0MzhlN2JlMWU2YjFkOTg4MjlkNTY2NDYyNzlmNDE2MGExZDEuLjk4ZWE1ODBkYmI0ODVlNTQz
MTM4ODdjNzc2MWM3ZWJhMTY4YmI3ODEgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9KYXZhU2NyaXB0Q29y
ZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwpAQCAtMSwz
ICsxLDEyIEBACisyMDExLTA0LTI2ICBBZHJpZW5uZSBXYWxrZXIgIDxlbm5lQGdvb2dsZS5jb20+
CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgUHJvaGli
aXQgdGhlIHVzZSBvZiBPd25QdHI8VCo+CisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3Jn
L3Nob3dfYnVnLmNnaT9pZD01OTQ3MAorCisgICAgICAgICogd3RmL093blB0ci5oOgorCiAyMDEx
LTA0LTI1ICBHZW9mZnJleSBHYXJlbiAgPGdnYXJlbkBhcHBsZS5jb20+CiAKICAgICAgICAgUmV2
aWV3ZWQgYnkgT2xpdmVyIEh1bnQuCmRpZmYgLS1naXQgYS9Tb3VyY2UvSmF2YVNjcmlwdENvcmUv
d3RmL093blB0ci5oIGIvU291cmNlL0phdmFTY3JpcHRDb3JlL3d0Zi9Pd25QdHIuaAppbmRleCBm
YjU5NDMyMWI1YTA1NDM4NzBmODY4MmViOGJkNTMxY2I1MDgzNjNiLi5iOWM0N2E1ZWFjNGViMDUz
YzA1ZWQ0ZTMxMGY3MGUzYmJiNjgyMzQ0IDEwMDY0NAotLS0gYS9Tb3VyY2UvSmF2YVNjcmlwdENv
cmUvd3RmL093blB0ci5oCisrKyBiL1NvdXJjZS9KYXZhU2NyaXB0Q29yZS93dGYvT3duUHRyLmgK
QEAgLTE3MCw2ICsxNzAsMTQgQEAgbmFtZXNwYWNlIFdURiB7CiAgICAgICAgIHJldHVybiBwLmdl
dCgpOwogICAgIH0KIAorI2lmbmRlZiBBTExPV19SQVdfUE9JTlRFUl9JTl9PV05QVFIKKyAgICAv
LyBQYXJ0aWFsIHRlbXBsYXRlIHNwZWNpYWxpemF0aW9uIHRvIHByb2hpYml0IHJhdyBwb2ludGVy
cyBiZWluZyBzdG9yZWQKKyAgICAvLyBpbiBhbiBPd25QdHIuIFVzZSBPd25BcnJheVB0ciBmb3Ig
YXJyYXlzLgorICAgIHRlbXBsYXRlPHR5cGVuYW1lIFQ+IGNsYXNzIE93blB0cjxUKj4geworICAg
ICAgICAvLyBQcmV2ZW50IHVzYWdlIGFzIGEgcmVhbCBPd25QdHIgYnkgbm90IGRlZmluaW5nIGFu
eSBmdW5jdGlvbnMuCisgICAgfTsKKyNlbmRpZgorCiB9IC8vIG5hbWVzcGFjZSBXVEYKIAogdXNp
bmcgV1RGOjpPd25QdHI7CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cgYi9T
b3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggNjVlYjU2MjdhODU3YmU4MmNjOWY4OTY1NTQ5
YWQwZDdiNTg3OGM0NC4uNWEwNDRiZDA3MjU1YmY5ZmQyMzRiY2EwOGIyMTg1OWU0YmJjMTgzOSAx
MDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1NvdXJjZS9XZWJDb3Jl
L0NoYW5nZUxvZwpAQCAtMSw1ICsxLDE2IEBACiAyMDExLTA0LTI2ICBBZHJpZW5uZSBXYWxrZXIg
IDxlbm5lQGdvb2dsZS5jb20+CiAKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISku
CisKKyAgICAgICAgUHJvaGliaXQgdGhlIHVzZSBvZiBPd25QdHI8VCo+CisgICAgICAgIGh0dHBz
Oi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD01OTQ3MAorCisgICAgICAgIEFsbG93
IHRoaXMgdGVtcG9yYXJpbHkgaW4gTWVkaWFQbGF5ZXIgYmVjYXVzZSBpdCBuZWVkcyB0byBiZSBm
aXhlZC4KKworICAgICAgICAqIHBsYXRmb3JtL2dyYXBoaWNzL01lZGlhUGxheWVyLmg6CisKKzIw
MTEtMDQtMjYgIEFkcmllbm5lIFdhbGtlciAgPGVubmVAZ29vZ2xlLmNvbT4KKwogICAgICAgICBS
ZXZpZXdlZCBieSBHZW9mZnJleSBHYXJlbi4KIAogICAgICAgICBVc2UgT3duQXJyYXlQdHI8VD4g
aW5zdGVhZCBvZiBPd25QdHI8VCo+IGluIFNjb3BlVHJhY2VyCmRpZmYgLS1naXQgYS9Tb3VyY2Uv
V2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy9NZWRpYVBsYXllci5oIGIvU291cmNlL1dlYkNvcmUv
cGxhdGZvcm0vZ3JhcGhpY3MvTWVkaWFQbGF5ZXIuaAppbmRleCAzNDgxYjBlMWQ2MzhiZDI3YWE4
ZTE4ZDVjYzA2OGNlNjIxNTBhMmVhLi43YmI4MGYxMzZhYzIzZDNmZDg0MmFjMDAyOWI0MTRkYmVm
NGY4Njc1IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy9NZWRp
YVBsYXllci5oCisrKyBiL1NvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL01lZGlhUGxh
eWVyLmgKQEAgLTI4LDYgKzI4LDEzIEBACiAKICNpZiBFTkFCTEUoVklERU8pCiAKKy8vIEZJWE1F
OiBSZW1vdmUgdGhpcyBhZnRlciBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/
aWQ9NTk0NjYKKyNpZm5kZWYgQUxMT1dfUkFXX1BPSU5URVJfSU5fT1dOUFRSCisjZGVmaW5lIEFM
TE9XX1JBV19QT0lOVEVSX0lOX09XTlBUUgorI2luY2x1ZGUgPHd0Zi9Pd25QdHIuaD4KKyN1bmRl
ZiBBTExPV19SQVdfUE9JTlRFUl9JTl9PV05QVFIKKyNlbmRpZgorCiAjaWYgRU5BQkxFKFBMVUdJ
Tl9QUk9YWV9GT1JfVklERU8pCiAjaW5jbHVkZSAiTWVkaWFQbGF5ZXJQcm94eS5oIgogI2VuZGlm
Cg==
</data>

          </attachment>
      

    </bug>

</bugzilla>