diff options
| author | Eric Paris <eparis@redhat.com> | 2010-04-07 15:14:27 -0400 |
|---|---|---|
| committer | James Morris <jmorris@namei.org> | 2010-04-11 22:18:30 -0400 |
| commit | 4b61d12c84293ac061909f27f567c1905e4d90e3 (patch) | |
| tree | dfab10311d1b6be7667600307c72379c6779d32d | |
| parent | 231923bd0e06cba69f7c2028f4a68602b8d22160 (diff) | |
security: remove dead hook sb_umount_busy
Unused hook. Remove.
Signed-off-by: Eric Paris <eparis@redhat.com>
Signed-off-by: James Morris <jmorris@namei.org>
| -rw-r--r-- | fs/namespace.c | 2 | ||||
| -rw-r--r-- | include/linux/security.h | 11 | ||||
| -rw-r--r-- | security/capability.c | 5 | ||||
| -rw-r--r-- | security/security.c | 5 |
4 files changed, 0 insertions, 23 deletions
diff --git a/fs/namespace.c b/fs/namespace.c index 8aea78c8e760..6c9ca7358aad 100644 --- a/fs/namespace.c +++ b/fs/namespace.c | |||
| @@ -1116,8 +1116,6 @@ static int do_umount(struct vfsmount *mnt, int flags) | |||
| 1116 | retval = 0; | 1116 | retval = 0; |
| 1117 | } | 1117 | } |
| 1118 | spin_unlock(&vfsmount_lock); | 1118 | spin_unlock(&vfsmount_lock); |
| 1119 | if (retval) | ||
| 1120 | security_sb_umount_busy(mnt); | ||
| 1121 | up_write(&namespace_sem); | 1119 | up_write(&namespace_sem); |
| 1122 | release_mounts(&umount_list); | 1120 | release_mounts(&umount_list); |
| 1123 | return retval; | 1121 | return retval; |
diff --git a/include/linux/security.h b/include/linux/security.h index 354b0ceefc87..b206795c09f7 100644 --- a/include/linux/security.h +++ b/include/linux/security.h | |||
| @@ -272,12 +272,6 @@ static inline void security_free_mnt_opts(struct security_mnt_opts *opts) | |||
| 272 | * @mnt contains the mounted file system. | 272 | * @mnt contains the mounted file system. |
| 273 | * @flags contains the unmount flags, e.g. MNT_FORCE. | 273 | * @flags contains the unmount flags, e.g. MNT_FORCE. |
| 274 | * Return 0 if permission is granted. | 274 | * Return 0 if permission is granted. |
| 275 | * @sb_umount_busy: | ||
| 276 | * Handle a failed umount of the @mnt mounted filesystem, e.g. re-opening | ||
| 277 | * any files that were closed by umount_close. This hook is called during | ||
| 278 | * an umount operation if the umount fails after a call to the | ||
| 279 | * umount_close hook. | ||
| 280 | * @mnt contains the mounted filesystem. | ||
| 281 | * @sb_post_remount: | 275 | * @sb_post_remount: |
| 282 | * Update the security module's state when a filesystem is remounted. | 276 | * Update the security module's state when a filesystem is remounted. |
| 283 | * This hook is only called if the remount was successful. | 277 | * This hook is only called if the remount was successful. |
| @@ -1474,7 +1468,6 @@ struct security_operations { | |||
| 1474 | int (*sb_mount) (char *dev_name, struct path *path, | 1468 | int (*sb_mount) (char *dev_name, struct path *path, |
| 1475 | char *type, unsigned long flags, void *data); | 1469 | char *type, unsigned long flags, void *data); |
| 1476 | int (*sb_umount) (struct vfsmount *mnt, int flags); | 1470 | int (*sb_umount) (struct vfsmount *mnt, int flags); |
| 1477 | void (*sb_umount_busy) (struct vfsmount *mnt); | ||
| 1478 | void (*sb_post_remount) (struct vfsmount *mnt, | 1471 | void (*sb_post_remount) (struct vfsmount *mnt, |
| 1479 | unsigned long flags, void *data); | 1472 | unsigned long flags, void *data); |
| 1480 | void (*sb_post_addmount) (struct vfsmount *mnt, | 1473 | void (*sb_post_addmount) (struct vfsmount *mnt, |
| @@ -1771,7 +1764,6 @@ int security_sb_statfs(struct dentry *dentry); | |||
| 1771 | int security_sb_mount(char *dev_name, struct path *path, | 1764 | int security_sb_mount(char *dev_name, struct path *path, |
| 1772 | char *type, unsigned long flags, void *data); | 1765 | char *type, unsigned long flags, void *data); |
| 1773 | int security_sb_umount(struct vfsmount *mnt, int flags); | 1766 | int security_sb_umount(struct vfsmount *mnt, int flags); |
| 1774 | void security_sb_umount_busy(struct vfsmount *mnt); | ||
| 1775 | void security_sb_post_remount(struct vfsmount *mnt, unsigned long flags, void *data); | 1767 | void security_sb_post_remount(struct vfsmount *mnt, unsigned long flags, void *data); |
| 1776 | void security_sb_post_addmount(struct vfsmount *mnt, struct path *mountpoint); | 1768 | void security_sb_post_addmount(struct vfsmount *mnt, struct path *mountpoint); |
| 1777 | int security_sb_pivotroot(struct path *old_path, struct path *new_path); | 1769 | int security_sb_pivotroot(struct path *old_path, struct path *new_path); |
| @@ -2089,9 +2081,6 @@ static inline int security_sb_umount(struct vfsmount *mnt, int flags) | |||
| 2089 | return 0; | 2081 | return 0; |
| 2090 | } | 2082 | } |
| 2091 | 2083 | ||
| 2092 | static inline void security_sb_umount_busy(struct vfsmount *mnt) | ||
| 2093 | { } | ||
| 2094 | |||
| 2095 | static inline void security_sb_post_remount(struct vfsmount *mnt, | 2084 | static inline void security_sb_post_remount(struct vfsmount *mnt, |
| 2096 | unsigned long flags, void *data) | 2085 | unsigned long flags, void *data) |
| 2097 | { } | 2086 | { } |
diff --git a/security/capability.c b/security/capability.c index db72f4f5a9e1..476d43030906 100644 --- a/security/capability.c +++ b/security/capability.c | |||
| @@ -85,10 +85,6 @@ static int cap_sb_umount(struct vfsmount *mnt, int flags) | |||
| 85 | return 0; | 85 | return 0; |
| 86 | } | 86 | } |
| 87 | 87 | ||
| 88 | static void cap_sb_umount_busy(struct vfsmount *mnt) | ||
| 89 | { | ||
| 90 | } | ||
| 91 | |||
| 92 | static void cap_sb_post_remount(struct vfsmount *mnt, unsigned long flags, | 88 | static void cap_sb_post_remount(struct vfsmount *mnt, unsigned long flags, |
| 93 | void *data) | 89 | void *data) |
| 94 | { | 90 | { |
| @@ -933,7 +929,6 @@ void security_fixup_ops(struct security_operations *ops) | |||
| 933 | set_to_cap_if_null(ops, sb_statfs); | 929 | set_to_cap_if_null(ops, sb_statfs); |
| 934 | set_to_cap_if_null(ops, sb_mount); | 930 | set_to_cap_if_null(ops, sb_mount); |
| 935 | set_to_cap_if_null(ops, sb_umount); | 931 | set_to_cap_if_null(ops, sb_umount); |
| 936 | set_to_cap_if_null(ops, sb_umount_busy); | ||
| 937 | set_to_cap_if_null(ops, sb_post_remount); | 932 | set_to_cap_if_null(ops, sb_post_remount); |
| 938 | set_to_cap_if_null(ops, sb_post_addmount); | 933 | set_to_cap_if_null(ops, sb_post_addmount); |
| 939 | set_to_cap_if_null(ops, sb_pivotroot); | 934 | set_to_cap_if_null(ops, sb_pivotroot); |
diff --git a/security/security.c b/security/security.c index 591c25fd430f..c1341fbe98ca 100644 --- a/security/security.c +++ b/security/security.c | |||
| @@ -311,11 +311,6 @@ int security_sb_umount(struct vfsmount *mnt, int flags) | |||
| 311 | return security_ops->sb_umount(mnt, flags); | 311 | return security_ops->sb_umount(mnt, flags); |
| 312 | } | 312 | } |
| 313 | 313 | ||
| 314 | void security_sb_umount_busy(struct vfsmount *mnt) | ||
| 315 | { | ||
| 316 | security_ops->sb_umount_busy(mnt); | ||
| 317 | } | ||
| 318 | |||
| 319 | void security_sb_post_remount(struct vfsmount *mnt, unsigned long flags, void *data) | 314 | void security_sb_post_remount(struct vfsmount *mnt, unsigned long flags, void *data) |
| 320 | { | 315 | { |
| 321 | security_ops->sb_post_remount(mnt, flags, data); | 316 | security_ops->sb_post_remount(mnt, flags, data); |
