<feed xmlns='http://www.w3.org/2005/Atom'>
<title>src/sys/nfsserver, branch release/6.2.0</title>
<subtitle>FreeBSD source tree</subtitle>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/'/>
<entry>
<title>MFC: The nfsm_srvpathsiz() macro in nfsrv_symlink() in nfs_serv.c</title>
<updated>2007-01-07T13:22:49+00:00</updated>
<author>
<name>Hiroki Sato</name>
<email>hrs@FreeBSD.org</email>
</author>
<published>2007-01-07T13:22:49+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=4b3d91785068d2e02d06fc7549e7286ea99b02de'/>
<id>4b3d91785068d2e02d06fc7549e7286ea99b02de</id>
<content type='text'>
should check length of the pathname in the range 0&lt;=n&lt;=NFS_MAXPATHLEN,
not 0&lt;n&lt;=NFS_MAXPATHLEN.  This fixes a minor interoperability problem
that the FreeBSD NFS server did not allow a symlink pointing the empty
pathname.

Approved by:	re (kensmith)
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
should check length of the pathname in the range 0&lt;=n&lt;=NFS_MAXPATHLEN,
not 0&lt;n&lt;=NFS_MAXPATHLEN.  This fixes a minor interoperability problem
that the FreeBSD NFS server did not allow a symlink pointing the empty
pathname.

Approved by:	re (kensmith)
</pre>
</div>
</content>
</entry>
<entry>
<title>MFC:	sys/nfsserver/nfs_serv.c	1.167</title>
<updated>2006-12-02T09:02:30+00:00</updated>
<author>
<name>Pawel Jakub Dawidek</name>
<email>pjd@FreeBSD.org</email>
</author>
<published>2006-12-02T09:02:30+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=e692dd83f7282464a676b7508158de6e7568cb63'/>
<id>e692dd83f7282464a676b7508158de6e7568cb63</id>
<content type='text'>
Protect nfsm_srvpathsiz() call with the nfsd_mtx lock.

Reviewed by:	mohans
Requested by:	kib
Approved by:	re (rwatson)
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Protect nfsm_srvpathsiz() call with the nfsd_mtx lock.

Reviewed by:	mohans
Requested by:	kib
Approved by:	re (rwatson)
</pre>
</div>
</content>
</entry>
<entry>
<title>MFC rev. 1.166:</title>
<updated>2006-11-03T09:16:48+00:00</updated>
<author>
<name>Konstantin Belousov</name>
<email>kib@FreeBSD.org</email>
</author>
<published>2006-11-03T09:16:48+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=426cd8e663c955b995a3c282190508232bd7f18b'/>
<id>426cd8e663c955b995a3c282190508232bd7f18b</id>
<content type='text'>
Fix leak in NAMEI zone caused by nfs server when VOP_RENAME fails.

Approved by:	re (hrs)
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Fix leak in NAMEI zone caused by nfs server when VOP_RENAME fails.

Approved by:	re (hrs)
</pre>
</div>
</content>
</entry>
<entry>
<title>MFC: Release references acquired by VOP_GETWRITEMOUNT() and vfs_getvfs().</title>
<updated>2006-09-27T00:37:46+00:00</updated>
<author>
<name>Tor Egge</name>
<email>tegge@FreeBSD.org</email>
</author>
<published>2006-09-27T00:37:46+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=a3eea63b66591af3457d79f857ca2c846b4a5a6b'/>
<id>a3eea63b66591af3457d79f857ca2c846b4a5a6b</id>
<content type='text'>
Approved by:	re (kensmith)
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Approved by:	re (kensmith)
</pre>
</div>
</content>
</entry>
<entry>
<title>MFC revision 1.99 nfs_srvsock.c</title>
<updated>2006-09-02T23:58:21+00:00</updated>
<author>
<name>Christian S.J. Peron</name>
<email>csjp@FreeBSD.org</email>
</author>
<published>2006-09-02T23:58:21+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=8b3e775474d3703875b462e708e87a0182c54cc9'/>
<id>8b3e775474d3703875b462e708e87a0182c54cc9</id>
<content type='text'>
MFC revision 1.111 mac_vfs.c
MFC revision 1.70 mac.h
MFC revision 1.71 mac_policy.h

MFC the mac_associate_nfsd_label entry point, only replace one of the
mac_policy_ops stub elements to ensure that we dont break compatability
with existing MAC modules.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
MFC revision 1.111 mac_vfs.c
MFC revision 1.70 mac.h
MFC revision 1.71 mac_policy.h

MFC the mac_associate_nfsd_label entry point, only replace one of the
mac_policy_ops stub elements to ensure that we dont break compatability
with existing MAC modules.
</pre>
</div>
</content>
</entry>
<entry>
<title>MFC revision 1.79 nfs.h</title>
<updated>2006-09-02T21:54:08+00:00</updated>
<author>
<name>Christian S.J. Peron</name>
<email>csjp@FreeBSD.org</email>
</author>
<published>2006-09-02T21:54:08+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=22be9a6e3eb87766dc2435dd9b65ae997e5a2353'/>
<id>22be9a6e3eb87766dc2435dd9b65ae997e5a2353</id>
<content type='text'>
MFC revision 1.161 nfs_serv.c
MFC revision 1.96 nfs_srvsock.c
MFC revision 1.105 nfs_syscalls.c

date: 2006/01/28 19:24:39;  author: csjp;  state: Exp;  lines: +1 -1
Manage the ucred for the NFS server using the crget/crfree API defined in
kern_prot.c. This API handles reference counting among many other things.
Notably, if MAC is compiled into the kernel, it will properly initialize the
MAC labels when the ucred is allocated.

This work is in preparation for a new MAC entry point which will be responsible
for properly initializing policy specific labels for the NFS server credential.
Utilization of the crfree/crget APIs reduce the complexity associated with
this label's management.

Submitted by:	green (with changes) [1]
Obtained from:	TrustedBSD Project
Discussed with:	rwatson, alfred

[1] I moved the ucred allocation outside the scope of the NFS server lock to
    prevent M_WAIKOK allocations from occurring with non-sleep-able locks held.
    Additionally, to reduce complexity, the ucred persist as long as the NFS
    server descriptor.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
MFC revision 1.161 nfs_serv.c
MFC revision 1.96 nfs_srvsock.c
MFC revision 1.105 nfs_syscalls.c

date: 2006/01/28 19:24:39;  author: csjp;  state: Exp;  lines: +1 -1
Manage the ucred for the NFS server using the crget/crfree API defined in
kern_prot.c. This API handles reference counting among many other things.
Notably, if MAC is compiled into the kernel, it will properly initialize the
MAC labels when the ucred is allocated.

This work is in preparation for a new MAC entry point which will be responsible
for properly initializing policy specific labels for the NFS server credential.
Utilization of the crfree/crget APIs reduce the complexity associated with
this label's management.

Submitted by:	green (with changes) [1]
Obtained from:	TrustedBSD Project
Discussed with:	rwatson, alfred

[1] I moved the ucred allocation outside the scope of the NFS server lock to
    prevent M_WAIKOK allocations from occurring with non-sleep-able locks held.
    Additionally, to reduce complexity, the ucred persist as long as the NFS
    server descriptor.
</pre>
</div>
</content>
</entry>
<entry>
<title>MFC: Free the server request cache on unload and use callout_drain().</title>
<updated>2006-08-11T19:13:25+00:00</updated>
<author>
<name>John Baldwin</name>
<email>jhb@FreeBSD.org</email>
</author>
<published>2006-08-11T19:13:25+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=03d58b8153769cdb1f4dbe671635a446f38f6875'/>
<id>03d58b8153769cdb1f4dbe671635a446f38f6875</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>MFC of the temporary fix for nfsd leaking GIANT.</title>
<updated>2006-06-13T10:50:29+00:00</updated>
<author>
<name>Konstantin Belousov</name>
<email>kib@FreeBSD.org</email>
</author>
<published>2006-06-13T10:50:29+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=b9a446158b7da012ad829a341177c6dbb7d2ac7d'/>
<id>b9a446158b7da012ad829a341177c6dbb7d2ac7d</id>
<content type='text'>
src/sys/nfsserver/nfs_serv.c rev. 1.165
src/sys/nfsserver/nfs_srvsubs.c rev. 1.141

Approved by:	pjd (mentor)
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
src/sys/nfsserver/nfs_serv.c rev. 1.165
src/sys/nfsserver/nfs_srvsubs.c rev. 1.141

Approved by:	pjd (mentor)
</pre>
</div>
</content>
</entry>
<entry>
<title>MFC r1.18:</title>
<updated>2006-04-30T03:55:45+00:00</updated>
<author>
<name>Kris Kennaway</name>
<email>kris@FreeBSD.org</email>
</author>
<published>2006-04-30T03:55:45+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=dca566cf7313670fdfdab054570fa5e5b83bdd06'/>
<id>dca566cf7313670fdfdab054570fa5e5b83bdd06</id>
<content type='text'>
&gt;  Bump up the NFS server dupreq cache limit to 2K (from 64). With a small
&gt;  duplicate request cache, under heavy load a lot of non-idempotent requests
&gt;  were getting served again, resulting in errors.

Approved by:	re (scottl)
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
&gt;  Bump up the NFS server dupreq cache limit to 2K (from 64). With a small
&gt;  duplicate request cache, under heavy load a lot of non-idempotent requests
&gt;  were getting served again, resulting in errors.

Approved by:	re (scottl)
</pre>
</div>
</content>
</entry>
<entry>
<title>rick says:</title>
<updated>2006-04-04T15:29:51+00:00</updated>
<author>
<name>Chuck Lever</name>
<email>cel@FreeBSD.org</email>
</author>
<published>2006-04-04T15:29:51+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=9bb78410987eec85c54aac634e857740387eb054'/>
<id>9bb78410987eec85c54aac634e857740387eb054</id>
<content type='text'>
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
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
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
</pre>
</div>
</content>
</entry>
</feed>
