[LUCENE-4162] MathUtil is publiuc and checks parameter only with assert. Incorrect usage can easily lead to endless loop

• Type: Bug
• Status: Closed
• Priority: Critical
• Resolution: Fixed
• Affects Version/s: 4.0-ALPHA, 6.0
• Fix Version/s: 4.0-ALPHA, 6.0
• Component/s: core/other
• Lucene Fields: New
• Assignee: Uwe Schindler
• Reporter: Uwe Schindler
• Created: 21/Jun/12 16:21
• Updated: 28/Aug/22 13:20
• Resolved: 21/Jun/12 16:49

Description

We should check the 2 parameters carefully before entering the loop. Those checks here cost nothing.

Otherwise this class must be hidden, we should not offer any public method to the users that can loop infinite without parameter checking.