<?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>6796</bug_id>
          
          <creation_ts>2006-01-25 11:03:13 -0800</creation_ts>
          <short_desc>Reproducible CSS Crasher (infinite recursion adding/removing scrollbar)</short_desc>
          <delta_ts>2007-04-28 22:31:17 -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>Layout and Rendering</component>
          <version>417.x</version>
          <rep_platform>Mac</rep_platform>
          <op_sys>OS X 10.4</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>HasReduction, InRadar</keywords>
          <priority>P1</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Eric Seidel (no email)">eric</reporter>
          <assigned_to name="Beth Dakin">bdakin</assigned_to>
          <cc>bdakin</cc>
    
    <cc>hyatt</cc>
    
    <cc>joost</cc>
    
    <cc>mitz</cc>
    
    <cc>mjs</cc>
    
    <cc>samuel.sidler</cc>
    
    <cc>webkitBugzilla</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>29870</commentid>
    <comment_count>0</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2006-01-25 11:03:13 -0800</bug_when>
    <thetext>Simple reduction attached.

Also here:

&lt;style&gt;
body{
    font-size:11px;
    line-height:127%
}
div{
    float:left;
    overflow:auto
}
div label{
    float:left;
    font-size:110%
}
div span{
    float:right;
}
&lt;/style&gt; 
&lt;div&gt;
    &lt;label&gt;foo&lt;/label&gt;
    &lt;span&gt;bar&lt;/span&gt;
&lt;/div&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>29871</commentid>
    <comment_count>1</comment_count>
      <attachid>5952</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2006-01-25 11:03:38 -0800</bug_when>
    <thetext>Created attachment 5952
Simple Test Case (causes crash)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>29872</commentid>
    <comment_count>2</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2006-01-25 11:04:06 -0800</bug_when>
    <thetext>This is in Radar as &lt;rdar://problem/4406377&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>29873</commentid>
    <comment_count>3</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2006-01-25 11:11:04 -0800</bug_when>
    <thetext>Crash Report:

It&apos;s a recursive loop.  We&apos;re blowing out the stack.

Exception:  EXC_BAD_ACCESS (0x0001)
Codes:      KERN_INVALID_ADDRESS (0x0001) at 0xbf7ffff0

Thread 0 Crashed:
0   com.apple.CoreFoundation 	0x90740b80 CFRetain + 24
1   com.apple.CoreFoundation 	0x907405c0 _CFArrayReplaceValues + 372
2   com.apple.CoreFoundation 	0x90740438 CFArrayAppendValue + 200
3   com.apple.AppKit         	0x93695748 -[NSView _addSubview:] + 124
4   com.apple.AppKit         	0x936949f0 -[NSView _setSuperview:] + 320
5   com.apple.AppKit         	0x936954f0 -[NSView addSubview:] + 352
6   com.apple.WebKit         	0x0127bac4 -[WebHTMLView addSubview:] + 96 (WebHTMLView.m:2237)
7   com.apple.WebCore        	0x01d025ec QWidget::addToSuperview(NSView*) + 532 (KWQWidget.mm:530)
8   com.apple.WebCore        	0x01cdfdb8 QScrollView::addChild(QWidget*, int, int) + 528 (KWQScrollView.mm:360)
9   com.apple.WebCore        	0x02009d00 WebCore::RenderLayer::setHasVerticalScrollbar(bool) + 196 (render_layer.cpp:743)
10  com.apple.WebCore        	0x0200a1b8 WebCore::RenderLayer::updateScrollInfoAfterLayout() + 856 (render_layer.cpp:883)
11  com.apple.WebCore        	0x01fd3b6c WebCore::RenderBlock::layoutBlock(bool) + 3648 (RenderBlock.cpp:546)
12  com.apple.WebCore        	0x0200a2cc WebCore::RenderLayer::updateScrollInfoAfterLayout() + 1132 (render_layer.cpp:893)
13  com.apple.WebCore        	0x01fd3b6c WebCore::RenderBlock::layoutBlock(bool) + 3648 (RenderBlock.cpp:546)
14  com.apple.WebCore        	0x0200a2cc WebCore::RenderLayer::updateScrollInfoAfterLayout() + 1132 (render_layer.cpp:893)
15  com.apple.WebCore        	0x01fd3b6c WebCore::RenderBlock::layoutBlock(bool) + 3648 (RenderBlock.cpp:546)
16  com.apple.WebCore        	0x0200a2cc WebCore::RenderLayer::updateScrollInfoAfterLayout() + 1132 (render_layer.cpp:893)
17  com.apple.WebCore        	0x01fd3b6c WebCore::RenderBlock::layoutBlock(bool) + 3648 (RenderBlock.cpp:546)
18  com.apple.WebCore        	0x0200a2cc WebCore::RenderLayer::updateScrollInfoAfterLayout() + 1132 (render_layer.cpp:893)
19  com.apple.WebCore        	0x01fd3b6c WebCore::RenderBlock::layoutBlock(bool) + 3648 (RenderBlock.cpp:546)
20  com.apple.WebCore        	0x0200a2cc WebCore::RenderLayer::updateScrollInfoAfterLayout() + 1132 (render_layer.cpp:893)
21  com.apple.WebCore        	0x01fd3b6c WebCore::RenderBlock::layoutBlock(bool) + 3648 (RenderBlock.cpp:546)
22  com.apple.WebCore        	0x0200a2cc WebCore::RenderLayer::updateScrollInfoAfterLayout() + 1132 (render_layer.cpp:893)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>29880</commentid>
    <comment_count>4</comment_count>
      <attachid>5953</attachid>
    <who name="Joost de Valk (AlthA)">joost</who>
    <bug_when>2006-01-25 11:23:33 -0800</bug_when>
    <thetext>Created attachment 5953
Reduced Testcase

Reduced it a bit more. Reducing the line-height percentage or font-size more, keeps the crash from occurring.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>29919</commentid>
    <comment_count>5</comment_count>
    <who name="">mitz</who>
    <bug_when>2006-01-25 15:08:57 -0800</bug_when>
    <thetext>The &lt;label&gt;, which is a float, has height 19 and overflow height 20.

The &lt;div&gt; grows to contain only 19 pixels, at which point it decides that it needs a vertical scrollbar. That leaves not enough room for the &lt;span&gt;, which makes it move to the next line, but then, the float&apos;s height discrepancy is no longer an issue, the &lt;div&gt; no longer needs that scrollbar, the &lt;span&gt; can move up again, and on and on it goes...

I think the problem is right there at the beginning, when the &lt;div&gt; uses the FloatingObject&apos;s endY, which is based on height, rather than something that takes overflow height into account.

Still no idea what to fix, let alone how.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>29920</commentid>
    <comment_count>6</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2006-01-25 15:11:13 -0800</bug_when>
    <thetext>We need some hyatt here.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>29933</commentid>
    <comment_count>7</comment_count>
    <who name="">mitz</who>
    <bug_when>2006-01-25 15:55:01 -0800</bug_when>
    <thetext>My last comment was wrong. Checking with Firefox, it looks like there shouldn&apos;t be a vertical scrollbar in the first place.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>35427</commentid>
    <comment_count>8</comment_count>
    <who name="Darin Adler">darin</who>
    <bug_when>2006-03-07 09:01:10 -0800</bug_when>
    <thetext>*** Bug 7631 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>35428</commentid>
    <comment_count>9</comment_count>
    <who name="Darin Adler">darin</who>
    <bug_when>2006-03-07 09:01:47 -0800</bug_when>
    <thetext>Another test case in bug 7631. Lets make sure both of these test cases are fixed.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>35846</commentid>
    <comment_count>10</comment_count>
      <attachid>7001</attachid>
    <who name="Beth Dakin">bdakin</who>
    <bug_when>2006-03-10 20:10:36 -0800</bug_when>
    <thetext>Created attachment 7001
First attempt at halting infinte recursion

I am not sure if this is exactly right, but it fixes the test case and doesn&apos;t break any layout tests. Thoughts?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>35878</commentid>
    <comment_count>11</comment_count>
    <who name="Maciej Stachowiak">mjs</who>
    <bug_when>2006-03-11 02:09:13 -0800</bug_when>
    <thetext>Looks good to me. Make sure to try both test cases, and to add them as layout tests. I&apos;ll give hyatt a chance to give it the once over.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>36146</commentid>
    <comment_count>12</comment_count>
      <attachid>7001</attachid>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2006-03-13 14:20:51 -0800</bug_when>
    <thetext>Comment on attachment 7001
First attempt at halting infinte recursion

Move the variables into RenderLayer and try to keep as much of the scrolling logic there as you can.

I&apos;d try to make the min/max width modification only be done if the infinite recursion case was hit.  I think it would also be fine to simply not have this code in an initial check-in as well.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>36155</commentid>
    <comment_count>13</comment_count>
      <attachid>7053</attachid>
    <who name="Beth Dakin">bdakin</who>
    <bug_when>2006-03-13 16:34:42 -0800</bug_when>
    <thetext>Created attachment 7053
New patch

After talking with Hyatt, I decided not to mess with the min and max width in this check-in. Here is a new patch!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>36158</commentid>
    <comment_count>14</comment_count>
      <attachid>7053</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2006-03-13 16:58:27 -0800</bug_when>
    <thetext>Comment on attachment 7053
New patch

There&apos;s one major thing I don&apos;t understand: When does anything ever reset sawHorizontalBar or sawVerticalBar back to false? What happens if the content get smaller and doesn&apos;t need a horizontal scrollbar any more? Does the layer get destroyed and re-created in that case?

+    bool sawHorizontalBar() { return m_sawHorizontalBar; }
+    bool sawVerticalBar() { return m_sawVerticalBar; }

Those should be const member functions.

+    if (needHorizontalBar)
+        setSawHorizontalBar(needHorizontalBar);

Could just be setSawHorizontalBar(true);

+    if (needVerticalBar)
+        setSawVerticalBar(needVerticalBar);

Same here.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>36159</commentid>
    <comment_count>15</comment_count>
      <attachid>7053</attachid>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2006-03-13 17:13:35 -0800</bug_when>
    <thetext>Comment on attachment 7053
New patch

r=me.  Make sure to run layout tests before landing (and to add a layout test for this problem).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>36160</commentid>
    <comment_count>16</comment_count>
      <attachid>7053</attachid>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2006-03-13 17:17:15 -0800</bug_when>
    <thetext>Comment on attachment 7053
New patch

Yeah, Darin is right.  I have an idea for how this can be adjusted.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>36161</commentid>
    <comment_count>17</comment_count>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2006-03-13 17:20:46 -0800</bug_when>
    <thetext>I think you can put a guard around the spot in RenderLayer that does a re-layout when overflow is AUTO.  A single bit re-entrancy guard on the layer should do it.
</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>36318</commentid>
    <comment_count>18</comment_count>
      <attachid>7062</attachid>
    <who name="Beth Dakin">bdakin</who>
    <bug_when>2006-03-14 14:00:58 -0800</bug_when>
    <thetext>Created attachment 7062
Single-bit guard around layoutBlock() call

Like this, Dave? In the specific test case for this bug, this patch prevents the crash but yields a nasty layout. &quot;foobar&quot; is on one line and a scroll bar is drawn over most of the &quot;bar&quot; as if &quot;foo&quot; and &quot;bar&quot; were stacked. It is worth noting, though, that some of the other sites that we know of that crash in this same way (like the one in the bug that is marked as a dup of this bugzilla) no longer crash with this patch and lay out just fine.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>36319</commentid>
    <comment_count>19</comment_count>
      <attachid>7063</attachid>
    <who name="Beth Dakin">bdakin</who>
    <bug_when>2006-03-14 14:15:40 -0800</bug_when>
    <thetext>Created attachment 7063
no this one!

Okay, newer patch with some Hyatt-comments from IRC taken into account. This one doesn&apos;t yield a nasty layout either. yay!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>36320</commentid>
    <comment_count>20</comment_count>
      <attachid>7063</attachid>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2006-03-14 14:18:58 -0800</bug_when>
    <thetext>Comment on attachment 7063
no this one!

r=me, be sure to file a followup bug about possibly adjusting min/maxwidth.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>36337</commentid>
    <comment_count>21</comment_count>
    <who name="Beth Dakin">bdakin</who>
    <bug_when>2006-03-14 16:03:57 -0800</bug_when>
    <thetext>I filed http://bugzilla.opendarwin.org/show_bug.cgi?id=7768 to represent the min/max_width problem.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>40541</commentid>
    <comment_count>22</comment_count>
    <who name="Marcus Pallinger">webkitBugzilla</who>
    <bug_when>2006-04-26 15:42:20 -0700</bug_when>
    <thetext>Has this been merged into the Safari-2-0 branch, given that this crash also occurs in Safari 2.0.3?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>22758</commentid>
    <comment_count>23</comment_count>
    <who name="">mitz</who>
    <bug_when>2007-02-19 23:16:55 -0800</bug_when>
    <thetext>*** Bug 12679 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>11767</commentid>
    <comment_count>24</comment_count>
    <who name="">mitz</who>
    <bug_when>2007-04-28 22:31:17 -0700</bug_when>
    <thetext>*** Bug 13536 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="0"
              isprivate="0"
          >
            <attachid>5952</attachid>
            <date>2006-01-25 11:03:38 -0800</date>
            <delta_ts>2006-01-25 11:23:33 -0800</delta_ts>
            <desc>Simple Test Case (causes crash)</desc>
            <filename>repro-crash.html</filename>
            <type>text/html</type>
            <size>242</size>
            <attacher name="Eric Seidel (no email)">eric</attacher>
            
              <data encoding="base64">PHN0eWxlPgpib2R5ewogICAgZm9udC1zaXplOjExcHg7CiAgICBsaW5lLWhlaWdodDoxMjclCn0K
ZGl2ewogICAgZmxvYXQ6bGVmdDsKICAgIG92ZXJmbG93OmF1dG8KfQpkaXYgbGFiZWx7CiAgICBm
bG9hdDpsZWZ0OwogICAgZm9udC1zaXplOjExMCUKfQpkaXYgc3BhbnsKICAgIGZsb2F0OnJpZ2h0
Owp9Cjwvc3R5bGU+IAo8ZGl2PgogICAgPGxhYmVsPmZvbzwvbGFiZWw+CiAgICA8c3Bhbj5iYXI8
L3NwYW4+CjwvZGl2Pgo=
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="0"
              isprivate="0"
          >
            <attachid>5953</attachid>
            <date>2006-01-25 11:23:33 -0800</date>
            <delta_ts>2006-01-25 11:23:33 -0800</delta_ts>
            <desc>Reduced Testcase</desc>
            <filename>repro-crash.html</filename>
            <type>text/html</type>
            <size>193</size>
            <attacher name="Joost de Valk (AlthA)">joost</attacher>
            
              <data encoding="base64">PHN0eWxlPgpib2R5ewogICAgbGluZS1oZWlnaHQ6MTIxJQp9CmRpdnsKICAgIGZsb2F0OmxlZnQ7
CiAgICBvdmVyZmxvdzphdXRvCn0KZGl2IGxhYmVsewogICAgZmxvYXQ6bGVmdDsKICAgIGZvbnQt
c2l6ZToxMDQlCn0KPC9zdHlsZT4gCjxkaXY+CiAgICA8bGFiZWw+Zm9vPC9sYWJlbD4KICAgIDxz
cGFuPmJhcjwvc3Bhbj4KPC9kaXY+Cg==
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>7001</attachid>
            <date>2006-03-10 20:10:36 -0800</date>
            <delta_ts>2006-03-13 16:34:42 -0800</delta_ts>
            <desc>First attempt at halting infinte recursion</desc>
            <filename>scrollers.txt</filename>
            <type>text/plain</type>
            <size>4273</size>
            <attacher name="Beth Dakin">bdakin</attacher>
            
              <data encoding="base64">SW5kZXg6IHJlbmRlcmluZy9SZW5kZXJCbG9jay5oCj09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIHJlbmRlcmluZy9S
ZW5kZXJCbG9jay5oCShyZXZpc2lvbiAxMzI0NSkKKysrIHJlbmRlcmluZy9SZW5kZXJCbG9jay5o
CSh3b3JraW5nIGNvcHkpCkBAIC02Nyw2ICs2NywxMCBAQAogICAgIHZpcnR1YWwgSW50UmVjdCBv
dmVyZmxvd1JlY3QoYm9vbCBpbmNsdWRlSW50ZXJpb3I9dHJ1ZSkgY29uc3Q7CiAgICAgdmlydHVh
bCB2b2lkIHNldE92ZXJmbG93SGVpZ2h0KGludCBoKSB7IG1fb3ZlcmZsb3dIZWlnaHQgPSBoOyB9
CiAgICAgdmlydHVhbCB2b2lkIHNldE92ZXJmbG93V2lkdGgoaW50IHcpIHsgbV9vdmVyZmxvd1dp
ZHRoID0gdzsgfQorICAgIHZpcnR1YWwgYm9vbCBuZWVkc0hvcml6b250YWxCYXIoKSB7IHJldHVy
biBtX25lZWRzSG9yaXpvbnRhbEJhcjsgfQorICAgIHZpcnR1YWwgYm9vbCBuZWVkc1ZlcnRpY2Fs
QmFyKCkgeyByZXR1cm4gbV9uZWVkc1ZlcnRpY2FsQmFyOyB9CisgICAgdmlydHVhbCB2b2lkIHNl
dE5lZWRzSG9yaXpvbnRhbEJhcihib29sIG5lZWRzSG9yaXpvbnRhbCkgeyBtX25lZWRzSG9yaXpv
bnRhbEJhciA9IG5lZWRzSG9yaXpvbnRhbDsgfQorICAgIHZpcnR1YWwgdm9pZCBzZXROZWVkc1Zl
cnRpY2FsQmFyKGJvb2wgbmVlZHNWZXJ0aWNhbCkgeyBtX25lZWRzVmVydGljYWxCYXIgPSBuZWVk
c1ZlcnRpY2FsOyB9CiAgICAgCiAgICAgdmlydHVhbCBib29sIGlzU2VsZkNvbGxhcHNpbmdCbG9j
aygpIGNvbnN0OwogICAgIHZpcnR1YWwgYm9vbCBpc1RvcE1hcmdpblF1aXJrKCkgY29uc3QgeyBy
ZXR1cm4gbV90b3BNYXJnaW5RdWlyazsgfQpAQCAtNDI3LDYgKzQzMSw5IEBACiAgICAgaW50IG1f
b3ZlcmZsb3dIZWlnaHQ7CiAgICAgaW50IG1fb3ZlcmZsb3dXaWR0aDsKICAgICAKKyAgICBib29s
IG1fbmVlZHNIb3Jpem9udGFsQmFyIDogMTsKKyAgICBib29sIG1fbmVlZHNWZXJ0aWNhbEJhciA6
IDE7CisgICAgCiAgICAgLy8gTGVmdCBhbmQgdG9wIG92ZXJmbG93LiAgRG9lcyBub3QgYWZmZWN0
IHNjcm9sbGluZyBkaW1lbnNpb25zLCBidXQgd2UgZG8gYXQgbGVhc3QgdXNlIGl0CiAgICAgLy8g
d2hlbiBkaXJ0eSByZWN0IGNoZWNraW5nIGFuZCBoaXQgdGVzdGluZy4KICAgICBpbnQgbV9vdmVy
Zmxvd0xlZnQ7CkluZGV4OiByZW5kZXJpbmcvcmVuZGVyX2xheWVyLmNwcAo9PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0t
LSByZW5kZXJpbmcvcmVuZGVyX2xheWVyLmNwcAkocmV2aXNpb24gMTMyNDUpCisrKyByZW5kZXJp
bmcvcmVuZGVyX2xheWVyLmNwcAkod29ya2luZyBjb3B5KQpAQCAtODUyLDggKzg1MiwyMyBAQAog
ICAgICAgICByZXR1cm47IC8vIEFsbCB3ZSBoYWQgdG8gZG8gd2FzIGRpcnR5LgogCiAgICAgYm9v
bCBuZWVkSG9yaXpvbnRhbEJhciwgbmVlZFZlcnRpY2FsQmFyOworICAgIGJvb2wgYWxyZWFkeVNl
dEhvcml6b250YWxCYXIgPSBtX29iamVjdC0+bmVlZHNIb3Jpem9udGFsQmFyKCk7CisgICAgYm9v
bCBhbHJlYWR5U2V0VmVydGljYWxCYXIgPSBtX29iamVjdC0+bmVlZHNWZXJ0aWNhbEJhcigpOwog
ICAgIGNvbXB1dGVTY3JvbGxEaW1lbnNpb25zKCZuZWVkSG9yaXpvbnRhbEJhciwgJm5lZWRWZXJ0
aWNhbEJhcik7CiAKKyAgICAvLyBJZiB3ZSBqdXN0IGRldGVybWluZWQgdGhhdCB3ZSBuZWVkIGVp
dGhlciBzY3JvbGwgYmFyLCB3ZSBzZXQgdGhlIGFwcHJvcHJpYXRlIG5lZWRzIAorICAgIC8vIHNj
cm9sbCBiYXIgZmxhZyBmb3IgbV9vYmplY3QuIElmIHdlIGp1c3QgZGV0ZXJtaW5lZCB0aGF0IHdl
IGRvIG5vdCBuZWVkIGEgc2Nyb2xsIAorICAgIC8vIGJhciwgYnV0IGluIGEgcHJldmlvdXMgcnVu
IHdlIGRlY2lkZWQgdGhhdCB3ZSBkaWQsIHRoZW4ga2VlcCB0aGVtIHNldCB0byBhdm9pZCAKKyAg
ICAvLyBpbmZpbml0ZSByZWN1cnNpb24uCisgICAgaWYgKG5lZWRIb3Jpem9udGFsQmFyKQorICAg
ICAgICBtX29iamVjdC0+c2V0TmVlZHNIb3Jpem9udGFsQmFyKG5lZWRIb3Jpem9udGFsQmFyKTsK
KyAgICBlbHNlIGlmIChhbHJlYWR5U2V0SG9yaXpvbnRhbEJhcikKKyAgICAgICAgbmVlZEhvcml6
b250YWxCYXIgPSBhbHJlYWR5U2V0SG9yaXpvbnRhbEJhcjsKKyAgICBpZiAobmVlZFZlcnRpY2Fs
QmFyKQorICAgICAgICBtX29iamVjdC0+c2V0TmVlZHNWZXJ0aWNhbEJhcihuZWVkVmVydGljYWxC
YXIpOworICAgIGVsc2UgaWYgKGFscmVhZHlTZXRWZXJ0aWNhbEJhcikKKyAgICAgICAgbmVlZFZl
cnRpY2FsQmFyID0gYWxyZWFkeVNldFZlcnRpY2FsQmFyOworICAgICAgICAKICAgICBpZiAobV9v
YmplY3QtPnN0eWxlKCktPm92ZXJmbG93KCkgIT0gT01BUlFVRUUpIHsKICAgICAgICAgLy8gTGF5
b3V0IG1heSBjYXVzZSB1cyB0byBiZSBpbiBhbiBpbnZhbGlkIHNjcm9sbCBwb3NpdGlvbi4gIElu
IHRoaXMgY2FzZSB3ZSBuZWVkCiAgICAgICAgIC8vIHRvIHB1bGwgb3VyIHNjcm9sbCBvZmZzZXRz
IGJhY2sgdG8gdGhlIG1heCAob3IgcHVzaCB0aGVtIHVwIHRvIHRoZSBtaW4pLgpJbmRleDogcmVu
ZGVyaW5nL3JlbmRlcl9vYmplY3QuaAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSByZW5kZXJpbmcvcmVuZGVyX29i
amVjdC5oCShyZXZpc2lvbiAxMzI0NSkKKysrIHJlbmRlcmluZy9yZW5kZXJfb2JqZWN0LmgJKHdv
cmtpbmcgY29weSkKQEAgLTU4MSw2ICs1ODEsMTEgQEAKICAgICB2aXJ0dWFsIGludCBvdmVyZmxv
d1RvcChib29sIGluY2x1ZGVJbnRlcmlvcj10cnVlKSBjb25zdCB7IHJldHVybiAwOyB9CiAgICAg
dmlydHVhbCBJbnRSZWN0IG92ZXJmbG93UmVjdChib29sIGluY2x1ZGVJbnRlcmlvcj10cnVlKSBj
b25zdCB7IHJldHVybiBib3JkZXJCb3goKTsgfQogCisgICAgdmlydHVhbCBib29sIG5lZWRzSG9y
aXpvbnRhbEJhcigpIHsgcmV0dXJuIGZhbHNlOyB9CisgICAgdmlydHVhbCBib29sIG5lZWRzVmVy
dGljYWxCYXIoKSB7IHJldHVybiBmYWxzZTsgfQorICAgIHZpcnR1YWwgdm9pZCBzZXROZWVkc0hv
cml6b250YWxCYXIoYm9vbCkge30KKyAgICB2aXJ0dWFsIHZvaWQgc2V0TmVlZHNWZXJ0aWNhbEJh
cihib29sKSB7fQorCiAgICAgLy8gSUUgZXh0ZW5zaW9ucy4gVXNlZCB0byBjYWxjdWxhdGUgb2Zm
c2V0V2lkdGgvSGVpZ2h0LiAgT3ZlcnJpZGRlbiBieSBpbmxpbmVzIChyZW5kZXJfZmxvdykgCiAg
ICAgLy8gdG8gcmV0dXJuIHRoZSByZW1haW5pbmcgd2lkdGggb24gYSBnaXZlbiBsaW5lIChhbmQg
dGhlIGhlaWdodCBvZiBhIHNpbmdsZSBsaW5lKS4gLWR3aAogICAgIHZpcnR1YWwgaW50IG9mZnNl
dFdpZHRoKCkgY29uc3QgeyByZXR1cm4gd2lkdGgoKTsgfQpJbmRleDogcmVuZGVyaW5nL1JlbmRl
ckJsb2NrLmNwcAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09Ci0tLSByZW5kZXJpbmcvUmVuZGVyQmxvY2suY3BwCShyZXZp
c2lvbiAxMzI0NSkKKysrIHJlbmRlcmluZy9SZW5kZXJCbG9jay5jcHAJKHdvcmtpbmcgY29weSkK
QEAgLTg5LDYgKzg5LDcgQEAKICAgICBtX21heFRvcFBvc01hcmdpbiA9IG1fbWF4VG9wTmVnTWFy
Z2luID0gbV9tYXhCb3R0b21Qb3NNYXJnaW4gPSBtX21heEJvdHRvbU5lZ01hcmdpbiA9IDA7CiAg
ICAgbV90b3BNYXJnaW5RdWlyayA9IG1fYm90dG9tTWFyZ2luUXVpcmsgPSBmYWxzZTsKICAgICBt
X292ZXJmbG93SGVpZ2h0ID0gbV9vdmVyZmxvd1dpZHRoID0gMDsKKyAgICBtX25lZWRzSG9yaXpv
bnRhbEJhciA9IG1fbmVlZHNWZXJ0aWNhbEJhciA9IGZhbHNlOwogICAgIG1fb3ZlcmZsb3dMZWZ0
ID0gbV9vdmVyZmxvd1RvcCA9IDA7CiAgICAgbV90YWJXaWR0aCA9IDA7CiB9CkBAIC00NDAsNiAr
NDQxLDEyIEBACiAKICAgICBpbnQgb2xkV2lkdGggPSBtX3dpZHRoOwogICAgIAorICAgIGlmIChu
ZWVkc1ZlcnRpY2FsQmFyKCkpIHsKKyAgICAgICAgaW50IHNjcm9sbGVyV2lkdGggPSBtX2xheWVy
LT52ZXJ0aWNhbFNjcm9sbGJhcldpZHRoKCk7CisgICAgICAgIG1fbWluV2lkdGggKz0gc2Nyb2xs
ZXJXaWR0aDsKKyAgICAgICAgbV9tYXhXaWR0aCArPSBzY3JvbGxlcldpZHRoOworICAgIH0KKyAg
ICAKICAgICBjYWxjV2lkdGgoKTsKICAgICBtX292ZXJmbG93V2lkdGggPSBtX3dpZHRoOwogCg==
</data>
<flag name="review"
          id="1816"
          type_id="1"
          status="-"
          setter="hyatt"
    />
          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>7053</attachid>
            <date>2006-03-13 16:34:42 -0800</date>
            <delta_ts>2006-03-14 14:00:58 -0800</delta_ts>
            <desc>New patch</desc>
            <filename>newScrollers.txt</filename>
            <type>text/plain</type>
            <size>3473</size>
            <attacher name="Beth Dakin">bdakin</attacher>
            
              <data encoding="base64">SW5kZXg6IHJlbmRlcmluZy9yZW5kZXJfbGF5ZXIuaAo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSByZW5kZXJpbmcv
cmVuZGVyX2xheWVyLmgJKHJldmlzaW9uIDEzMjY1KQorKysgcmVuZGVyaW5nL3JlbmRlcl9sYXll
ci5oCSh3b3JraW5nIGNvcHkpCkBAIC0yNDYsNiArMjQ2LDEwIEBACiAgICAgdm9pZCB1cGRhdGVT
Y3JvbGxJbmZvQWZ0ZXJMYXlvdXQoKTsKICAgICB2b2lkIHNsb3RWYWx1ZUNoYW5nZWQoaW50KTsK
ICAgICBib29sIHNjcm9sbChLV1FTY3JvbGxEaXJlY3Rpb24gZGlyZWN0aW9uLCBLV1FTY3JvbGxH
cmFudWxhcml0eSBncmFudWxhcml0eSwgZmxvYXQgbXVsdGlwbGllcj0xLjApOworICAgIGJvb2wg
c2F3SG9yaXpvbnRhbEJhcigpIHsgcmV0dXJuIG1fc2F3SG9yaXpvbnRhbEJhcjsgfQorICAgIGJv
b2wgc2F3VmVydGljYWxCYXIoKSB7IHJldHVybiBtX3Nhd1ZlcnRpY2FsQmFyOyB9CisgICAgdm9p
ZCBzZXRTYXdIb3Jpem9udGFsQmFyKGJvb2wgbmVlZHNIb3Jpem9udGFsKSB7IG1fc2F3SG9yaXpv
bnRhbEJhciA9IG5lZWRzSG9yaXpvbnRhbDsgfQorICAgIHZvaWQgc2V0U2F3VmVydGljYWxCYXIo
Ym9vbCBuZWVkc1ZlcnRpY2FsKSB7IG1fc2F3VmVydGljYWxCYXIgPSBuZWVkc1ZlcnRpY2FsOyB9
CiAgICAgCiAgICAgdm9pZCB1cGRhdGVMYXllclBvc2l0aW9uKCk7CiAgICAgdm9pZCB1cGRhdGVM
YXllclBvc2l0aW9ucyhib29sIGRvRnVsbFJlcGFpbnQgPSBmYWxzZSwgYm9vbCBjaGVja0ZvclJl
cGFpbnQ9dHJ1ZSk7CkBAIC0zODQsNiArMzg4LDggQEAKICAgICBib29sIG1fdXNlZFRyYW5zcGFy
ZW5jeSA6IDE7IC8vIFRyYWNrcyB3aGV0aGVyIHdlIG5lZWQgdG8gY2xvc2UgYSB0cmFuc3BhcmVu
dCBsYXllciwgaS5lLiwgd2hldGhlcgogICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
Ly8gd2UgZW5kZWQgdXAgcGFpbnRpbmcgdGhpcyBsYXllciBvciBhbnkgZGVzY2VuZGFudHMgKGFu
ZCB0aGVyZWZvcmUgbmVlZCB0bwogICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgLy8g
YmxlbmQpLgorICAgIGJvb2wgbV9zYXdIb3Jpem9udGFsQmFyIDogMTsKKyAgICBib29sIG1fc2F3
VmVydGljYWxCYXIgOiAxOwogCiAgICAgTWFycXVlZSogbV9tYXJxdWVlOyAvLyBVc2VkIGJ5IGxh
eWVycyB3aXRoIG92ZXJmbG93Om1hcnF1ZWUKIH07CkluZGV4OiByZW5kZXJpbmcvcmVuZGVyX2xh
eWVyLmNwcAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09Ci0tLSByZW5kZXJpbmcvcmVuZGVyX2xheWVyLmNwcAkocmV2aXNp
b24gMTMyNjUpCisrKyByZW5kZXJpbmcvcmVuZGVyX2xheWVyLmNwcAkod29ya2luZyBjb3B5KQpA
QCAtMTA1LDMzICsxMDUsMzUgQEAKIH0KIAogUmVuZGVyTGF5ZXI6OlJlbmRlckxheWVyKFJlbmRl
ck9iamVjdCogb2JqZWN0KQotOiBtX29iamVjdCggb2JqZWN0ICksCi1tX3BhcmVudCggMCApLAot
bV9wcmV2aW91cyggMCApLAotbV9uZXh0KCAwICksCi1tX2ZpcnN0KCAwICksCi1tX2xhc3QoIDAg
KSwKLW1fcmVsWCggMCApLAotbV9yZWxZKCAwICksCi1tX3goIDAgKSwKLW1feSggMCApLAotbV93
aWR0aCggMCApLAotbV9oZWlnaHQoIDAgKSwKLW1fc2Nyb2xsWCggMCApLAotbV9zY3JvbGxZKCAw
ICksCi1tX3Njcm9sbE9yaWdpblgoIDAgKSwKLW1fc2Nyb2xsTGVmdE92ZXJmbG93KCAwICksCi1t
X3Njcm9sbFdpZHRoKCAwICksCi1tX3Njcm9sbEhlaWdodCggMCApLAotbV9oQmFyKCAwICksCi1t
X3ZCYXIoIDAgKSwKLW1fcG9zWk9yZGVyTGlzdCggMCApLAotbV9uZWdaT3JkZXJMaXN0KCAwICks
Ci1tX2NsaXBSZWN0cyggMCApICwKLW1fc2Nyb2xsRGltZW5zaW9uc0RpcnR5KCB0cnVlICksCi1t
X3pPcmRlckxpc3RzRGlydHkoIHRydWUgKSwKLW1fdXNlZFRyYW5zcGFyZW5jeSggZmFsc2UgKSwK
LW1fbWFycXVlZSggMCApCis6IG1fb2JqZWN0KG9iamVjdCksCittX3BhcmVudCgwKSwKK21fcHJl
dmlvdXMoMCksCittX25leHQoMCksCittX2ZpcnN0KDApLAorbV9sYXN0KDApLAorbV9yZWxYKDAp
LAorbV9yZWxZKDApLAorbV94KDApLAorbV95KDApLAorbV93aWR0aCgwKSwKK21faGVpZ2h0KDAp
LAorbV9zY3JvbGxYKDApLAorbV9zY3JvbGxZKDApLAorbV9zY3JvbGxPcmlnaW5YKDApLAorbV9z
Y3JvbGxMZWZ0T3ZlcmZsb3coMCksCittX3Njcm9sbFdpZHRoKDApLAorbV9zY3JvbGxIZWlnaHQo
MCksCittX2hCYXIoMCksCittX3ZCYXIoMCksCittX3Bvc1pPcmRlckxpc3QoMCksCittX25lZ1pP
cmRlckxpc3QoMCksCittX2NsaXBSZWN0cygwKSAsCittX3Njcm9sbERpbWVuc2lvbnNEaXJ0eSh0
cnVlKSwKK21fek9yZGVyTGlzdHNEaXJ0eSh0cnVlKSwKK21fdXNlZFRyYW5zcGFyZW5jeShmYWxz
ZSksCittX3Nhd0hvcml6b250YWxCYXIoZmFsc2UpLAorbV9zYXdWZXJ0aWNhbEJhcihmYWxzZSks
CittX21hcnF1ZWUoMCkKIHsKIH0KIApAQCAtODU0LDYgKzg1NiwxOSBAQAogICAgIGJvb2wgbmVl
ZEhvcml6b250YWxCYXIsIG5lZWRWZXJ0aWNhbEJhcjsKICAgICBjb21wdXRlU2Nyb2xsRGltZW5z
aW9ucygmbmVlZEhvcml6b250YWxCYXIsICZuZWVkVmVydGljYWxCYXIpOwogCisgICAgLy8gSWYg
d2UganVzdCBkZXRlcm1pbmVkIHRoYXQgd2UgbmVlZCBlaXRoZXIgc2Nyb2xsIGJhciwgd2Ugc2V0
IHRoZSBhcHByb3ByaWF0ZSBzYXcgCisgICAgLy8gc2Nyb2xsIGJhciBmbGFnLiBJZiB3ZSBqdXN0
IGRldGVybWluZWQgdGhhdCB3ZSBkbyBub3QgbmVlZCBhIHNjcm9sbCAKKyAgICAvLyBiYXIsIGJ1
dCBpbiBhIHByZXZpb3VzIHJ1biB3ZSBkZWNpZGVkIHRoYXQgd2UgZGlkLCB0aGVuIGtlZXAgdGhl
bSBzZXQgdG8gYXZvaWQgCisgICAgLy8gaW5maW5pdGUgcmVjdXJzaW9uLgorICAgIGlmIChuZWVk
SG9yaXpvbnRhbEJhcikKKyAgICAgICAgc2V0U2F3SG9yaXpvbnRhbEJhcihuZWVkSG9yaXpvbnRh
bEJhcik7CisgICAgZWxzZSBpZiAoc2F3SG9yaXpvbnRhbEJhcigpKQorICAgICAgICBuZWVkSG9y
aXpvbnRhbEJhciA9IHRydWU7CisgICAgaWYgKG5lZWRWZXJ0aWNhbEJhcikKKyAgICAgICAgc2V0
U2F3VmVydGljYWxCYXIobmVlZFZlcnRpY2FsQmFyKTsKKyAgICBlbHNlIGlmIChzYXdWZXJ0aWNh
bEJhcigpKQorICAgICAgICBuZWVkVmVydGljYWxCYXIgPSB0cnVlOworICAgICAgICAKICAgICBp
ZiAobV9vYmplY3QtPnN0eWxlKCktPm92ZXJmbG93KCkgIT0gT01BUlFVRUUpIHsKICAgICAgICAg
Ly8gTGF5b3V0IG1heSBjYXVzZSB1cyB0byBiZSBpbiBhbiBpbnZhbGlkIHNjcm9sbCBwb3NpdGlv
bi4gIEluIHRoaXMgY2FzZSB3ZSBuZWVkCiAgICAgICAgIC8vIHRvIHB1bGwgb3VyIHNjcm9sbCBv
ZmZzZXRzIGJhY2sgdG8gdGhlIG1heCAob3IgcHVzaCB0aGVtIHVwIHRvIHRoZSBtaW4pLgo=
</data>
<flag name="review"
          id="1833"
          type_id="1"
          status="-"
          setter="hyatt"
    />
          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>7062</attachid>
            <date>2006-03-14 14:00:58 -0800</date>
            <delta_ts>2006-03-14 14:15:40 -0800</delta_ts>
            <desc>Single-bit guard around layoutBlock() call</desc>
            <filename>guardAuto.txt</filename>
            <type>text/plain</type>
            <size>3063</size>
            <attacher name="Beth Dakin">bdakin</attacher>
            
              <data encoding="base64">SW5kZXg6IHJlbmRlcmluZy9yZW5kZXJfbGF5ZXIuaAo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSByZW5kZXJpbmcv
cmVuZGVyX2xheWVyLmgJKHJldmlzaW9uIDEzMjg3KQorKysgcmVuZGVyaW5nL3JlbmRlcl9sYXll
ci5oCSh3b3JraW5nIGNvcHkpCkBAIC0zODcsNiArMzg3LDcgQEAKICAgICBib29sIG1fdXNlZFRy
YW5zcGFyZW5jeSA6IDE7IC8vIFRyYWNrcyB3aGV0aGVyIHdlIG5lZWQgdG8gY2xvc2UgYSB0cmFu
c3BhcmVudCBsYXllciwgaS5lLiwgd2hldGhlcgogICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgLy8gd2UgZW5kZWQgdXAgcGFpbnRpbmcgdGhpcyBsYXllciBvciBhbnkgZGVzY2VuZGFu
dHMgKGFuZCB0aGVyZWZvcmUgbmVlZCB0bwogICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgLy8gYmxlbmQpLgorICAgIGJvb2wgbV9sYXlvdXRBdXRvIDogMTsKIAogICAgIE1hcnF1ZWUq
IG1fbWFycXVlZTsgLy8gVXNlZCBieSBsYXllcnMgd2l0aCBvdmVyZmxvdzptYXJxdWVlCiB9OwpJ
bmRleDogcmVuZGVyaW5nL3JlbmRlcl9sYXllci5jcHAKPT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gcmVuZGVyaW5n
L3JlbmRlcl9sYXllci5jcHAJKHJldmlzaW9uIDEzMjg3KQorKysgcmVuZGVyaW5nL3JlbmRlcl9s
YXllci5jcHAJKHdvcmtpbmcgY29weSkKQEAgLTEwNiwzMyArMTA2LDM0IEBACiB9CiAKIFJlbmRl
ckxheWVyOjpSZW5kZXJMYXllcihSZW5kZXJPYmplY3QqIG9iamVjdCkKLTogbV9vYmplY3QoIG9i
amVjdCApLAotbV9wYXJlbnQoIDAgKSwKLW1fcHJldmlvdXMoIDAgKSwKLW1fbmV4dCggMCApLAot
bV9maXJzdCggMCApLAotbV9sYXN0KCAwICksCi1tX3JlbFgoIDAgKSwKLW1fcmVsWSggMCApLAot
bV94KCAwICksCi1tX3koIDAgKSwKLW1fd2lkdGgoIDAgKSwKLW1faGVpZ2h0KCAwICksCi1tX3Nj
cm9sbFgoIDAgKSwKLW1fc2Nyb2xsWSggMCApLAotbV9zY3JvbGxPcmlnaW5YKCAwICksCi1tX3Nj
cm9sbExlZnRPdmVyZmxvdyggMCApLAotbV9zY3JvbGxXaWR0aCggMCApLAotbV9zY3JvbGxIZWln
aHQoIDAgKSwKLW1faEJhciggMCApLAotbV92QmFyKCAwICksCi1tX3Bvc1pPcmRlckxpc3QoIDAg
KSwKLW1fbmVnWk9yZGVyTGlzdCggMCApLAotbV9jbGlwUmVjdHMoIDAgKSAsCi1tX3Njcm9sbERp
bWVuc2lvbnNEaXJ0eSggdHJ1ZSApLAotbV96T3JkZXJMaXN0c0RpcnR5KCB0cnVlICksCi1tX3Vz
ZWRUcmFuc3BhcmVuY3koIGZhbHNlICksCi1tX21hcnF1ZWUoIDAgKQorOiBtX29iamVjdChvYmpl
Y3QpLAorbV9wYXJlbnQoMCksCittX3ByZXZpb3VzKDApLAorbV9uZXh0KDApLAorbV9maXJzdCgw
KSwKK21fbGFzdCgwKSwKK21fcmVsWCgwKSwKK21fcmVsWSgwKSwKK21feCgwKSwKK21feSgwKSwK
K21fd2lkdGgoMCksCittX2hlaWdodCgwKSwKK21fc2Nyb2xsWCgwKSwKK21fc2Nyb2xsWSgwKSwK
K21fc2Nyb2xsT3JpZ2luWCgwKSwKK21fc2Nyb2xsTGVmdE92ZXJmbG93KDApLAorbV9zY3JvbGxX
aWR0aCgwKSwKK21fc2Nyb2xsSGVpZ2h0KDApLAorbV9oQmFyKDApLAorbV92QmFyKDApLAorbV9w
b3NaT3JkZXJMaXN0KDApLAorbV9uZWdaT3JkZXJMaXN0KDApLAorbV9jbGlwUmVjdHMoMCkgLAor
bV9zY3JvbGxEaW1lbnNpb25zRGlydHkodHJ1ZSksCittX3pPcmRlckxpc3RzRGlydHkodHJ1ZSks
CittX3VzZWRUcmFuc3BhcmVuY3koZmFsc2UpLAorbV9sYXlvdXRBdXRvKHRydWUpLAorbV9tYXJx
dWVlKDApCiB7CiB9CiAKQEAgLTg3Niw3ICs4NzcsNyBAQAogCiAgICAgYm9vbCBuZWVkSG9yaXpv
bnRhbEJhciwgbmVlZFZlcnRpY2FsQmFyOwogICAgIGNvbXB1dGVTY3JvbGxEaW1lbnNpb25zKCZu
ZWVkSG9yaXpvbnRhbEJhciwgJm5lZWRWZXJ0aWNhbEJhcik7Ci0KKyAgICAgICAgCiAgICAgaWYg
KG1fb2JqZWN0LT5zdHlsZSgpLT5vdmVyZmxvdygpICE9IE9NQVJRVUVFKSB7CiAgICAgICAgIC8v
IExheW91dCBtYXkgY2F1c2UgdXMgdG8gYmUgaW4gYW4gaW52YWxpZCBzY3JvbGwgcG9zaXRpb24u
ICBJbiB0aGlzIGNhc2Ugd2UgbmVlZAogICAgICAgICAvLyB0byBwdWxsIG91ciBzY3JvbGwgb2Zm
c2V0cyBiYWNrIHRvIHRoZSBtYXggKG9yIHB1c2ggdGhlbSB1cCB0byB0aGUgbWluKS4KQEAgLTkx
MywxMiArOTE0LDE1IEBACiAgICAgICAgIG1fb2JqZWN0LT5yZXBhaW50KCk7CiAKICAgICAgICAg
aWYgKG1fb2JqZWN0LT5zdHlsZSgpLT5vdmVyZmxvdygpID09IE9BVVRPKSB7Ci0gICAgICAgICAg
ICAvLyBPdXIgcHJvcHJpZXRhcnkgb3ZlcmZsb3c6IG92ZXJsYXkgdmFsdWUgZG9lc24ndCB0cmln
Z2VyIGEgbGF5b3V0LgotICAgICAgICAgICAgbV9vYmplY3QtPnNldE5lZWRzTGF5b3V0KHRydWUp
OwotICAgICAgICAgICAgaWYgKG1fb2JqZWN0LT5pc1JlbmRlckJsb2NrKCkpCi0gICAgICAgICAg
ICAgICAgc3RhdGljX2Nhc3Q8UmVuZGVyQmxvY2sqPihtX29iamVjdCktPmxheW91dEJsb2NrKHRy
dWUpOwotICAgICAgICAgICAgZWxzZQotICAgICAgICAgICAgICAgIG1fb2JqZWN0LT5sYXlvdXQo
KTsKKyAgICAgICAgICAgIGlmIChtX2xheW91dEF1dG8pIHsKKyAgICAgICAgICAgICAgICAvLyBP
dXIgcHJvcHJpZXRhcnkgb3ZlcmZsb3c6IG92ZXJsYXkgdmFsdWUgZG9lc24ndCB0cmlnZ2VyIGEg
bGF5b3V0LgorICAgICAgICAgICAgICAgIG1fb2JqZWN0LT5zZXROZWVkc0xheW91dCh0cnVlKTsK
KyAgICAgICAgICAgICAgICBpZiAobV9vYmplY3QtPmlzUmVuZGVyQmxvY2soKSkgeworICAgICAg
ICAgICAgICAgICAgICBtX2xheW91dEF1dG8gPSBmYWxzZTsKKyAgICAgICAgICAgICAgICAgICAg
c3RhdGljX2Nhc3Q8UmVuZGVyQmxvY2sqPihtX29iamVjdCktPmxheW91dEJsb2NrKHRydWUpOwor
ICAgICAgICAgICAgICAgIH0gZWxzZQorICAgICAgICAgICAgICAgICAgICBtX29iamVjdC0+bGF5
b3V0KCk7CisgICAgICAgICAgICB9CiAgICAgICAgIH0KICAgICB9CiAK
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>7063</attachid>
            <date>2006-03-14 14:15:40 -0800</date>
            <delta_ts>2006-03-14 14:18:58 -0800</delta_ts>
            <desc>no this one!</desc>
            <filename>guardAuto.txt</filename>
            <type>text/plain</type>
            <size>3135</size>
            <attacher name="Beth Dakin">bdakin</attacher>
            
              <data encoding="base64">SW5kZXg6IHJlbmRlcmluZy9yZW5kZXJfbGF5ZXIuaAo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSByZW5kZXJpbmcv
cmVuZGVyX2xheWVyLmgJKHJldmlzaW9uIDEzMjg3KQorKysgcmVuZGVyaW5nL3JlbmRlcl9sYXll
ci5oCSh3b3JraW5nIGNvcHkpCkBAIC0zODcsNiArMzg3LDcgQEAKICAgICBib29sIG1fdXNlZFRy
YW5zcGFyZW5jeSA6IDE7IC8vIFRyYWNrcyB3aGV0aGVyIHdlIG5lZWQgdG8gY2xvc2UgYSB0cmFu
c3BhcmVudCBsYXllciwgaS5lLiwgd2hldGhlcgogICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgLy8gd2UgZW5kZWQgdXAgcGFpbnRpbmcgdGhpcyBsYXllciBvciBhbnkgZGVzY2VuZGFu
dHMgKGFuZCB0aGVyZWZvcmUgbmVlZCB0bwogICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgLy8gYmxlbmQpLgorICAgIGJvb2wgbV9pbk92ZXJmbG93UmVsYXlvdXQgOiAxOwogCiAgICAg
TWFycXVlZSogbV9tYXJxdWVlOyAvLyBVc2VkIGJ5IGxheWVycyB3aXRoIG92ZXJmbG93Om1hcnF1
ZWUKIH07CkluZGV4OiByZW5kZXJpbmcvcmVuZGVyX2xheWVyLmNwcAo9PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBy
ZW5kZXJpbmcvcmVuZGVyX2xheWVyLmNwcAkocmV2aXNpb24gMTMyODcpCisrKyByZW5kZXJpbmcv
cmVuZGVyX2xheWVyLmNwcAkod29ya2luZyBjb3B5KQpAQCAtMTA2LDMzICsxMDYsMzQgQEAKIH0K
IAogUmVuZGVyTGF5ZXI6OlJlbmRlckxheWVyKFJlbmRlck9iamVjdCogb2JqZWN0KQotOiBtX29i
amVjdCggb2JqZWN0ICksCi1tX3BhcmVudCggMCApLAotbV9wcmV2aW91cyggMCApLAotbV9uZXh0
KCAwICksCi1tX2ZpcnN0KCAwICksCi1tX2xhc3QoIDAgKSwKLW1fcmVsWCggMCApLAotbV9yZWxZ
KCAwICksCi1tX3goIDAgKSwKLW1feSggMCApLAotbV93aWR0aCggMCApLAotbV9oZWlnaHQoIDAg
KSwKLW1fc2Nyb2xsWCggMCApLAotbV9zY3JvbGxZKCAwICksCi1tX3Njcm9sbE9yaWdpblgoIDAg
KSwKLW1fc2Nyb2xsTGVmdE92ZXJmbG93KCAwICksCi1tX3Njcm9sbFdpZHRoKCAwICksCi1tX3Nj
cm9sbEhlaWdodCggMCApLAotbV9oQmFyKCAwICksCi1tX3ZCYXIoIDAgKSwKLW1fcG9zWk9yZGVy
TGlzdCggMCApLAotbV9uZWdaT3JkZXJMaXN0KCAwICksCi1tX2NsaXBSZWN0cyggMCApICwKLW1f
c2Nyb2xsRGltZW5zaW9uc0RpcnR5KCB0cnVlICksCi1tX3pPcmRlckxpc3RzRGlydHkoIHRydWUg
KSwKLW1fdXNlZFRyYW5zcGFyZW5jeSggZmFsc2UgKSwKLW1fbWFycXVlZSggMCApCis6IG1fb2Jq
ZWN0KG9iamVjdCksCittX3BhcmVudCgwKSwKK21fcHJldmlvdXMoMCksCittX25leHQoMCksCitt
X2ZpcnN0KDApLAorbV9sYXN0KDApLAorbV9yZWxYKDApLAorbV9yZWxZKDApLAorbV94KDApLAor
bV95KDApLAorbV93aWR0aCgwKSwKK21faGVpZ2h0KDApLAorbV9zY3JvbGxYKDApLAorbV9zY3Jv
bGxZKDApLAorbV9zY3JvbGxPcmlnaW5YKDApLAorbV9zY3JvbGxMZWZ0T3ZlcmZsb3coMCksCitt
X3Njcm9sbFdpZHRoKDApLAorbV9zY3JvbGxIZWlnaHQoMCksCittX2hCYXIoMCksCittX3ZCYXIo
MCksCittX3Bvc1pPcmRlckxpc3QoMCksCittX25lZ1pPcmRlckxpc3QoMCksCittX2NsaXBSZWN0
cygwKSAsCittX3Njcm9sbERpbWVuc2lvbnNEaXJ0eSh0cnVlKSwKK21fek9yZGVyTGlzdHNEaXJ0
eSh0cnVlKSwKK21fdXNlZFRyYW5zcGFyZW5jeShmYWxzZSksCittX2luT3ZlcmZsb3dSZWxheW91
dChmYWxzZSksCittX21hcnF1ZWUoMCkKIHsKIH0KIApAQCAtODc2LDcgKzg3Nyw3IEBACiAKICAg
ICBib29sIG5lZWRIb3Jpem9udGFsQmFyLCBuZWVkVmVydGljYWxCYXI7CiAgICAgY29tcHV0ZVNj
cm9sbERpbWVuc2lvbnMoJm5lZWRIb3Jpem9udGFsQmFyLCAmbmVlZFZlcnRpY2FsQmFyKTsKLQor
ICAgICAgICAKICAgICBpZiAobV9vYmplY3QtPnN0eWxlKCktPm92ZXJmbG93KCkgIT0gT01BUlFV
RUUpIHsKICAgICAgICAgLy8gTGF5b3V0IG1heSBjYXVzZSB1cyB0byBiZSBpbiBhbiBpbnZhbGlk
IHNjcm9sbCBwb3NpdGlvbi4gIEluIHRoaXMgY2FzZSB3ZSBuZWVkCiAgICAgICAgIC8vIHRvIHB1
bGwgb3VyIHNjcm9sbCBvZmZzZXRzIGJhY2sgdG8gdGhlIG1heCAob3IgcHVzaCB0aGVtIHVwIHRv
IHRoZSBtaW4pLgpAQCAtOTEzLDEyICs5MTQsMTYgQEAKICAgICAgICAgbV9vYmplY3QtPnJlcGFp
bnQoKTsKIAogICAgICAgICBpZiAobV9vYmplY3QtPnN0eWxlKCktPm92ZXJmbG93KCkgPT0gT0FV
VE8pIHsKLSAgICAgICAgICAgIC8vIE91ciBwcm9wcmlldGFyeSBvdmVyZmxvdzogb3ZlcmxheSB2
YWx1ZSBkb2Vzbid0IHRyaWdnZXIgYSBsYXlvdXQuCi0gICAgICAgICAgICBtX29iamVjdC0+c2V0
TmVlZHNMYXlvdXQodHJ1ZSk7Ci0gICAgICAgICAgICBpZiAobV9vYmplY3QtPmlzUmVuZGVyQmxv
Y2soKSkKLSAgICAgICAgICAgICAgICBzdGF0aWNfY2FzdDxSZW5kZXJCbG9jayo+KG1fb2JqZWN0
KS0+bGF5b3V0QmxvY2sodHJ1ZSk7Ci0gICAgICAgICAgICBlbHNlCi0gICAgICAgICAgICAgICAg
bV9vYmplY3QtPmxheW91dCgpOworICAgICAgICAgICAgaWYgKCFtX2luT3ZlcmZsb3dSZWxheW91
dCkgeworICAgICAgICAgICAgICAgIC8vIE91ciBwcm9wcmlldGFyeSBvdmVyZmxvdzogb3Zlcmxh
eSB2YWx1ZSBkb2Vzbid0IHRyaWdnZXIgYSBsYXlvdXQuCisgICAgICAgICAgICAgICAgbV9pbk92
ZXJmbG93UmVsYXlvdXQgPSB0cnVlOworICAgICAgICAgICAgICAgIG1fb2JqZWN0LT5zZXROZWVk
c0xheW91dCh0cnVlKTsKKyAgICAgICAgICAgICAgICBpZiAobV9vYmplY3QtPmlzUmVuZGVyQmxv
Y2soKSkKKyAgICAgICAgICAgICAgICAgICAgc3RhdGljX2Nhc3Q8UmVuZGVyQmxvY2sqPihtX29i
amVjdCktPmxheW91dEJsb2NrKHRydWUpOworICAgICAgICAgICAgICAgIGVsc2UKKyAgICAgICAg
ICAgICAgICAgICAgbV9vYmplY3QtPmxheW91dCgpOworICAgICAgICAgICAgICAgIG1faW5PdmVy
Zmxvd1JlbGF5b3V0ID0gZmFsc2U7CisgICAgICAgICAgICB9CiAgICAgICAgIH0KICAgICB9CiAK
</data>
<flag name="review"
          id="1839"
          type_id="1"
          status="+"
          setter="hyatt"
    />
          </attachment>
      

    </bug>

</bugzilla>