Re: RFR: JDK-8260426: awt debug_mem.c DMem_AllocateBlock might leak memory

2021-01-27 Thread Alexey Ivanov
On Wed, 27 Jan 2021 09:56:05 GMT, Matthias Baesken  wrote:

> It seems that the function DMem_AllocateBlock (debug_mem.c) has an early 
> return that might lead to memory leaks.
> See also the sonar issue Potential leak of memory pointed to by 'header' :
> https://sonarcloud.io/project/issues?id=shipilev_jdk&languages=c&open=AXck8B7zBBG2CXpcngZM&resolved=false&severities=BLOCKER&types=BUG

Marked as reviewed by aivanov (Reviewer).

-

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


Re: RFR: JDK-8260426: awt debug_mem.c DMem_AllocateBlock might leak memory

2021-01-27 Thread Prasanta Sadhukhan
On Wed, 27 Jan 2021 09:56:05 GMT, Matthias Baesken  wrote:

> It seems that the function DMem_AllocateBlock (debug_mem.c) has an early 
> return that might lead to memory leaks.
> See also the sonar issue Potential leak of memory pointed to by 'header' :
> https://sonarcloud.io/project/issues?id=shipilev_jdk&languages=c&open=AXck8B7zBBG2CXpcngZM&resolved=false&severities=BLOCKER&types=BUG

Marked as reviewed by psadhukhan (Reviewer).

-

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


RFR: JDK-8260426: awt debug_mem.c DMem_AllocateBlock might leak memory

2021-01-27 Thread Matthias Baesken
It seems that the function DMem_AllocateBlock (debug_mem.c) has an early return 
that might lead to memory leaks.
See also the sonar issue Potential leak of memory pointed to by 'header' :
https://sonarcloud.io/project/issues?id=shipilev_jdk&languages=c&open=AXck8B7zBBG2CXpcngZM&resolved=false&severities=BLOCKER&types=BUG

-

Commit messages:
 - JDK-8260426

Changes: https://git.openjdk.java.net/jdk/pull/2252/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=2252&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8260426
  Stats: 2 lines in 1 file changed: 1 ins; 0 del; 1 mod
  Patch: https://git.openjdk.java.net/jdk/pull/2252.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/2252/head:pull/2252

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