diff options
author | Konstantin Belousov <kib@FreeBSD.org> | 2021-03-03 19:40:34 +0000 |
---|---|---|
committer | Konstantin Belousov <kib@FreeBSD.org> | 2021-03-12 11:31:08 +0000 |
commit | 2af934cc15bd8e7daa2daeb806321d0daddf3b7a (patch) | |
tree | 2c8e1b47e077f1dde3c28f354a3b7174253164f0 /sys | |
parent | f776c54cee81b4297b59ffe87a0f154e3924ee7f (diff) | |
download | src-2af934cc15bd8e7daa2daeb806321d0daddf3b7a.tar.gz src-2af934cc15bd8e7daa2daeb806321d0daddf3b7a.zip |
Assert that um_softdep is NULL on free(ump), i.e. softdep_unmount() was called
Reviewed by: mckusick
Tested by: pho
Sponsored by: The FreeBSD Foundation
MFC after: 2 weeks
Differential revision: https://reviews.freebsd.org/D29178
Diffstat (limited to 'sys')
-rw-r--r-- | sys/ufs/ffs/ffs_vfsops.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/sys/ufs/ffs/ffs_vfsops.c b/sys/ufs/ffs/ffs_vfsops.c index 584a20adb54d..321ed03f7f67 100644 --- a/sys/ufs/ffs/ffs_vfsops.c +++ b/sys/ufs/ffs/ffs_vfsops.c @@ -1355,6 +1355,7 @@ out: free(mp->mnt_gjprovider, M_UFSMNT); mp->mnt_gjprovider = NULL; } + MPASS(ump->um_softdep == NULL); free(ump, M_UFSMNT); mp->mnt_data = NULL; } @@ -1537,6 +1538,7 @@ ffs_unmount(mp, mntflags) UFS_UNLOCK(ump); if (MOUNTEDSOFTDEP(mp)) softdep_unmount(mp); + MPASS(ump->um_softdep == NULL); if (fs->fs_ronly == 0 || ump->um_fsckpid > 0) { fs->fs_clean = fs->fs_flags & (FS_UNCLEAN|FS_NEEDSFSCK) ? 0 : 1; error = ffs_sbupdate(ump, MNT_WAIT, 0); |