erofs: use poison pointer to replace the hard-coded address
authorGao Xiang <hsiangkao@linux.alibaba.com>
Fri, 26 May 2023 20:14:59 +0000 (04:14 +0800)
committerGao Xiang <hsiangkao@linux.alibaba.com>
Sun, 18 Jun 2023 04:10:53 +0000 (12:10 +0800)
It's safer and cleaner to replace such hard-coded illegal pointer
with poison pointers.

Signed-off-by: Gao Xiang <hsiangkao@linux.alibaba.com>
Reviewed-by: Yue Hu <huyue2@coolpad.com>
Link: https://lore.kernel.org/r/20230526201459.128169-7-hsiangkao@linux.alibaba.com
fs/erofs/zdata.c

index 637a964ff110c2c571eddaa2a1409960a4030546..264bf553c28743099ee9744cba0aca3f304a142e 100644 (file)
@@ -91,10 +91,8 @@ struct z_erofs_pcluster {
        struct z_erofs_bvec compressed_bvecs[];
 };
 
-/* let's avoid the valid 32-bit kernel addresses */
-
 /* the end of a chain of pclusters */
-#define Z_EROFS_PCLUSTER_TAIL           ((void *)0x5F0ECAFE)
+#define Z_EROFS_PCLUSTER_TAIL           ((void *) 0x700 + POISON_POINTER_DELTA)
 #define Z_EROFS_PCLUSTER_NIL            (NULL)
 
 struct z_erofs_decompressqueue {