ASSERTION: the given url scheme contains invalid characters

RESOLVED FIXED

Status

()

Core
Networking
RESOLVED FIXED
13 years ago
2 years ago

People

(Reporter: Jesse Ruderman, Unassigned)

Tracking

(Blocks: 1 bug, {assertion, testcase})

Trunk
PowerPC
Mac OS X
assertion, testcase
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

13 years ago
###!!! ASSERTION: the given url scheme contains invalid characters: 'Error', file /Users/admin/trunk/mozilla/netwerk/base/src/nsStandardURL.cpp, line 1159
(Reporter)

Comment 1

13 years ago
Created attachment 211357 [details]
testcase
(Reporter)

Updated

13 years ago
Blocks: 326633

Updated

13 years ago
Target Milestone: --- → mozilla1.9alpha
What is this bug asking for, to change all the NS_ERROR in that file into NS_WARNING as part of the "get rid of assertions" jihad?

Or maybe we should reimplement NS_ERROR to not use assertions since it's useful to make the error/warning distinction for debug output. This location seems to be only worth a warning though.

Updated

12 years ago
Assignee: darin → nobody
QA Contact: benc → networking
Target Milestone: mozilla1.9alpha → ---
(Reporter)

Comment 3

11 years ago
Right, assertions are for checks that indicate bugs in the code, not for "error conditions" that can be triggered by content (and handled reasonably).  Perhaps NS_ERROR should be renamed to make that more clear, but that would be a separate bug.

Dan, do you know which assertions in nsStandardURL.cpp should be turned into warnings?
(Reporter)

Comment 4

11 years ago
Bug 369179 is another bogus assertion in nsStandardURL.cpp.
(Reporter)

Comment 5

9 years ago
Still happens on trunk.
warning now
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.