Fix #2078

wtdcode · wtdcode · commit 6974b535884a · 2025-01-04T15:57:02.000+08:00
We shall only go through the else branch for code_read
diff --git a/qemu/accel/tcg/cputlb.c b/qemu/accel/tcg/cputlb.c
@@ -1665,7 +1665,7 @@ load_helper(CPUArchState *env, target_ulong addr, TCGMemOpIdx oi,
                 return 0;
             }
         }
-    } else {
+    } else if (code_read) {
         // code fetching
         // Unicorn: callback on fetch from NX
         if (mr != NULL && !(mr->perms & UC_PROT_EXEC)) {  // non-executable

Original file line number	Diff line number	Diff line change
`@@ -1665,7 +1665,7 @@ load_helper(CPUArchState *env, target_ulong addr, TCGMemOpIdx oi,`
`1665`	`1665`	`return 0;`
`1666`	`1666`	`}`
`1667`	`1667`	`}`
`1668`		`- } else {`
	`1668`	`+ } else if (code_read) {`
`1669`	`1669`	`// code fetching`
`1670`	`1670`	`// Unicorn: callback on fetch from NX`
`1671`	`1671`	`if (mr != NULL && !(mr->perms & UC_PROT_EXEC)) { // non-executable`