> **Issue:** 
> AwtWindow::SetIconData leaks the old icon handles in hOldIcon and hOldIconSm 
> if CreateIconFromRaster raises an exception. Additionally, an exception is 
> checked only after the first call to CreateIconFromRaster.
> 
> **Solution:**
> I have added the exception handling code to take care that the handles are 
> properly destroyed and not leaked.
> 
> **Testing:**
> I have tested the code to make sure there are no regressions caused by this.

Rajat Mahajan has updated the pull request incrementally with one additional 
commit since the last revision:

  leave opening bracre as is

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/22932/files
  - new: https://git.openjdk.org/jdk/pull/22932/files/e22d8119..505f804b

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=22932&range=04
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=22932&range=03-04

  Stats: 2 lines in 1 file changed: 1 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/22932.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/22932/head:pull/22932

PR: https://git.openjdk.org/jdk/pull/22932

Reply via email to