Re: RFR: 8313856: Replace VLA with malloc in pango

2023-08-07 Thread Kevin Rushforth
On Mon, 7 Aug 2023 04:55:23 GMT, Jayathirth D V wrote: > We should not use stack for array memory allocations in JNI. > Updated pango.c to use malloc for arrays and release them at appropriate > places. Looks good. One slight clarification: > We should not use stack for array memory allocatio

Re: RFR: 8313856: Replace VLA with malloc in pango

2023-08-06 Thread Ambarish Rapte
On Mon, 7 Aug 2023 04:55:23 GMT, Jayathirth D V wrote: > We should not use stack for array memory allocations in JNI. > Updated pango.c to use malloc for arrays and release them at appropriate > places. LGTM - Marked as reviewed by arapte (Reviewer). PR Review: https://git.openjd