Commit 7d683a09990ff095a91b6e724ecee0ff8733274a
Committed by
Al Viro
1 parent
971b2e8a3f
Exists in
master
and in
39 other branches
wrong type for 'magic' argument in simple_fill_super()
It's used to superblock ->s_magic, which is unsigned long. Signed-off-by: Roberto Sassu <roberto.sassu@polito.it> Reviewed-by: Mimi Zohar <zohar@us.ibm.com> Signed-off-by: Eric Paris <eparis@redhat.com> CC: stable@kernel.org Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Showing 2 changed files with 3 additions and 2 deletions Side-by-side Diff
fs/libfs.c
... | ... | @@ -489,7 +489,8 @@ |
489 | 489 | * unique inode values later for this filesystem, then you must take care |
490 | 490 | * to pass it an appropriate max_reserved value to avoid collisions. |
491 | 491 | */ |
492 | -int simple_fill_super(struct super_block *s, int magic, struct tree_descr *files) | |
492 | +int simple_fill_super(struct super_block *s, unsigned long magic, | |
493 | + struct tree_descr *files) | |
493 | 494 | { |
494 | 495 | struct inode *inode; |
495 | 496 | struct dentry *root; |
include/linux/fs.h
... | ... | @@ -2388,7 +2388,7 @@ |
2388 | 2388 | extern const struct inode_operations simple_dir_inode_operations; |
2389 | 2389 | struct tree_descr { char *name; const struct file_operations *ops; int mode; }; |
2390 | 2390 | struct dentry *d_alloc_name(struct dentry *, const char *); |
2391 | -extern int simple_fill_super(struct super_block *, int, struct tree_descr *); | |
2391 | +extern int simple_fill_super(struct super_block *, unsigned long, struct tree_descr *); | |
2392 | 2392 | extern int simple_pin_fs(struct file_system_type *, struct vfsmount **mount, int *count); |
2393 | 2393 | extern void simple_release_fs(struct vfsmount **mount, int *count); |
2394 | 2394 |