Bug 37247 - Limit XML node depth
Summary: Limit XML node depth
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: XML (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Nobody
URL:
Keywords:
: 35746 (view as bug list)
Depends on:
Blocks: 18282
  Show dependency treegraph
 
Reported: 2010-04-07 22:53 PDT by Chris Evans
Modified: 2010-05-18 09:34 PDT (History)
7 users (show)

See Also:


Attachments
Limit excessive XML recursion, which can cause stack overflows in various paths. (3.06 KB, patch)
2010-04-07 23:20 PDT, Chris Evans
no flags Details | Formatted Diff | Diff
Fix type mismatch. (3.06 KB, patch)
2010-04-08 01:02 PDT, Chris Evans
commit-queue: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Evans 2010-04-07 22:53:16 PDT
Patch forthcoming.
Comment 1 Chris Evans 2010-04-07 23:20:31 PDT
Created attachment 52833 [details]
Limit excessive XML recursion, which can cause stack overflows in various paths.
Comment 2 Adam Barth 2010-04-07 23:31:28 PDT
Comment on attachment 52833 [details]
Limit excessive XML recursion, which can cause stack overflows in various paths.

ok
Comment 3 WebKit Commit Bot 2010-04-08 00:56:58 PDT
Comment on attachment 52833 [details]
Limit excessive XML recursion, which can cause stack overflows in various paths.

Rejecting patch 52833 from commit-queue.

Failed to run "['WebKitTools/Scripts/build-webkit', '--debug']" exit_code: 1
Last 500 characters of output:
comparison between signed and unsigned integer expressions
distcc[50702] ERROR: compile /Users/eseidel/Projects/CommitQueue/WebCore/dom/XMLTokenizer.cpp on localhost failed
** BUILD FAILED **

The following build commands failed:
WebCore:
	Distributed-CompileC /Users/eseidel/Projects/CommitQueue/WebKitBuild/WebCore.build/Debug/WebCore.build/Objects-normal/i386/XMLTokenizer.o /Users/eseidel/Projects/CommitQueue/WebCore/dom/XMLTokenizer.cpp normal i386 c++ com.apple.compilers.gcc.4_2
(1 failure)


Full output: http://webkit-commit-queue.appspot.com/results/1688051
Comment 4 Chris Evans 2010-04-08 01:02:59 PDT
Created attachment 52846 [details]
Fix type mismatch.
Comment 5 Adam Barth 2010-04-08 01:04:06 PDT
Comment on attachment 52846 [details]
Fix type mismatch.

k
Comment 6 WebKit Commit Bot 2010-04-08 01:19:31 PDT
Comment on attachment 52846 [details]
Fix type mismatch.

Clearing flags on attachment: 52846

Committed r57263: <http://trac.webkit.org/changeset/57263>
Comment 7 Eric Seidel (no email) 2010-04-08 02:36:49 PDT
I am concerned this test runs too slowly.
Chromium will need to mark it as SLOW if nothing else.
Comment 8 Chris Evans 2010-04-08 02:40:24 PDT
How long do you see it taking? At what timing do things be a concern?
Comment 9 Alexey Proskuryakov 2010-04-08 12:48:13 PDT
*** Bug 35746 has been marked as a duplicate of this bug. ***
Comment 10 Alexey Proskuryakov 2010-04-08 12:50:15 PDT
The test takes about a second for me on a debug build (on Mac Book Pro). This doesn't seem to be excessively slow.
Comment 11 WebKit Commit Bot 2010-04-22 00:19:35 PDT
Comment on attachment 52846 [details]
Fix type mismatch.

Rejecting patch 52846 from commit-queue.

Failed to run "[u'/Users/eseidel/Projects/CommitQueue/WebKitTools/Scripts/svn-apply', u'--reviewer', u'Eric Seidel', u'--force']" exit_code: 1
patching file WebCore/dom/XMLTokenizer.cpp
Hunk #1 FAILED at 67.
Hunk #2 FAILED at 87.
2 out of 2 hunks FAILED -- saving rejects to file WebCore/dom/XMLTokenizer.cpp.rej
patching file WebCore/ChangeLog
Hunk #1 succeeded at 1 with fuzz 3.
patching file LayoutTests/fast/images/svg-nested.html
patching file LayoutTests/fast/images/svg-nested-expected.txt
patching file LayoutTests/ChangeLog
Hunk #1 succeeded at 1 with fuzz 3.

Full output: http://webkit-commit-queue.appspot.com/results/1853042
Comment 12 Adam Barth 2010-05-15 00:12:21 PDT
Comment on attachment 52846 [details]
Fix type mismatch.

Clearing Eric's r+ from this patch so that it doesn't show up in pending-commit.  Please upload an updated version of this patch that applies cleanly to top of tree.

BTW, the tools say that you're not a committer, but I thought your paperwork when through.  If you're not listed in committers.py, please add yourself and feel free to land an updated version of this patch with Eric's review.
Comment 13 Chris Evans 2010-05-18 07:25:56 PDT
Hi Adam,

This was already landed as per c6. Looks like the patch was added to the commit-queue twice and the second one obviously failed.
So nothing to do here.

I'll add myself to committers.py shortly.

Cheers
Chris
Comment 14 Adam Barth 2010-05-18 09:34:52 PDT
If this patch as been landed, we should close the bug.