Skip to content

Commit e0f26fc

Browse files
committed
Correct replication origin's use of UINT16_MAX to PG_UINT16_MAX.
We can't rely on UINT16_MAX being present, which is why we introduced PG_UINT16_MAX... Buildfarm animal bowerbird via Andrew Gierth.
1 parent 49601ab commit e0f26fc

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

src/backend/replication/logical/origin.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -266,7 +266,7 @@ replorigin_create(char *roname)
266266

267267
rel = heap_open(ReplicationOriginRelationId, ExclusiveLock);
268268

269-
for (roident = InvalidOid + 1; roident < UINT16_MAX; roident++)
269+
for (roident = InvalidOid + 1; roident < PG_UINT16_MAX; roident++)
270270
{
271271
bool nulls[Natts_pg_replication_origin];
272272
Datum values[Natts_pg_replication_origin];

src/include/replication/origin.h

+1-1
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ typedef struct xl_replorigin_drop
3030
#define XLOG_REPLORIGIN_DROP 0x10
3131

3232
#define InvalidRepOriginId 0
33-
#define DoNotReplicateId UINT16_MAX
33+
#define DoNotReplicateId PG_UINT16_MAX
3434

3535
extern PGDLLIMPORT RepOriginId replorigin_sesssion_origin;
3636
extern PGDLLIMPORT XLogRecPtr replorigin_sesssion_origin_lsn;

0 commit comments

Comments
 (0)