This is an archived snapshot of W3C's public bugzilla bug tracker, decommissioned in April 2019. Please see the home page for more details.

Bug 4516 - Escape SGML:Parser:OpenSP's croaking on some error messages
Summary: Escape SGML:Parser:OpenSP's croaking on some error messages
Status: RESOLVED FIXED
Alias: None
Product: Validator
Classification: Unclassified
Component: Parser (show other bugs)
Version: 0.8.0b1
Hardware: All All
: P2 normal
Target Milestone: 0.8.0
Assignee: Terje Bless
QA Contact: qa-dev tracking
URL: http://lists.w3.org/Archives/Public/w...
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-05-01 15:03 UTC by Olivier Thereaux
Modified: 2008-06-15 18:43 UTC (History)
0 users

See Also:

Attachments
test case (103 bytes, text/html)
2008-06-15 18:43 UTC, Olivier Thereaux 		Details

Description Olivier Thereaux 2007-05-01 15:03:54 UTC 
As reported by Frank on www-validator:
[[
| Unexpected error message format ((invalid
location):47239440633056.2322:W: URL | Redirected to
| "http://www.xyzzy.claranet.de/w3c/html2070.dtd"
| ) at /usr/local/lib/perl/5.8.8/SGML/Parser/OpenSP.pm line 45
]]

SPO has a regexp to match what is an acceptable error message from OpenSP
    my @comp = split(/(?:^|\n)(?:<[^>]+>)?\Q$name\E:(\d+):(\d+):\s*/, $mess);
but this regexp seems to not catch some rare messages (in this case, a redirected DTD SI)
Comment 1 Olivier Thereaux 2007-05-01 15:05:36 UTC 
Fixed in CVS:
http://lists.w3.org/Archives/Public/www-validator-cvs/2007May/0003.html
by eval{}ing the call to split_message and moving on if the routine has croak'd.
Comment 2 Olivier Thereaux 2008-06-15 18:43:38 UTC 
Created attachment 557 [details]
test case

the original (online) test case is no longer displaying the same behavior, so creating a new test case with:
* a system ID
* redirecting
* to a 404