[OAK-5125] Some implementations of CacheValue.getMemory() don't care about integer overflow.

• Type: Bug
• Status: Closed
• Priority: Blocker
• Resolution: Fixed
• Affects Version/s: 1.0.34, 1.2.21, 1.4.10, 1.5.13
• Fix Version/s: 1.4.11, 1.5.15, 1.6.0
• Component/s: core
• Assignee: Manfred Baedke
• Reporter: Manfred Baedke
• Created: 17/Nov/16 14:15
• Updated: 16/Feb/21 09:52
• Resolved: 05/Dec/16 20:51

Description

The interface method org.apache.jackrabbit.oak.cache.CacheValue.getMemory() returns an int value, while some implementations measure values larger than Integer.MAX_VALUE, producing an int overflow. This actually breaks in real life when testing with huge repos and machines, since it's used to measure the size of local diffs (e.g. a reindexing may diff the whole repo against an empty root).

Issue Links

• relates to: OAK-5132 Limit diff cache entries in size