Closed Bug 1556829 Opened 6 years ago Closed 6 years ago

The version comparator still uses invalid values in the presence of large numbers

Categories

(Core :: XPCOM, task)

task
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla69
Tracking Status
firefox69 --- fixed

People

(Reporter: mossop, Assigned: mossop)

References

Details

Attachments

(1 file)

In bug 1555179 I attempted to correct things where version strings contain numbers larger than 32-bits by normalizing that case to 0. But I failed to notice that the function we use to parse the number actually supports 64-bit numbers, its just that we then stuff the 64-bit number into a 32-bit variable. So bug 1555179 actually only corrects parsing >64-bit numbers. For 32-bit to 64-bit numbers we still get an incorrect value.

Summary: The version comparator still uses invalid values in the presence invalid numbers → The version comparator still uses invalid values in the presence of large numbers
Blocks: 1556612

Bugbug thinks this bug is a task, but please change it back in case of error.

Type: defect → task
Pushed by dtownsend@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/b3627356da98 Convert version numbers larger or smaller than the 32-bit limit to 0. r=froydnj
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla69
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: