From c71fe716401f6919068f84b389dcd1b7ec2b8b0e Mon Sep 17 00:00:00 2001
From: Pawel Jakub Dawidek <pawel@dawidek.net>
Date: Thu, 13 Apr 2023 08:15:05 +0900
Subject: [PATCH] Fix data corruption when cloning embedded blocks

Don't overwrite blk_phys_birth, as for embedded blocks it is part of
the payload.

Reviewed-by: Richard Yao <richard.yao@alumni.stonybrook.edu>
Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Signed-off-by: Pawel Jakub Dawidek <pawel@dawidek.net>
Issue #13392
Closes #14739
---
 module/zfs/dmu.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/module/zfs/dmu.c b/module/zfs/dmu.c
index ce985d833..cda1472a7 100644
--- a/module/zfs/dmu.c
+++ b/module/zfs/dmu.c
@@ -2312,8 +2312,10 @@ dmu_brt_clone(objset_t *os, uint64_t object, uint64_t offset, uint64_t length,
 			dl->dr_overridden_by.blk_phys_birth = 0;
 		} else {
 			dl->dr_overridden_by.blk_birth = dr->dr_txg;
-			dl->dr_overridden_by.blk_phys_birth =
-			    BP_PHYSICAL_BIRTH(bp);
+			if (!BP_IS_EMBEDDED(bp)) {
+				dl->dr_overridden_by.blk_phys_birth =
+				    BP_PHYSICAL_BIRTH(bp);
+			}
 		}
 
 		mutex_exit(&db->db_mtx);