Page MenuHomePhabricator
Differential D40217

WIP: Bug 1501881 - Instrument the build to dump L10n IDs during startup for DTD/Properties/FTL.
Changes PlannedPublic
Actions

Authored by zbraniecki on Aug 1 2019, 2:55 PM.
Tags
None
Referenced Files
Unknown Object (File)
Oct 15 2025, 8:28 AM
Unknown Object (File)
Oct 14 2025, 9:03 PM
Unknown Object (File)
Oct 14 2025, 12:03 AM
Unknown Object (File)
Oct 9 2025, 1:27 AM
Unknown Object (File)
May 28 2025, 2:19 AM
Unknown Object (File)
May 10 2025, 5:50 PM
Unknown Object (File)
Apr 18 2025, 7:59 AM
Unknown Object (File)
Mar 16 2025, 10:47 PM
View All 15 Files
Subscribers
reviewbot

Details

Reviewers
None
Bugzilla Bug ID
1501881

Diff Detail

Repository
rMOZILLACENTRAL mozilla-central
Branch
default

Event Timeline

zbraniecki created this revision.Aug 1 2019, 2:55 PM
Herald added a project: secure-revision. · View Herald TranscriptAug 1 2019, 2:55 PM
Harbormaster completed remote builds in B98590: Diff 140713.Aug 1 2019, 2:55 PM
phab-bot requested review of this revision.Aug 1 2019, 2:55 PM
phab-bot changed the visibility from "Custom Policy" to "Public (No Login Required)".
phab-bot changed the edit policy from "Custom Policy" to "Restricted Project (Project)".
phab-bot removed a project: secure-revision.
zbraniecki updated this revision to Diff 144150.Aug 8 2019, 6:06 AM

Revision updated.

Harbormaster completed remote builds in B101194: Diff 144150.Aug 8 2019, 6:06 AM
zbraniecki updated this revision to Diff 144151.Aug 8 2019, 6:10 AM

Revision updated.

Harbormaster completed remote builds in B101195: Diff 144151.Aug 8 2019, 6:10 AM
reviewbot added a subscriber: reviewbot.Aug 8 2019, 6:55 AM

Code analysis found 1 defect in the diff 144150:

  • 1 defect found by Coverity

If you see a problem in this automated review, please report it here.

parser/expat/lib/xmlparse.c
805

Checker reliability is medium, meaning that the false positive ratio is medium.
Argument "(char *)name[i]" to format specifier "%c" was expected to have type "int" but has type "char *".
The path that leads to this defect is:

  • parser/expat/lib/xmlparse.c:805:
    • invalid_type: Argument "(char *)name[i]" to format specifier "%c" was expected to have type "int" but has type "char *"..
reviewbot added a comment.Aug 8 2019, 6:56 AM

Code analysis found 1 defect in the diff 144151:

  • 1 defect found by Coverity

If you see a problem in this automated review, please report it here.

parser/expat/lib/xmlparse.c
805

Checker reliability is medium, meaning that the false positive ratio is medium.
Argument "(char *)name[i]" to format specifier "%c" was expected to have type "int" but has type "char *".
The path that leads to this defect is:

  • parser/expat/lib/xmlparse.c:805:
    • invalid_type: Argument "(char *)name[i]" to format specifier "%c" was expected to have type "int" but has type "char *"..
zbraniecki updated this revision to Diff 245366.Mar 16 2020, 11:28 PM
Harbormaster completed remote builds in B172952: Diff 245366.Mar 16 2020, 11:28 PM
reviewbot added a comment.Mar 16 2020, 11:50 PM

Code analysis found 1 defect in the diff 245366:

  • 1 defect found by coverity-coverity

If you see a problem in this automated review, please report it here.

You can view these defects on the code-review frontend and on Treeherder.

parser/expat/lib/xmlparse.c
805

Checker reliability is medium, meaning that the false positive ratio is medium.
Argument "(char *)name[i]" to format specifier "%c" was expected to have type "int" but has type "char *".
The path that leads to this defect is:

  • parser/expat/lib/xmlparse.c:805:
    • invalid_type: Argument "(char *)name[i]" to format specifier "%c" was expected to have type "int" but has type "char *"..
zbraniecki planned changes to this revision.Aug 10 2021, 7:59 PM
zbraniecki updated this revision to Diff 469448.
zbraniecki retitled this revision from Bug 1501881 - Instrument the build to dump L10n IDs during startup for DTD/Properties/FTL. to WIP: Bug 1501881 - Instrument the build to dump L10n IDs during startup for DTD/Properties/FTL..
Harbormaster completed remote builds in B342869: Diff 469448.Aug 10 2021, 7:59 PM
reviewbot added a comment.Aug 10 2021, 8:39 PM

Code analysis found 1 defect in the diff 469448:

  • 1 defect found by Coverity

If you see a problem in this automated review, please report it here.

You can view these defects on the code-review frontend and on Treeherder.

Revision Contents
Changeset List

PathSize
intl/
l10n/
rust/
fluent-ffi/
src/
1 line
localization-ffi/
src/
14 lines
strres/
37 lines
parser/
expat/
lib/
21 lines

Diff 469448

View Options

intl/l10n/rust/fluent-ffi/src/bundle.rs

Loading...
View Options

intl/l10n/rust/localization-ffi/src/lib.rs

Loading...
View Options

intl/strres/nsStringBundle.cpp

Loading...
View Options

parser/expat/lib/xmlparse.c

Loading...
Privacy · Cookies · Legal