Closed
Bug 1859292
Opened 2 years ago
Closed 2 years ago
GenericPrinter::vprintf no longer short-circuit after the lack of interpolation.
Categories
(Core :: JavaScript Engine, defect, P1)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
120 Branch
| Tracking | Status | |
|---|---|---|
| firefox-esr115 | --- | unaffected |
| firefox118 | --- | unaffected |
| firefox119 | --- | unaffected |
| firefox120 | --- | fixed |
People
(Reporter: nbp, Assigned: nbp)
References
(Regression)
Details
(Keywords: regression)
Attachments
(1 file)
After Bug 1839396 part 13, which made printer function infallible, the vprintf function prints the content of the string twice if there is no % character in the string to be interpolated.
The problem comes from the removal of the return statement which prevented going to the generic code path.
|
||
Comment 1•2 years ago
|
||
Set release status flags based on info from the regressing bug 1839396
status-firefox118:
--- → unaffected
status-firefox119:
--- → unaffected
status-firefox120:
--- → affected
status-firefox-esr115:
--- → unaffected
|
Assignee | |
Comment 2•2 years ago
|
||
Going over the content of Bug 1839396 part 13, I was not able to spot any additional error similar to the missing return, only the one in vprintf.
|
|
Assignee | |
Updated•2 years ago
|
Severity: -- → S4
Priority: -- → P1
|
|
Assignee | |
Comment 3•2 years ago
|
||
Pushed by npierron@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/008bb178c280
Restore GenericPrinter::vprintf expected behaviour for non-interpolating strings. r=jseward
|
||
Comment 5•2 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 120 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•