From owner-cvs-src@FreeBSD.ORG Tue Apr 4 15:29:52 2006 Return-Path: X-Original-To: cvs-src@FreeBSD.org Delivered-To: cvs-src@FreeBSD.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 5A4A016A401; Tue, 4 Apr 2006 15:29:52 +0000 (UTC) (envelope-from cel@FreeBSD.org) Received: from repoman.freebsd.org (repoman.freebsd.org [216.136.204.115]) by mx1.FreeBSD.org (Postfix) with ESMTP id 290B843D67; Tue, 4 Apr 2006 15:29:52 +0000 (GMT) (envelope-from cel@FreeBSD.org) Received: from repoman.freebsd.org (localhost [127.0.0.1]) by repoman.freebsd.org (8.13.1/8.13.1) with ESMTP id k34FTqrN062833; Tue, 4 Apr 2006 15:29:52 GMT (envelope-from cel@repoman.freebsd.org) Received: (from cel@localhost) by repoman.freebsd.org (8.13.1/8.13.1/Submit) id k34FTqOD062832; Tue, 4 Apr 2006 15:29:52 GMT (envelope-from cel) Message-Id: <200604041529.k34FTqOD062832@repoman.freebsd.org> From: Chuck Lever Date: Tue, 4 Apr 2006 15:29:51 +0000 (UTC) To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org X-FreeBSD-CVS-Branch: RELENG_6 Cc: Subject: cvs commit: src/sys/nfsserver nfs_srvsubs.c X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 04 Apr 2006 15:29:52 -0000 cel 2006-04-04 15:29:51 UTC FreeBSD src repository Modified files: (Branch: RELENG_6) sys/nfsserver nfs_srvsubs.c Log: rick says: The following bug was just identified in OpenBSD and it looks like the same bug exists in the other BSDen NFS servers. A Linux client (don't know which version, but you can look at http://bugzilla.kernel.org/show_bug.cgi?id=6256) does a Setattr of mtime to the server's time, where the file is mode 0664 and the client user has group access (ie. caller is not the file owner). The BSD servers fail the Setattr with EPERM, since the VA_UTIMES_NULL flag isn't set before doing the VOP_SETATTR. It seems to me that this should be allowed, since it is allowed for a local utimes(2). If so, the fix is to set VA_UTIMES_NULL for the "set-time-to-server-time" cases of setting atime and/or mtime. MFC of revision 1.140 to RELENG_6. Submitted by: rick@snowhite.cis.uoguelph.ca Reviewed by: cel Approved by: re (scottl), silby Revision Changes Path 1.136.2.2 +6 -0 src/sys/nfsserver/nfs_srvsubs.c