[jdk17u-dev] RFR: 8299677: Formatter.format might take a long time to format an integer or floating-point [v6]

[Paul Hohensee]

On Tue, 5 Sep 2023 21:47:33 GMT, Paul Hohensee <phh at openjdk.org> wrote:

>> Simple almost clean backport of a potential DOS attack vector fix. Copyright date conflict plus changed
>> 
>> Flags.contains(f, Flags.ZERO_PAD)
>> 
>> to
>> 
>> f.contains(Flags.ZERO_PAD)
>> 
>> Passes new and old Formatter tests.
>
> Paul Hohensee has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 411 commits:
> 
>  - Merge
>  - Merge branch 'master' into backport-8299677
>  - 8311033: [macos] PrinterJob does not take into account Sides attribute
>    
>    Backport-of: a3d67231a71fbe37c509fcedd54c679b4644c0d9
>  - 8295737: macOS: Print content cut off when width > height with portrait orientation
>    
>    Backport-of: d00a767047ec41e233e711dbc5fe7b8818e72f28
>  - 8313815: The exception messages printed by jcmd ManagementAgent.start are corrupted on Japanese Windows
>    
>    Reviewed-by: kevinw, cjplummer, phh
>  - 8314426: runtime/os/TestTrimNative.java is failing on slow machines
>    
>    Reviewed-by: stuefe
>    Backport-of: 20e94784c9f7c30e95550c72aedb5e986a153114
>  - 8312525: New test runtime/os/TestTrimNative.java#trimNative is failing: did not see the expected RSS reduction
>    
>    Reviewed-by: mdoerr
>    Backport-of: ad34be1f329edc8e7155983835cc70d733c014b8
>  - 8293114: JVM should trim the native heap
>    
>    Reviewed-by: stuefe
>    Backport-of: 9e4fc568a6f1a93c84a84d6cc5220c6eb4e546a5
>  - 8276651: java/lang/ProcessHandle tests fail with "RuntimeException: Input/output error" in java.lang.ProcessHandleImpl$Info.info0
>    
>    Backport-of: d24b7b7026cf85f1aecf44f60819762872cfd5c1
>  - 8314730: GHA: Drop libfreetype6-dev transitional package in favor of libfreetype-dev
>    
>    Reviewed-by: stuefe
>    Backport-of: 69d900d2ce97e5479020cff9a63c471d07e39989
>  - ... and 401 more: https://git.openjdk.org/jdk17u-dev/compare/75e14419...b8c04b4f

Indeed it does. :) Thanks for the review!

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

PR Comment: https://git.openjdk.org/jdk17u-dev/pull/1077#issuecomment-1708462663