[PATCH 1/2] mceliece6688128f: fix stack overflow crash on win64/wine

Jacob Bachmeyer jcb62281 at gmail.com
Sat Sep 27 01:19:00 CEST 2025


On 9/26/25 01:53, Jussi Kivilinna wrote:
> * cipher/mceliece6688128f.c (pk_gen): Remove 'mat' array allocation and
> rename function to ...
> (pk_gen_mat): ... this.
> (pk_gen): New warper for 'pk_gen_mat' with dynamic allocation of 'mat' array.

Typo?  Did you mean "wrapper"?

Why do it this way instead of just adding xmalloc()/xfree() calls to 
pk_gen()?  Also, should the buffer be zeroed before releasing it?


-- Jacob





More information about the Gcrypt-devel mailing list