ocfs2: Use ERR_CAST() to avoid cross-structure cast
PATCH 1/1 ocfs2: Use ERR_CAST() to avoid cross-structure cast
When trying to propegate an error result, the error return path attempts to retain the error, but does this with an open cast across very different types, which the upcoming structure layout randomization plugin flags as being potentially dangerous in the face of randomization. This is a false positive, but what this code actually wants to do is use ERR_CAST() to retain the error value.
Signed-off-by Kees Cook <[email protected]>
--- fs/ocfs2/export.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
fs/ocfs2/export.c +1 -1
@@ -119,7 +119,7 @@ static struct dentry *ocfs2_get_dentry(struct super_block *sb,if (IS_ERR (inode)) {mlog_errno (PTR_ERR (inode)); - result = (void *)inode; + result =ERR_CAST (inode); goto bail; }
Signature · 6 lines
2.7.4 -- Kees Cook Pixel Security