Hi Maks,

On Tue, Jul 02, 2024 at 12:33:06AM +0300, Maks Mishin wrote:
> A string is copied into the buffer 's' of size 577 without checking its 
> length first at sparc_attrs.c:95.

I don't really like having runtime asserts in the library code.
The buffer really should be big enough as the comment says:

  /* NAME should be big enough to hold any possible comma-separated
     list (no repetitions allowed) of attribute names from one of the
     arrays above.  */
  static char name[32*17+32+1];

Maybe there is a way to add a static assert for this?

Cheers,

Mark

Reply via email to