We're testing the nullness of `sb' potentially after rereferencing it (although the compiler will have reordered things to avoid such a bug). Just remove the test - the superblock pointer shouldn't be null in there. --- 25-akpm/fs/dquot.c | 3 --- 1 files changed, 3 deletions(-) diff -puN fs/dquot.c~dquot-unneeded-test fs/dquot.c --- 25/fs/dquot.c~dquot-unneeded-test 2004-04-22 16:19:55.836081376 -0700 +++ 25-akpm/fs/dquot.c 2004-04-22 16:19:55.839080920 -0700 @@ -1295,9 +1295,6 @@ int vfs_quota_off(struct super_block *sb int cnt; struct quota_info *dqopt = sb_dqopt(sb); - if (!sb) - goto out; - /* We need to serialize quota_off() for device */ down(&dqopt->dqonoff_sem); for (cnt = 0; cnt < MAXQUOTAS; cnt++) { _