[LUCENE-6075] SimpleRateLimiter cast overflow results in Thread.sleep exception

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 4.10.3, 5.0, 6.0
• Component/s: core/store
• Lucene Fields: New
• Assignee: Michael McCandless
• Reporter: Boaz Leskes
• Created: 24/Nov/14 12:47
• Updated: 28/Aug/22 14:20
• Resolved: 01/Dec/14 11:21

Description

SimpleRateLimiter.pause() uses an uncheck cast of longs to ints:

Thread.sleep((int) (pauseNS/1000000), (int) (pauseNS % 1000000));

Although we check that pauseNS is positive, however if it's large enough the cast to int produces a negative value, causing Thread.sleep to throw an exception.

We should protect for it.