Closed Bug 47288 Opened 24 years ago Closed 23 years ago

base64 padding incorrectly interpreted in data: URI scheme

Categories

(Core :: Networking, defect, P3)

x86
Linux
defect

Tracking

()

VERIFIED DUPLICATE of bug 53792
mozilla0.9.1

People

(Reporter: david+bugs, Assigned: neeti)

References

()

Details

(Keywords: testcase)

Attachments

(1 file)

The data: URI scheme (RFC 2397) handler seems to have a minor problem interpreting base64-encoded strings (RFC 2045, section 6.8) when they are padded (see RFC 2045, page 24, penultimate paragraph). This corresponds to the trailing equal signs in the base64 data. The page referenced by the URI given above should contain the text "This is a test." and *nothing else*. Whereas Mozilla seems to add a random character on the second line. This is probably an error in the computation of the size of the data.
not critical for beta3
Target Milestone: --- → Future
Confirmed on Linux 2000080308.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Adding "patch" keyword.
Keywords: patch
->me
Assignee: gagan → darin
david: my appologies for not responding to this bug sooner... thanks for the patch! hopefully, i'll be able to get it in by 0.9.1.
Keywords: nsbeta1
Target Milestone: Future → mozilla0.9.1
neeti: any chance you could review this patch and possibly help land it for 0.9.1?
Assignee: darin → neeti
Keywords: nsbeta1nsbeta1+
david: I just see the line "This is a test." both before and after applying the patch, when I load the url mentioned above. I did not see any random characters on the second line in mozilla before and after applying the patch. Do you still see the random characters on the second line in mozilla on the second line?
This has indeed been fixed on 2000/11/29 while resolving bug 53792 (essentially a duplicate of this one). That's the end of that bug, *BUT* that's not the end of the story as far as "data:" scheme URIs are concerned; because the resolution by sspitzer@netscape.com of bug 53792 introduces another bug, which I will be filing in an minute, and which would have been avoided had my patch been used instead (<"told you so" grin>). I will add the new bug number as soon as I check it and file it. *** This bug has been marked as a duplicate of 53792 ***
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → DUPLICATE
followup to bug 78876
verified dups
Status: RESOLVED → VERIFIED
Keywords: testcase
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: