From 3b0ee7f583dc347c4aecb57d35830d2f99956e5c Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Tue, 15 Feb 2022 17:28:17 -0500 Subject: [PATCH] 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. Part of a general push to remove off-the-beaten-track warnings where we can easily do so. --- src/backend/utils/adt/tsvector_op.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/backend/utils/adt/tsvector_op.c b/src/backend/utils/adt/tsvector_op.c index 3a138186f1f..8ba9ef22f56 100644 --- a/src/backend/utils/adt/tsvector_op.c +++ b/src/backend/utils/adt/tsvector_op.c @@ -1167,7 +1167,7 @@ tsCompareString(char *a, int lena, char *b, int lenb, bool prefix) } else { - cmp = memcmp(a, b, Min(lena, lenb)); + cmp = memcmp(a, b, Min((unsigned int) lena, (unsigned int) lenb)); if (prefix) {