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

fix bug (pointed out as sequence point violation warning with current-ish gcc) 

caused by use of makedev(major,minor++).  makedev() now (since 32-bit
dev_t conversion) evaluates its second argument twice.
This commit is contained in:
cgd 2001-02-24 00:05:22 +00:00
parent 98af50970f
commit 4dddddfe62
1 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
sys/ufs/mfs/mfs_vfsops.c
View File

@ -1,4 +1,4 @@
/* $NetBSD: mfs_vfsops.c,v 1.31 2001/01/22 12:17:43 jdolecek Exp $ */
/* $NetBSD: mfs_vfsops.c,v 1.32 2001/02/24 00:05:22 cgd Exp $ */
/*
* Copyright (c) 1989, 1990, 1993, 1994
@ -197,7 +197,8 @@ mfs_initminiroot(base)
mountroot = mfs_mountroot;
mfs_rootbase = base;
mfs_rootsize = fs->fs_fsize * fs->fs_size;
rootdev = makedev(255, mfs_minor++);
rootdev = makedev(255, mfs_minor);
mfs_minor++;
return (mfs_rootsize);
}
@ -252,8 +253,9 @@ mfs_mount(mp, path, data, ndp, p)
if (error)
return (error);
devvp->v_type = VBLK;
if (checkalias(devvp, makedev(255, mfs_minor++), (struct mount *)0))
if (checkalias(devvp, makedev(255, mfs_minor), (struct mount *)0))
panic("mfs_mount: dup dev");
mfs_minor++;
mfsp = (struct mfsnode *)malloc(sizeof *mfsp, M_MFSNODE, M_WAITOK);
devvp->v_data = mfsp;
mfsp->mfs_baseoff = args.base;