diff options
| author |   beck <beck@openbsd.org> | 2014-04-22 20:14:39 +0000 |
|---|---|---|
| committer | beck <beck@openbsd.org> | 2014-04-22 20:14:39 +0000 |
| commit | c767a7f23ef10448da21641945be9484b221c382 (patch) | |
| tree | 1073057cec7496359ff34050acabf33edc0547bd | |
| parent | turns out there are exactly 3 ports that actually use longnames: (diff) | |
| download | wireguard-openbsd-c767a7f23ef10448da21641945be9484b221c382.tar.xz wireguard-openbsd-c767a7f23ef10448da21641945be9484b221c382.zip | |
Fix issue where we could jump into getdirtybuf without splbio() on a retry
that probably crashed espie.
ok tedu@
| -rw-r--r-- | sys/ufs/ffs/ffs_softdep.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sys/ufs/ffs/ffs_softdep.c b/sys/ufs/ffs/ffs_softdep.c index 4c6efb6d4cc..cb4ea661f6c 100644 --- a/sys/ufs/ffs/ffs_softdep.c +++ b/sys/ufs/ffs/ffs_softdep.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ffs_softdep.c,v 1.125 2014/02/04 01:04:03 tedu Exp $ */ +/* $OpenBSD: ffs_softdep.c,v 1.126 2014/04/22 20:14:39 beck Exp $ */ /* * Copyright 1998, 2000 Marshall Kirk McKusick. All Rights Reserved. @@ -5151,9 +5151,9 @@ flush_pagedep_deps(struct vnode *pvp, struct mount *mp, if ((inodedep->id_state & DEPCOMPLETE) == 0) { bp = inodedep->id_buf; gotit = getdirtybuf(bp, MNT_WAIT); - FREE_LOCK(&lk); if (gotit == -1) goto retry; + FREE_LOCK(&lk); if (gotit && (error = bwrite(bp)) != 0) break; ACQUIRE_LOCK(&lk); |