No need for extra code to log freezing zero tuples. Callers already check that

they are freezing a nonzero amount anyway.

diff --git a/src/backend/access/heap/heapam.c b/src/backend/access/heap/heapam.c
index 1d43b0b866759ac908836a7ac9cbe8109574f1ea..c528c7cf80e49a4ed05fe30b57f3825defaddd42 100644 (file)
--- a/src/backend/access/heap/heapam.c
+++ b/src/backend/access/heap/heapam.c
@@ -3818,6 +3818,8 @@ log_heap_freeze(Relation reln, Buffer buffer,
 
        /* Caller should not call me on a temp relation */
        Assert(!reln->rd_istemp);
+       /* nor when there are no tuples to freeze */
+       Assert(offcnt > 0);
 
        xlrec.node = reln->rd_node;
        xlrec.block = BufferGetBlockNumber(buffer);
@@ -3833,16 +3835,8 @@ log_heap_freeze(Relation reln, Buffer buffer,
         * it is.  When XLogInsert stores the whole buffer, the offsets array need
         * not be stored too.
         */
-       if (offcnt > 0)
-       {
-               rdata[1].data = (char *) offsets;
-               rdata[1].len = offcnt * sizeof(OffsetNumber);
-       }
-       else
-       {
-               rdata[1].data = NULL;
-               rdata[1].len = 0;
-       }
+       rdata[1].data = (char *) offsets;
+       rdata[1].len = offcnt * sizeof(OffsetNumber);
        rdata[1].buffer = buffer;
        rdata[1].buffer_std = true;
        rdata[1].next = NULL;