Avoid runtime failure in CopyXLogRecordToWAL with sanitizers (cf. 46ab07f) (#554)

alexanderlaw · MMeent · commit d2fc3dbfb295 · 2025-07-08T15:54:20.000+02:00
diff --git a/src/backend/access/transam/xlog.c b/src/backend/access/transam/xlog.c
@@ -1332,7 +1332,8 @@ CopyXLogRecordToWAL(int write_len, bool isLogSwitch, XLogRecData *rdata,
 		}
 
 		Assert(CurrPos % XLOG_BLCKSZ >= SizeOfXLogShortPHD || rdata_len == 0);
-		memcpy(currpos, rdata_data, rdata_len);
+		if (rdata_len > 0)
+			memcpy(currpos, rdata_data, rdata_len);
 		currpos += rdata_len;
 		CurrPos += rdata_len;
 		freespace -= rdata_len;

Original file line number	Diff line number	Diff line change
`@@ -1332,7 +1332,8 @@ CopyXLogRecordToWAL(int write_len, bool isLogSwitch, XLogRecData *rdata,`
`1332`	`1332`	`}`
`1333`	`1333`
`1334`	`1334`	`Assert(CurrPos % XLOG_BLCKSZ >= SizeOfXLogShortPHD \|\| rdata_len == 0);`
`1335`		`- memcpy(currpos, rdata_data, rdata_len);`
	`1335`	`+ if (rdata_len > 0)`
	`1336`	`+ memcpy(currpos, rdata_data, rdata_len);`
`1336`	`1337`	`currpos += rdata_len;`
`1337`	`1338`	`CurrPos += rdata_len;`
`1338`	`1339`	`freespace -= rdata_len;`