On Tue, 10 Nov 2026 13:39:11 GMT, Matthias Baesken <[email protected]> wrote:
> IconSwap.cpp ChangeIcon checks malloc - results for NULL but seems to check > at one place the wrong variable. > There is also a place in the coding where we currently miss a malloc - check > after the call and should better add it. > > --------- > - [x] I confirm that I make this contribution in accordance with the [OpenJDK > Interim AI Policy](https://openjdk.org/legal/ai). Maybe we should centralize the free- and CloseHandle - calls at the end of the function and goto there to avoid a bit of duplication ? ------------- PR Comment: https://git.openjdk.org/jdk/pull/33820#issuecomment-6768846419
