From owner-cvs-src@FreeBSD.ORG Wed Apr 5 00:33:34 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 5E23D16A400; Wed, 5 Apr 2006 00:33:34 +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 2FCA843D46; Wed, 5 Apr 2006 00:33:34 +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 k350XYxE071812; Wed, 5 Apr 2006 00:33:34 GMT (envelope-from cel@repoman.freebsd.org) Received: (from cel@localhost) by repoman.freebsd.org (8.13.1/8.13.1/Submit) id k350XYkp071811; Wed, 5 Apr 2006 00:33:34 GMT (envelope-from cel) Message-Id: <200604050033.k350XYkp071811@repoman.freebsd.org> From: Chuck Lever Date: Wed, 5 Apr 2006 00:33:33 +0000 (UTC) To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org X-FreeBSD-CVS-Branch: RELENG_5 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: Wed, 05 Apr 2006 00:33:34 -0000 cel 2006-04-05 00:33:33 UTC FreeBSD src repository Modified files: (Branch: RELENG_5) 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_5. Submitted by: rick@snowhite.cis.uoguelph.ca Reviewed by: cel Approved by: re (scottl), silby Revision Changes Path 1.131.2.2 +6 -0 src/sys/nfsserver/nfs_srvsubs.c