Paul Eggert wrote: > > IMO, it would be better to have code like this in base64_encode_alloc: > > > > if (inlen < 0) > > /* This argument is invalid, since the API change from 2021-08-28. */ > > abort (); > > Another possibility would be to treat inlen < 0 the same as integer > overflow. I could go either way.
Yes, either an abort() or an integer overflow return indicator would be better than undefined behaviour, for something that was defined behaviour until last week. Bruno
