Fix oversight in commit aa1f93a33.

Since the remote column names of a foreign table could be longer than
NAMEDATALEN, remattrmap_cmp(), which compares such column names, should
have used strcmp(), not strncmp() with n=NAMEDATALEN.

Author: Chao Li <li.evan.chao@gmail.com>
Reviewed-by: Etsuro Fujita <etsuro.fujita@gmail.com>
Discussion: https://postgr.es/m/81D981EB-ECC1-495D-8EAC-5CFB67B2CF77%40gmail.com
This commit is contained in:
Etsuro Fujita 2026-06-14 16:00:00 +09:00
parent da1eff08a5
commit 0131e8fc50

View file

@ -6024,7 +6024,7 @@ remattrmap_cmp(const void *v1, const void *v2)
const RemoteAttributeMapping *r1 = v1;
const RemoteAttributeMapping *r2 = v2;
return strncmp(r1->remote_attname, r2->remote_attname, NAMEDATALEN);
return strcmp(r1->remote_attname, r2->remote_attname);
}
/*