Hi,

On 2022-02-15 22:28:23 +0000, Tom Lane wrote:
> Ensure that length argument of memcmp() isn't seen as negative.
> 
> I think this will shut up a weird warning from buildfarm member
> serinus.  Perhaps it'd be better to change tsCompareString's
> length arguments to unsigned, but that seems more invasive
> than is justified.

Just FYI, I'd reported this to gcc some time last year
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100442
and then forgot about it.

They're not wrong there is the possibility of exceeding the bounds, but the
error message still is quite confusing.

Greetings,

Andres Freund


Reply via email to