Unverified Commit 89491faf authored by Wedson Almeida Filho's avatar Wedson Almeida Filho Committed by Christian Brauner
Browse files

gfs2: move gfs2_xattr_handlers_max to .rodata 



This makes it harder for accidental or malicious changes to
gfs2_xattr_handlers_max at runtime.

Cc: Bob Peterson <rpeterso@redhat.com>
Cc: Andreas Gruenbacher <agruenba@redhat.com>
Cc: cluster-devel@redhat.com
Signed-off-by: default avatarWedson Almeida Filho <walmeida@microsoft.com>
Link: https://lore.kernel.org/r/20230930050033.41174-13-wedsonaf@gmail.com


Signed-off-by: default avatarChristian Brauner <brauner@kernel.org>
parent 34271edb

fs/gfs2/super.h

+2 −2
Original line number Diff line number Diff line
@@ -60,8 +60,8 @@ extern const struct export_operations gfs2_export_ops; 
extern const struct super_operations gfs2_super_ops;

extern const struct dentry_operations gfs2_dops;



extern const struct xattr_handler *gfs2_xattr_handlers_max[];

extern const struct xattr_handler **gfs2_xattr_handlers_min;

extern const struct xattr_handler * const gfs2_xattr_handlers_max[];

extern const struct xattr_handler * const *gfs2_xattr_handlers_min;



#endif /* __SUPER_DOT_H__ */

fs/gfs2/xattr.c

+2 −2
Original line number Diff line number Diff line
@@ -1494,7 +1494,7 @@ static const struct xattr_handler gfs2_xattr_trusted_handler = { 
	.set    = gfs2_xattr_set,

};



const struct xattr_handler *gfs2_xattr_handlers_max[] = {

const struct xattr_handler * const gfs2_xattr_handlers_max[] = {

	/* GFS2_FS_FORMAT_MAX */

	&gfs2_xattr_trusted_handler,
@@ -1504,4 +1504,4 @@ const struct xattr_handler *gfs2_xattr_handlers_max[] = { 
	NULL,

};



const struct xattr_handler **gfs2_xattr_handlers_min = gfs2_xattr_handlers_max + 1;
 
const struct xattr_handler * const *gfs2_xattr_handlers_min = gfs2_xattr_handlers_max + 1;