Aren
/
NetBSD
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

bread() will always return a valid bp. So remplace the (always true) if (bp) 

with a KASSERT.
Should fix Coverity ID 2444.
This commit is contained in:
bouyer 2006-03-18 12:48:38 +00:00
parent 57c63d8912
commit f7123013b8
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
sys/ufs/ext2fs/ext2fs_vfsops.c
View File

@ -1,4 +1,4 @@
/* $NetBSD: ext2fs_vfsops.c,v 1.95 2006/02/21 04:32:39 thorpej Exp $ */
/* $NetBSD: ext2fs_vfsops.c,v 1.96 2006/03/18 12:48:38 bouyer Exp $ */
/*
* Copyright (c) 1989, 1991, 1993, 1994
@ -65,7 +65,7 @@
*/
#include <sys/cdefs.h>
__KERNEL_RCSID(0, "$NetBSD: ext2fs_vfsops.c,v 1.95 2006/02/21 04:32:39 thorpej Exp $");
__KERNEL_RCSID(0, "$NetBSD: ext2fs_vfsops.c,v 1.96 2006/03/18 12:48:38 bouyer Exp $");
#if defined(_KERNEL_OPT)
#include "opt_compat_netbsd.h"
@ -702,8 +702,8 @@ ext2fs_mountfs(struct vnode *devvp, struct mount *mp, struct lwp *l)
return (0);
out:
if (bp)
brelse(bp);
KASSERT(bp != NULL);
brelse(bp);
if (ump) {
free(ump->um_e2fs, M_UFSMNT);
free(ump, M_UFSMNT);