Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(57)

Issue 7875008: Desist libxml from continuing the parse after a SAX callback has stopped the (Closed)

Created:
9 years, 3 months ago by Chris Evans
Modified:
9 years, 3 months ago
Reviewers:
Cris Neckar
CC:
chromium-reviews
Visibility:
Public.

Description

Desist libxml from continuing the parse after a SAX callback has stopped the parse. BUG=95465 TEST=covered by existing tests under ASAN Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=100883

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+13 lines, -4 lines) Patch
M third_party/libxml/README.chromium View 1 chunk +2 lines, -1 line 0 comments Download
M third_party/libxml/src/parser.c View 5 chunks +11 lines, -3 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Chris Evans
(Note: the lint errors are all about tabs, which is standard libxml style, FWIW)
9 years, 3 months ago (2011-09-12 23:11:03 UTC) #1
Cris Neckar
9 years, 3 months ago (2011-09-12 23:46:17 UTC) #2
On 2011/09/12 23:11:03, Chris Evans wrote:
> (Note: the lint errors are all about tabs, which is standard libxml style,
FWIW)

LGTM

Where "G" = "goddamn ugly"

Powered by Google App Engine
This is Rietveld 408576698