[src/trunk]: src/sys/ufs/ufs Fix locking protocol to avoid a panic on extattr...

[manu <manu%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/c51cc49a2737
branches:  trunk
changeset: 767053:c51cc49a2737
user:      manu <manu%NetBSD.org@localhost>
date:      Thu Jul 07 14:56:45 2011 +0000

description:
Fix locking protocol to avoid a panic on extattrctl stop and on umount.

diffstat:

 sys/ufs/ufs/ufs_extattr.c |  5 +++--
 1 files changed, 3 insertions(+), 2 deletions(-)

diffs (26 lines):

diff -r 4cdb13d2d104 -r c51cc49a2737 sys/ufs/ufs/ufs_extattr.c
--- a/sys/ufs/ufs/ufs_extattr.c Thu Jul 07 14:29:59 2011 +0000
+++ b/sys/ufs/ufs/ufs_extattr.c Thu Jul 07 14:56:45 2011 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: ufs_extattr.c,v 1.34 2011/07/04 08:07:32 manu Exp $    */
+/*     $NetBSD: ufs_extattr.c,v 1.35 2011/07/07 14:56:45 manu Exp $    */
 
 /*-
  * Copyright (c) 1999-2002 Robert N. M. Watson
@@ -48,7 +48,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: ufs_extattr.c,v 1.34 2011/07/04 08:07:32 manu Exp $");
+__KERNEL_RCSID(0, "$NetBSD: ufs_extattr.c,v 1.35 2011/07/07 14:56:45 manu Exp $");
 
 #ifdef _KERNEL_OPT
 #include "opt_ffs.h"
@@ -133,6 +133,7 @@
        if (ump->um_extattr.uepm_lockcnt != 0) {
                KASSERT(mutex_owned(&ump->um_extattr.uepm_lock));
                ump->um_extattr.uepm_lockcnt--;
+               return;
        }
        mutex_exit(&ump->um_extattr.uepm_lock);
 }