xml parsing error in phishing code

RESOLVED FIXED

Status

RESOLVED FIXED
9 years ago
5 years ago

People

(Reporter: Pike, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(2 attachments)

(Reporter)

Description

9 years ago
In http://mxr.mozilla.org/l10n-mozilla1.9.2/source/bn-IN/browser/chrome/browser/safebrowsing/phishing-afterload-warning-message.dtd#41, the entry for safeb.blocked.malwarePage.longDesc is missing an opening <p>, introducing an XML parsing error.

Comment 1

7 years ago
Created attachment 601986 [details] [diff] [review]
Patched XML parsing error.

Added <p> tag to the entry for safeb.blocked.malwarePage.longDesc http://mxr.mozilla.org/l10n-mozilla1.9.2/source/bn-IN/browser/chrome/browser/safebrowsing/phishing-afterload-warning-message.dtd#41
Attachment #601986 - Flags: review?(runabh)

Comment 2

7 years ago
Comment on attachment 601986 [details] [diff] [review]
Patched XML parsing error.

Hi Sayak,
You may uploaded a tar.gz file while the file type has been marked as 'text'. As a result I can't read the contents. Can you please upload the patch or the corrected text file once more, this time not as an archive?
Am marking it as a 'negative' for now, but dont worry too much about it. :)

cheers
Runa
Attachment #601986 - Flags: review?(runabh) → review-

Comment 3

7 years ago
Created attachment 602257 [details] [diff] [review]
Patched DTD File

Patched DTD for phishing-afterload-warning-message.
Attachment #602257 - Flags: review?(runabh)

Updated

7 years ago
Attachment #602257 - Flags: feedback+

Comment 4

7 years ago
Committed: http://hg.mozilla.org/releases/l10n-mozilla-1.9.2/bn-IN/

cheers
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
(Reporter)

Updated

5 years ago
Attachment #602257 - Flags: review?(runabh)
You need to log in before you can comment on or make changes to this bug.