Closed Bug 935193 Opened 11 years ago Closed 5 years ago

StartupCache shouldn't assert when writing to an existing entry

Categories

(Core :: General, defect)

Product:
Core
Component:
General
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED WORKSFORME

People

(Reporter: Gijs, Unassigned)

References

Details

(Keywords: assertion)

https://tbpl.mozilla.org/php/getParsedLog.php?id=30158326&tree=Mozilla-Inbound&full=1#error0 Of course, that's me assuming that you can actually do that. See also bug 821162, bug 744652. In particular, that first writing to entry a with size 5, then writing to other stuff, and then back at entry a with size 1000, doesn't overwrite the other stuff. For now, I added an expectAssertions to the test file. If this is actually Really Bad, then we should probably remove the bits from 821726 which added this flag, and remove the extra bits in the test.

Looks like these got removed with the switch to lz4.

Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.