[src/trunk]: src/sys/ufs/ufs In ufs_extattr_stop(), if we haven't started yet...

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

details:   https://anonhg.NetBSD.org/src/rev/62530b6a612d
branches:  trunk
changeset: 584520:62530b6a612d
user:      rpaulo <rpaulo%NetBSD.org@localhost>
date:      Thu Sep 22 13:49:03 2005 +0000

description:
In ufs_extattr_stop(), if we haven't started yet, errno must be set
before bailing out.

>From FreeBSD.

diffstat:

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

diffs (30 lines):

diff -r 5231f97ed2e8 -r 62530b6a612d sys/ufs/ufs/ufs_extattr.c
--- a/sys/ufs/ufs/ufs_extattr.c Thu Sep 22 12:29:31 2005 +0000
+++ b/sys/ufs/ufs/ufs_extattr.c Thu Sep 22 13:49:03 2005 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: ufs_extattr.c,v 1.3 2005/09/12 16:10:11 rpaulo Exp $   */
+/*     $NetBSD: ufs_extattr.c,v 1.4 2005/09/22 13:49:03 rpaulo Exp $   */
 
 /*-
  * Copyright (c) 1999-2002 Robert N. M. Watson
@@ -48,7 +48,7 @@
  */
 
 #include <sys/cdefs.h>
-__RCSID("$NetBSD: ufs_extattr.c,v 1.3 2005/09/12 16:10:11 rpaulo Exp $");
+__RCSID("$NetBSD: ufs_extattr.c,v 1.4 2005/09/22 13:49:03 rpaulo Exp $");
 
 #include "opt_ffs.h"
 
@@ -566,8 +566,10 @@
         * If we haven't been started, no big deal.  Just short-circuit
         * the processing work.
         */
-       if (!(ump->um_extattr.uepm_flags & UFS_EXTATTR_UEPM_STARTED))
+       if (!(ump->um_extattr.uepm_flags & UFS_EXTATTR_UEPM_STARTED)) {
+               error = EOPNOTSUPP;
                goto unlock;
+       }
 
        while (LIST_FIRST(&ump->um_extattr.uepm_list) != NULL) {
                uele = LIST_FIRST(&ump->um_extattr.uepm_list);