On Wed, 5 May 2021 14:11:16 GMT, Matthias Baesken <mbaes...@openjdk.org> wrote:

>> In java/awt/font/TextJustifier.java justify-method there is a potential code 
>> path where divison by zero might happen , see also the Sonar finding 
>> https://sonarcloud.io/project/issues?id=shipilev_jdk&open=AXcqMwpm8sPJZZzONu1k&resolved=false&severities=CRITICAL&types=BUG
>> 
>> 
>>             boolean hitLimit = (weight == 0) || (!lastPass && ((delta < 0) 
>> == (delta < gslimit)));
>>             boolean absorbing = hitLimit && absorbweight > 0;
>>             // predivide delta by weight
>>             float weightedDelta = delta / weight; // not used if weight == 0
>> 
>> In case of (weight == 0) the division should not be done because the value 
>> of weightedDelta is unused in this case anyway.
>
> Matthias Baesken has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   adjust absorbweight check

Marked as reviewed by psadhukhan (Reviewer).

-------------

PR: https://git.openjdk.java.net/jdk/pull/2912

Reply via email to