Hi Goetz,
Maybe instead of increasing the stack size we could move the increment
from the assignment to the previous if statement where we check for the
overwrite possibility?
There are similar code patterns in this file.
Also there is almost identical file LigatureSubstProc.cpp which also
contains similar code.
Thanks,
Vadim
On 20.07.2016 16:13, Lindenmaier, Goetz wrote:
Hi
This changes fixes two memory issues.
In awt_PrintControl.cpp, a wrong pointer is freed.
In LigatureSubstProc2.cpp, line 157:
stack[++mm] = componentGlyph;
can overwrite the stack by one element. It will write
stack[nComponents], because ++mm increments before
accessing the array.
Fix: increase the size of the array by one.
Please review this change:
http://cr.openjdk.java.net/~goetz/wr16/8161923-jdkMem/webrev.01/
<http://cr.openjdk.java.net/%7Egoetz/wr16/8161923-jdkMem/webrev.01/>
Best regards,
Goetz.