<?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>36473</bug_id>
          
          <creation_ts>2010-03-22 18:20:39 -0700</creation_ts>
          <short_desc>Missing lock in call to doneCreatingDatabase</short_desc>
          <delta_ts>2010-03-25 23:06:28 -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>DOM</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</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="Eric U.">ericu</reporter>
          <assigned_to name="Eric U.">ericu</assigned_to>
          <cc>commit-queue</cc>
    
    <cc>dumi</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>202865</commentid>
    <comment_count>0</comment_count>
    <who name="Eric U.">ericu</who>
    <bug_when>2010-03-22 18:20:39 -0700</bug_when>
    <thetext>The last call to doneCreatingDatabase in DatabaseTracker::canEstablishDatabase is called without locking m_databaseGuard.  This will assert in debug builds and use a supposedly-protected data structure without locking in release builds.  This is a bug in my last checkin [56293].

This wasn&apos;t caught by layout tests because we don&apos;t have one that tries to create a database with a large quota.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>202882</commentid>
    <comment_count>1</comment_count>
      <attachid>51382</attachid>
    <who name="Eric U.">ericu</who>
    <bug_when>2010-03-22 18:51:53 -0700</bug_when>
    <thetext>Created attachment 51382
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>202885</commentid>
    <comment_count>2</comment_count>
    <who name="Dumitru Daniliuc">dumi</who>
    <bug_when>2010-03-22 19:01:40 -0700</bug_when>
    <thetext>LGTM, fwiw.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>203863</commentid>
    <comment_count>3</comment_count>
      <attachid>51382</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2010-03-25 00:28:14 -0700</bug_when>
    <thetext>Comment on attachment 51382
Patch

The test seems missing from this patch.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>204137</commentid>
    <comment_count>4</comment_count>
      <attachid>51656</attachid>
    <who name="Eric U.">ericu</who>
    <bug_when>2010-03-25 10:47:31 -0700</bug_when>
    <thetext>Created attachment 51656
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>204138</commentid>
    <comment_count>5</comment_count>
    <who name="Eric U.">ericu</who>
    <bug_when>2010-03-25 10:48:03 -0700</bug_when>
    <thetext>Thanks Eric; forgot the svn add.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>204234</commentid>
    <comment_count>6</comment_count>
      <attachid>51656</attachid>
    <who name="David Levin">levin</who>
    <bug_when>2010-03-25 14:51:45 -0700</bug_when>
    <thetext>Comment on attachment 51656
Patch

&gt; Index: WebCore/ChangeLog
&gt; +        * storage/DatabaseTracker.cpp:  Added missing lock call.

Ideally your comment would be at the function level (which is where you did this change).

&gt; +        (WebCore::DatabaseTracker::canEstablishDatabase):
&gt; +

&gt; Index: WebCore/storage/DatabaseTracker.cpp
&gt; +    MutexLocker lockDatabase(m_databaseGuard);
&gt;      doneCreatingDatabase(origin, name);

It seems nicer to do this:

  MutexLocker lockDatabase(m_databaseGuard);
  if (requirement &lt;= quotaForOriginNoLock(origin))
      return true;
  
  doneCreatingDatabase(origin, name);

Opinion?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>204250</commentid>
    <comment_count>7</comment_count>
    <who name="Eric U.">ericu</who>
    <bug_when>2010-03-25 15:13:15 -0700</bug_when>
    <thetext>(In reply to comment #6)
&gt; (From update of attachment 51656 [details])
&gt; &gt; Index: WebCore/ChangeLog
&gt; &gt; +        * storage/DatabaseTracker.cpp:  Added missing lock call.
&gt; 
&gt; Ideally your comment would be at the function level (which is where you did
&gt; this change).
&gt; 
&gt; &gt; +        (WebCore::DatabaseTracker::canEstablishDatabase):
&gt; &gt; +

Fixed.

&gt; &gt; Index: WebCore/storage/DatabaseTracker.cpp
&gt; &gt; +    MutexLocker lockDatabase(m_databaseGuard);
&gt; &gt;      doneCreatingDatabase(origin, name);
&gt; 
&gt; It seems nicer to do this:
&gt; 
&gt;   MutexLocker lockDatabase(m_databaseGuard);
&gt;   if (requirement &lt;= quotaForOriginNoLock(origin))
&gt;       return true;
&gt; 
&gt;   doneCreatingDatabase(origin, name);
&gt; 
&gt; Opinion?

Sure, and I&apos;ll do you one better; I can let the previous lock call out of its braces.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>204251</commentid>
    <comment_count>8</comment_count>
      <attachid>51682</attachid>
    <who name="Eric U.">ericu</who>
    <bug_when>2010-03-25 15:13:57 -0700</bug_when>
    <thetext>Created attachment 51682
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>204391</commentid>
    <comment_count>9</comment_count>
      <attachid>51682</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-03-25 20:22:20 -0700</bug_when>
    <thetext>Comment on attachment 51682
Patch

Rejecting patch 51682 from commit-queue.

Unexpected failure when landing patch!  Please file a bug against webkit-patch.
Failed to run &quot;[&apos;WebKitTools/Scripts/webkit-patch&apos;, &apos;--status-host=webkit-commit-queue.appspot.com&apos;, &apos;land-attachment&apos;, &apos;--force-clean&apos;, &apos;--non-interactive&apos;, &apos;--parent-command=commit-queue&apos;, &apos;--build-style=both&apos;, &apos;--quiet&apos;, &apos;51682&apos;, &apos;--no-update&apos;]&quot; exit_code: 1
Last 500 characters of output:
ne 60, in _run
    step(tool, options).run(state)
  File &quot;/Users/eseidel/Projects/CommitQueue/WebKitTools/Scripts/webkitpy/tool/steps/commit.py&quot;, line 34, in run
    commit_message = self._tool.checkout().commit_message_for_this_commit()
  File &quot;/Users/eseidel/Projects/CommitQueue/WebKitTools/Scripts/webkitpy/common/checkout/api.py&quot;, line 65, in commit_message_for_this_commit
    return CommitMessage(&quot;&quot;.join(changelog_messages).splitlines())
NameError: global name &apos;CommitMessage&apos; is not defined</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>204396</commentid>
    <comment_count>10</comment_count>
      <attachid>51682</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2010-03-25 21:18:36 -0700</bug_when>
    <thetext>Comment on attachment 51682
Patch

I&apos;ve restarted the commit-bot.  Sorry for the noise.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>204421</commentid>
    <comment_count>11</comment_count>
      <attachid>51682</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-03-25 23:06:24 -0700</bug_when>
    <thetext>Comment on attachment 51682
Patch

Clearing flags on attachment: 51682

Committed r56604: &lt;http://trac.webkit.org/changeset/56604&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>204422</commentid>
    <comment_count>12</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-03-25 23:06:28 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>51382</attachid>
            <date>2010-03-22 18:51:53 -0700</date>
            <delta_ts>2010-03-25 10:47:28 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-36473-20100322185152.patch</filename>
            <type>text/plain</type>
            <size>1802</size>
            <attacher name="Eric U.">ericu</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n
CShyZXZpc2lvbiA1NjM3MykKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA
IC0xLDMgKzEsMTUgQEAKKzIwMTAtMDMtMjIgIEVyaWMgVWhyaGFuZSAgPGVyaWN1QGNocm9taXVt
Lm9yZz4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBN
aXNzaW5nIGxvY2sgaW4gY2FsbCB0byBkb25lQ3JlYXRpbmdEYXRhYmFzZQorICAgICAgICBodHRw
czovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MzY0NzMKKworICAgICAgICBBZGRl
ZCBuZXcgdGVzdCBzdG9yYWdlL29wZW4tZGF0YWJhc2Utb3Zlci1xdW90YS5odG1sLgorCisgICAg
ICAgICogc3RvcmFnZS9EYXRhYmFzZVRyYWNrZXIuY3BwOiAgQWRkZWQgbWlzc2luZyBsb2NrIGNh
bGwuCisgICAgICAgIChXZWJDb3JlOjpEYXRhYmFzZVRyYWNrZXI6OmNhbkVzdGFibGlzaERhdGFi
YXNlKToKKwogMjAxMC0wMy0yMiAgRGFyaW4gRmlzaGVyICA8ZGFyaW5AY2hyb21pdW0ub3JnPgog
CiAgICAgICAgIFJldmlld2VkIGJ5IEJyYWR5IEVpZHNvbi4KSW5kZXg6IFdlYkNvcmUvc3RvcmFn
ZS9EYXRhYmFzZVRyYWNrZXIuY3BwCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvc3RvcmFnZS9EYXRh
YmFzZVRyYWNrZXIuY3BwCShyZXZpc2lvbiA1NjM2MykKKysrIFdlYkNvcmUvc3RvcmFnZS9EYXRh
YmFzZVRyYWNrZXIuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC0xNzcsNiArMTc3LDcgQEAgYm9vbCBE
YXRhYmFzZVRyYWNrZXI6OmNhbkVzdGFibGlzaERhdGFiYQogICAgIGlmIChyZXF1aXJlbWVudCA8
PSBxdW90YUZvck9yaWdpbihvcmlnaW4pKQogICAgICAgICByZXR1cm4gdHJ1ZTsKIAorICAgIE11
dGV4TG9ja2VyIGxvY2tEYXRhYmFzZShtX2RhdGFiYXNlR3VhcmQpOwogICAgIGRvbmVDcmVhdGlu
Z0RhdGFiYXNlKG9yaWdpbiwgbmFtZSk7CiAgICAgcmV0dXJuIGZhbHNlOwogfQpJbmRleDogTGF5
b3V0VGVzdHMvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIExheW91dFRlc3RzL0NoYW5nZUxvZwko
cmV2aXNpb24gNTYzNjMpCisrKyBMYXlvdXRUZXN0cy9DaGFuZ2VMb2cJKHdvcmtpbmcgY29weSkK
QEAgLTEsMyArMSwxNSBAQAorMjAxMC0wMy0yMiAgRXJpYyBVaHJoYW5lICA8ZXJpY3VAY2hyb21p
dW0ub3JnPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAg
IE1pc3NpbmcgbG9jayBpbiBjYWxsIHRvIGRvbmVDcmVhdGluZ0RhdGFiYXNlCisgICAgICAgIGh0
dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0zNjQ3MworCisgICAgICAgIEFk
ZGVkIG5ldyB0ZXN0IHN0b3JhZ2Uvb3Blbi1kYXRhYmFzZS1vdmVyLXF1b3RhLmh0bWwuCisKKyAg
ICAgICAgKiBzdG9yYWdlL29wZW4tZGF0YWJhc2Utb3Zlci1xdW90YS5odG1sOiBBZGRlZC4KKyAg
ICAgICAgKiBzdG9yYWdlL29wZW4tZGF0YWJhc2Utb3Zlci1xdW90YS1leHBlY3RlZC50eHQ6IEFk
ZGVkLgorCiAyMDEwLTAzLTIyICBLZW5uZXRoIFJvaGRlIENocmlzdGlhbnNlbiAgPGtlbm5ldGhA
d2Via2l0Lm9yZz4KIAogICAgICAgICBVbnJldmlld2VkLgo=
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>51656</attachid>
            <date>2010-03-25 10:47:31 -0700</date>
            <delta_ts>2010-03-25 15:13:54 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-36473-20100325104730.patch</filename>
            <type>text/plain</type>
            <size>3140</size>
            <attacher name="Eric U.">ericu</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n
CShyZXZpc2lvbiA1NjM3MykKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA
IC0xLDMgKzEsMTUgQEAKKzIwMTAtMDMtMjIgIEVyaWMgVWhyaGFuZSAgPGVyaWN1QGNocm9taXVt
Lm9yZz4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBN
aXNzaW5nIGxvY2sgaW4gY2FsbCB0byBkb25lQ3JlYXRpbmdEYXRhYmFzZQorICAgICAgICBodHRw
czovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MzY0NzMKKworICAgICAgICBBZGRl
ZCBuZXcgdGVzdCBzdG9yYWdlL29wZW4tZGF0YWJhc2Utb3Zlci1xdW90YS5odG1sLgorCisgICAg
ICAgICogc3RvcmFnZS9EYXRhYmFzZVRyYWNrZXIuY3BwOiAgQWRkZWQgbWlzc2luZyBsb2NrIGNh
bGwuCisgICAgICAgIChXZWJDb3JlOjpEYXRhYmFzZVRyYWNrZXI6OmNhbkVzdGFibGlzaERhdGFi
YXNlKToKKwogMjAxMC0wMy0yMiAgRGFyaW4gRmlzaGVyICA8ZGFyaW5AY2hyb21pdW0ub3JnPgog
CiAgICAgICAgIFJldmlld2VkIGJ5IEJyYWR5IEVpZHNvbi4KSW5kZXg6IFdlYkNvcmUvc3RvcmFn
ZS9EYXRhYmFzZVRyYWNrZXIuY3BwCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvc3RvcmFnZS9EYXRh
YmFzZVRyYWNrZXIuY3BwCShyZXZpc2lvbiA1NjM2MykKKysrIFdlYkNvcmUvc3RvcmFnZS9EYXRh
YmFzZVRyYWNrZXIuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC0xNzcsNiArMTc3LDcgQEAgYm9vbCBE
YXRhYmFzZVRyYWNrZXI6OmNhbkVzdGFibGlzaERhdGFiYQogICAgIGlmIChyZXF1aXJlbWVudCA8
PSBxdW90YUZvck9yaWdpbihvcmlnaW4pKQogICAgICAgICByZXR1cm4gdHJ1ZTsKIAorICAgIE11
dGV4TG9ja2VyIGxvY2tEYXRhYmFzZShtX2RhdGFiYXNlR3VhcmQpOwogICAgIGRvbmVDcmVhdGlu
Z0RhdGFiYXNlKG9yaWdpbiwgbmFtZSk7CiAgICAgcmV0dXJuIGZhbHNlOwogfQpJbmRleDogTGF5
b3V0VGVzdHMvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIExheW91dFRlc3RzL0NoYW5nZUxvZwko
cmV2aXNpb24gNTYzNjMpCisrKyBMYXlvdXRUZXN0cy9DaGFuZ2VMb2cJKHdvcmtpbmcgY29weSkK
QEAgLTEsMyArMSwxNSBAQAorMjAxMC0wMy0yMiAgRXJpYyBVaHJoYW5lICA8ZXJpY3VAY2hyb21p
dW0ub3JnPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAg
IE1pc3NpbmcgbG9jayBpbiBjYWxsIHRvIGRvbmVDcmVhdGluZ0RhdGFiYXNlCisgICAgICAgIGh0
dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0zNjQ3MworCisgICAgICAgIEFk
ZGVkIG5ldyB0ZXN0IHN0b3JhZ2Uvb3Blbi1kYXRhYmFzZS1vdmVyLXF1b3RhLmh0bWwuCisKKyAg
ICAgICAgKiBzdG9yYWdlL29wZW4tZGF0YWJhc2Utb3Zlci1xdW90YS5odG1sOiBBZGRlZC4KKyAg
ICAgICAgKiBzdG9yYWdlL29wZW4tZGF0YWJhc2Utb3Zlci1xdW90YS1leHBlY3RlZC50eHQ6IEFk
ZGVkLgorCiAyMDEwLTAzLTIyICBLZW5uZXRoIFJvaGRlIENocmlzdGlhbnNlbiAgPGtlbm5ldGhA
d2Via2l0Lm9yZz4KIAogICAgICAgICBVbnJldmlld2VkLgpJbmRleDogTGF5b3V0VGVzdHMvc3Rv
cmFnZS9vcGVuLWRhdGFiYXNlLW92ZXItcXVvdGEtZXhwZWN0ZWQudHh0Cj09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0t
IExheW91dFRlc3RzL3N0b3JhZ2Uvb3Blbi1kYXRhYmFzZS1vdmVyLXF1b3RhLWV4cGVjdGVkLnR4
dAkocmV2aXNpb24gMCkKKysrIExheW91dFRlc3RzL3N0b3JhZ2Uvb3Blbi1kYXRhYmFzZS1vdmVy
LXF1b3RhLWV4cGVjdGVkLnR4dAkocmV2aXNpb24gMCkKQEAgLTAsMCArMSwyIEBACitUaGlzIHRl
c3RzIHRoYXQgY2FsbGluZyBvcGVuRGF0YWJhc2Ugd2l0aCBhIHNpemUgb3ZlciA1TUIgZG9lc24n
dCBhc3NlcnQgb24gZGVidWcgYnVpbGRzLgorUEFTUwpJbmRleDogTGF5b3V0VGVzdHMvc3RvcmFn
ZS9vcGVuLWRhdGFiYXNlLW92ZXItcXVvdGEuaHRtbAo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0
cy9zdG9yYWdlL29wZW4tZGF0YWJhc2Utb3Zlci1xdW90YS5odG1sCShyZXZpc2lvbiAwKQorKysg
TGF5b3V0VGVzdHMvc3RvcmFnZS9vcGVuLWRhdGFiYXNlLW92ZXItcXVvdGEuaHRtbAkocmV2aXNp
b24gMCkKQEAgLTAsMCArMSwyNCBAQAorPGh0bWw+Cis8aGVhZD4KKzxzY3JpcHQ+CitmdW5jdGlv
biBydW5UZXN0KCkgeworICAgIGlmICh3aW5kb3cubGF5b3V0VGVzdENvbnRyb2xsZXIpIHsKKyAg
ICAgICAgbGF5b3V0VGVzdENvbnRyb2xsZXIuZHVtcEFzVGV4dCgpOworICAgICAgICBsYXlvdXRU
ZXN0Q29udHJvbGxlci5jbGVhckFsbERhdGFiYXNlcygpOworICAgIH0KKyAgICAgICAgCisgICAg
CisgICAgdmFyIGRiID0gb3BlbkRhdGFiYXNlKCdPdmVyUXVvdGFPcGVuJywgJycsICdUZXN0IGZv
ciBidWcgMzY0NzM6IG1pc3NpbmcgbG9jayBpbiBjYWxsIHRvIGRvbmVDcmVhdGluZ0RhdGFiYXNl
JywgMTAwMDAwMDApOworICAgIGlmICghZGIpIHsKKyAgICAgICAgZG9jdW1lbnQuZ2V0RWxlbWVu
dEJ5SWQoJ3Jlc3VsdCcpLmlubmVySFRNTCA9ICdQQVNTJworICAgIH0KK30KKzwvc2NyaXB0Pgor
PC9oZWFkPgorPGJvZHkgb25sb2FkPSJydW5UZXN0KCkiPgorPGRpdj5UaGlzIHRlc3RzIHRoYXQg
Y2FsbGluZyBvcGVuRGF0YWJhc2Ugd2l0aCBhIHNpemUgb3ZlciA1TUIgZG9lc24ndCBhc3NlcnQg
b24gZGVidWcgYnVpbGRzLgorPGRpdiBpZD0icmVzdWx0Ij4KK0ZBSUw6IFdlIHNob3VsZG4ndCBo
YXZlIGJlZW4gYWJsZSB0byBvcGVuIHRoZSBkYXRhYmFzZS4KKzwvZGl2PgorPC9ib2R5PgorPC9o
dG1sPgo=
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>51682</attachid>
            <date>2010-03-25 15:13:57 -0700</date>
            <delta_ts>2010-03-25 23:06:24 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-36473-20100325151356.patch</filename>
            <type>text/plain</type>
            <size>3544</size>
            <attacher name="Eric U.">ericu</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n
CShyZXZpc2lvbiA1NjM3MykKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA
IC0xLDMgKzEsMTUgQEAKKzIwMTAtMDMtMjIgIEVyaWMgVWhyaGFuZSAgPGVyaWN1QGNocm9taXVt
Lm9yZz4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBN
aXNzaW5nIGxvY2sgaW4gY2FsbCB0byBkb25lQ3JlYXRpbmdEYXRhYmFzZQorICAgICAgICBodHRw
czovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MzY0NzMKKworICAgICAgICBBZGRl
ZCBuZXcgdGVzdCBzdG9yYWdlL29wZW4tZGF0YWJhc2Utb3Zlci1xdW90YS5odG1sLgorCisgICAg
ICAgICogc3RvcmFnZS9EYXRhYmFzZVRyYWNrZXIuY3BwOgorICAgICAgICAoV2ViQ29yZTo6RGF0
YWJhc2VUcmFja2VyOjpjYW5Fc3RhYmxpc2hEYXRhYmFzZSk6ICBBZGRlZCBtaXNzaW5nIGxvY2sg
Y2FsbC4KKwogMjAxMC0wMy0yMiAgRGFyaW4gRmlzaGVyICA8ZGFyaW5AY2hyb21pdW0ub3JnPgog
CiAgICAgICAgIFJldmlld2VkIGJ5IEJyYWR5IEVpZHNvbi4KSW5kZXg6IFdlYkNvcmUvc3RvcmFn
ZS9EYXRhYmFzZVRyYWNrZXIuY3BwCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvc3RvcmFnZS9EYXRh
YmFzZVRyYWNrZXIuY3BwCShyZXZpc2lvbiA1NjM2MykKKysrIFdlYkNvcmUvc3RvcmFnZS9EYXRh
YmFzZVRyYWNrZXIuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC0xNjgsMTYgKzE2OCwxNyBAQCBib29s
IERhdGFiYXNlVHJhY2tlcjo6Y2FuRXN0YWJsaXNoRGF0YWJhCiAgICAgfQogICAgIC8vIERyb3Ag
YWxsIGxvY2tzIGJlZm9yZSBjYWxsaW5nIG91dDsgd2UgZG9uJ3Qga25vdyB3aGF0IHRoZXknbGwg
ZG8uCiAgICAgY29udGV4dC0+ZGF0YWJhc2VFeGNlZWRlZFF1b3RhKG5hbWUpOwotICAgIHsKLSAg
ICAgICAgTXV0ZXhMb2NrZXIgbG9ja0RhdGFiYXNlKG1fZGF0YWJhc2VHdWFyZCk7Ci0gICAgICAg
IG1fcHJvcG9zZWREYXRhYmFzZXMucmVtb3ZlKCZkZXRhaWxzKTsKLSAgICB9CisKKyAgICBNdXRl
eExvY2tlciBsb2NrRGF0YWJhc2UobV9kYXRhYmFzZUd1YXJkKTsKKworICAgIG1fcHJvcG9zZWRE
YXRhYmFzZXMucmVtb3ZlKCZkZXRhaWxzKTsKIAogICAgIC8vIElmIHRoZSBkYXRhYmFzZSB3aWxs
IGZpdCBub3csIGFsbG93IGl0cyBjcmVhdGlvbi4KLSAgICBpZiAocmVxdWlyZW1lbnQgPD0gcXVv
dGFGb3JPcmlnaW4ob3JpZ2luKSkKKyAgICBpZiAocmVxdWlyZW1lbnQgPD0gcXVvdGFGb3JPcmln
aW5Ob0xvY2sob3JpZ2luKSkKICAgICAgICAgcmV0dXJuIHRydWU7CiAKICAgICBkb25lQ3JlYXRp
bmdEYXRhYmFzZShvcmlnaW4sIG5hbWUpOworCiAgICAgcmV0dXJuIGZhbHNlOwogfQogCkluZGV4
OiBMYXlvdXRUZXN0cy9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gTGF5b3V0VGVzdHMvQ2hhbmdl
TG9nCShyZXZpc2lvbiA1NjM2MykKKysrIExheW91dFRlc3RzL0NoYW5nZUxvZwkod29ya2luZyBj
b3B5KQpAQCAtMSwzICsxLDE1IEBACisyMDEwLTAzLTIyICBFcmljIFVocmhhbmUgIDxlcmljdUBj
aHJvbWl1bS5vcmc+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAg
ICAgICAgTWlzc2luZyBsb2NrIGluIGNhbGwgdG8gZG9uZUNyZWF0aW5nRGF0YWJhc2UKKyAgICAg
ICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTM2NDczCisKKyAgICAg
ICAgQWRkZWQgbmV3IHRlc3Qgc3RvcmFnZS9vcGVuLWRhdGFiYXNlLW92ZXItcXVvdGEuaHRtbC4K
KworICAgICAgICAqIHN0b3JhZ2Uvb3Blbi1kYXRhYmFzZS1vdmVyLXF1b3RhLmh0bWw6IEFkZGVk
LgorICAgICAgICAqIHN0b3JhZ2Uvb3Blbi1kYXRhYmFzZS1vdmVyLXF1b3RhLWV4cGVjdGVkLnR4
dDogQWRkZWQuCisKIDIwMTAtMDMtMjIgIEtlbm5ldGggUm9oZGUgQ2hyaXN0aWFuc2VuICA8a2Vu
bmV0aEB3ZWJraXQub3JnPgogCiAgICAgICAgIFVucmV2aWV3ZWQuCkluZGV4OiBMYXlvdXRUZXN0
cy9zdG9yYWdlL29wZW4tZGF0YWJhc2Utb3Zlci1xdW90YS1leHBlY3RlZC50eHQKPT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PQotLS0gTGF5b3V0VGVzdHMvc3RvcmFnZS9vcGVuLWRhdGFiYXNlLW92ZXItcXVvdGEtZXhwZWN0
ZWQudHh0CShyZXZpc2lvbiAwKQorKysgTGF5b3V0VGVzdHMvc3RvcmFnZS9vcGVuLWRhdGFiYXNl
LW92ZXItcXVvdGEtZXhwZWN0ZWQudHh0CShyZXZpc2lvbiAwKQpAQCAtMCwwICsxLDIgQEAKK1Ro
aXMgdGVzdHMgdGhhdCBjYWxsaW5nIG9wZW5EYXRhYmFzZSB3aXRoIGEgc2l6ZSBvdmVyIDVNQiBk
b2Vzbid0IGFzc2VydCBvbiBkZWJ1ZyBidWlsZHMuCitQQVNTCkluZGV4OiBMYXlvdXRUZXN0cy9z
dG9yYWdlL29wZW4tZGF0YWJhc2Utb3Zlci1xdW90YS5odG1sCj09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIExheW91
dFRlc3RzL3N0b3JhZ2Uvb3Blbi1kYXRhYmFzZS1vdmVyLXF1b3RhLmh0bWwJKHJldmlzaW9uIDAp
CisrKyBMYXlvdXRUZXN0cy9zdG9yYWdlL29wZW4tZGF0YWJhc2Utb3Zlci1xdW90YS5odG1sCShy
ZXZpc2lvbiAwKQpAQCAtMCwwICsxLDI0IEBACis8aHRtbD4KKzxoZWFkPgorPHNjcmlwdD4KK2Z1
bmN0aW9uIHJ1blRlc3QoKSB7CisgICAgaWYgKHdpbmRvdy5sYXlvdXRUZXN0Q29udHJvbGxlcikg
eworICAgICAgICBsYXlvdXRUZXN0Q29udHJvbGxlci5kdW1wQXNUZXh0KCk7CisgICAgICAgIGxh
eW91dFRlc3RDb250cm9sbGVyLmNsZWFyQWxsRGF0YWJhc2VzKCk7CisgICAgfQorICAgICAgICAK
KyAgICAKKyAgICB2YXIgZGIgPSBvcGVuRGF0YWJhc2UoJ092ZXJRdW90YU9wZW4nLCAnJywgJ1Rl
c3QgZm9yIGJ1ZyAzNjQ3MzogbWlzc2luZyBsb2NrIGluIGNhbGwgdG8gZG9uZUNyZWF0aW5nRGF0
YWJhc2UnLCAxMDAwMDAwMCk7CisgICAgaWYgKCFkYikgeworICAgICAgICBkb2N1bWVudC5nZXRF
bGVtZW50QnlJZCgncmVzdWx0JykuaW5uZXJIVE1MID0gJ1BBU1MnCisgICAgfQorfQorPC9zY3Jp
cHQ+Cis8L2hlYWQ+Cis8Ym9keSBvbmxvYWQ9InJ1blRlc3QoKSI+Cis8ZGl2PlRoaXMgdGVzdHMg
dGhhdCBjYWxsaW5nIG9wZW5EYXRhYmFzZSB3aXRoIGEgc2l6ZSBvdmVyIDVNQiBkb2Vzbid0IGFz
c2VydCBvbiBkZWJ1ZyBidWlsZHMuCis8ZGl2IGlkPSJyZXN1bHQiPgorRkFJTDogV2Ugc2hvdWxk
bid0IGhhdmUgYmVlbiBhYmxlIHRvIG9wZW4gdGhlIGRhdGFiYXNlLgorPC9kaXY+Cis8L2JvZHk+
Cis8L2h0bWw+Cg==
</data>

          </attachment>
      

    </bug>

</bugzilla>