<feed xmlns='http://www.w3.org/2005/Atom'>
<title>linux.git/net/9p, branch v2.6.23</title>
<subtitle>Linux kernel source tree</subtitle>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/'/>
<entry>
<title>9p: fix bad error path in conversion routines</title>
<updated>2007-08-23T15:25:05+00:00</updated>
<author>
<name>Mariusz Kozlowski</name>
<email>m.kozlowski@tuxland.pl</email>
</author>
<published>2007-08-23T15:24:28+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/commit/?id=02881d94780faa86e32952e46381f7cd4c78d5ac'/>
<id>02881d94780faa86e32952e46381f7cd4c78d5ac</id>
<content type='text'>
When buf_check_overflow() returns != 0 we will hit kfree(ERR_PTR(err))
and it will not be happy about it.

Signed-off-by: Mariusz Kozlowski &lt;m.kozlowski@tuxland.pl&gt;
Signed-off-by: Eric Van Hensbergen &lt;ericvh@gmail.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
When buf_check_overflow() returns != 0 we will hit kfree(ERR_PTR(err))
and it will not be happy about it.

Signed-off-by: Mariusz Kozlowski &lt;m.kozlowski@tuxland.pl&gt;
Signed-off-by: Eric Van Hensbergen &lt;ericvh@gmail.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>9p: fix use after free</title>
<updated>2007-08-23T15:12:48+00:00</updated>
<author>
<name>Eric Van Hensbergen</name>
<email>ericvh@ericvh-desktop.austin.ibm.com</email>
</author>
<published>2007-07-26T19:04:54+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/commit/?id=1a3cac6c6d1f56dc26939eb41be29844f897c15a'/>
<id>1a3cac6c6d1f56dc26939eb41be29844f897c15a</id>
<content type='text'>
On 7/22/07, Adrian Bunk &lt;bunk@stusta.de&gt; wrote:
     The Coverity checker spotted the following use-after-free
     in net/9p/mux.c:

     &lt;--  snip  --&gt;

     ...
     struct p9_conn *p9_conn_create(struct p9_transport *trans, int msize,
                                         unsigned char *extended)
     {
     ...
             if (!m-&gt;tagpool) {
                     kfree(m);
                     return ERR_PTR(PTR_ERR(m-&gt;tagpool));
             }
     ...

     &lt;--  snip  --&gt;

Also spotted was a leak of the same structure further down in the function.

Signed-off-by: Eric Van Hensbergen &lt;ericvh@gmail.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
On 7/22/07, Adrian Bunk &lt;bunk@stusta.de&gt; wrote:
     The Coverity checker spotted the following use-after-free
     in net/9p/mux.c:

     &lt;--  snip  --&gt;

     ...
     struct p9_conn *p9_conn_create(struct p9_transport *trans, int msize,
                                         unsigned char *extended)
     {
     ...
             if (!m-&gt;tagpool) {
                     kfree(m);
                     return ERR_PTR(PTR_ERR(m-&gt;tagpool));
             }
     ...

     &lt;--  snip  --&gt;

Also spotted was a leak of the same structure further down in the function.

Signed-off-by: Eric Van Hensbergen &lt;ericvh@gmail.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>fs: 9p/conv.c error path fix</title>
<updated>2007-07-31T22:39:42+00:00</updated>
<author>
<name>Mariusz Kozlowski</name>
<email>m.kozlowski@tuxland.pl</email>
</author>
<published>2007-07-31T07:39:13+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/commit/?id=bbe06f6bf7e764a9eb0e3cdcec2b12c743f35757'/>
<id>bbe06f6bf7e764a9eb0e3cdcec2b12c743f35757</id>
<content type='text'>
When buf_check_overflow() returns != 0 we will hit kfree(ERR_PTR(err)) and
it will not be happy about it.

Signed-off-by: Mariusz Kozlowski &lt;m.kozlowski@tuxland.pl&gt;
Cc: Latchesar Ionkov &lt;lucho@ionkov.net&gt;
Cc: Eric Van Hensbergen &lt;ericvh@ericvh.myip.org&gt;
Cc: Andrew Morton &lt;akpm@linux-foundation.org&gt;
Signed-off-by: Andrew Morton &lt;akpm@linux-foundation.org&gt;
Signed-off-by: Linus Torvalds &lt;torvalds@linux-foundation.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
When buf_check_overflow() returns != 0 we will hit kfree(ERR_PTR(err)) and
it will not be happy about it.

Signed-off-by: Mariusz Kozlowski &lt;m.kozlowski@tuxland.pl&gt;
Cc: Latchesar Ionkov &lt;lucho@ionkov.net&gt;
Cc: Eric Van Hensbergen &lt;ericvh@ericvh.myip.org&gt;
Cc: Andrew Morton &lt;akpm@linux-foundation.org&gt;
Signed-off-by: Andrew Morton &lt;akpm@linux-foundation.org&gt;
Signed-off-by: Linus Torvalds &lt;torvalds@linux-foundation.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>9p: Don't use binary sysctl numbers.</title>
<updated>2007-07-22T19:43:28+00:00</updated>
<author>
<name>Eric W. Biederman</name>
<email>ebiederm@xmission.com</email>
</author>
<published>2007-07-21T18:53:19+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/commit/?id=b053c204edbc82b068fe8cfc809b7f90b885fae0'/>
<id>b053c204edbc82b068fe8cfc809b7f90b885fae0</id>
<content type='text'>
The recent 9p commit: bd238fb431f31989898423c8b6496bc8c4204a86 that
supposedly only moved files also introduced a new 9p sysctl interface
that did not properly register it's sysctl binary numbers.

And since it was only for debugging clearly did not need a binary fast
path in any case.  So this patch just remove the binary numbers.

See Documentation/sysctl/ctl_unnumbered.txt for more details.

While I was at it I cleaned up the sysctl initializers a little as
well so there is less to read.

Cc: Latchesar Ionkov &lt;lucho@ionkov.net&gt;
Cc: Eric Van Hensbergen &lt;ericvh@gmail.com&gt;
Signed-off-by: Eric W. Biederman &lt;ebiederm@xmission.com&gt;
Signed-off-by: Linus Torvalds &lt;torvalds@linux-foundation.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The recent 9p commit: bd238fb431f31989898423c8b6496bc8c4204a86 that
supposedly only moved files also introduced a new 9p sysctl interface
that did not properly register it's sysctl binary numbers.

And since it was only for debugging clearly did not need a binary fast
path in any case.  So this patch just remove the binary numbers.

See Documentation/sysctl/ctl_unnumbered.txt for more details.

While I was at it I cleaned up the sysctl initializers a little as
well so there is less to read.

Cc: Latchesar Ionkov &lt;lucho@ionkov.net&gt;
Cc: Eric Van Hensbergen &lt;ericvh@gmail.com&gt;
Signed-off-by: Eric W. Biederman &lt;ebiederm@xmission.com&gt;
Signed-off-by: Linus Torvalds &lt;torvalds@linux-foundation.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>9p: fix a race condition bug in umount which caused a segfault</title>
<updated>2007-07-14T20:14:19+00:00</updated>
<author>
<name>Eric Van Hensbergen</name>
<email>ericvh@ericvh-desktop.austin.ibm.com</email>
</author>
<published>2007-07-13T21:47:58+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/commit/?id=0af8887ebf4556a76680a61b0bb156d934702c63'/>
<id>0af8887ebf4556a76680a61b0bb156d934702c63</id>
<content type='text'>
umounting partitions after heavy activity would sometimes trigger a
segmentation violation.  This fix appears to remove that problem.
Fix originally provided by Latchesar Ionkov.

Signed-off-by: Eric Van Hensbergen &lt;ericvh@gmail.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
umounting partitions after heavy activity would sometimes trigger a
segmentation violation.  This fix appears to remove that problem.
Fix originally provided by Latchesar Ionkov.

Signed-off-by: Eric Van Hensbergen &lt;ericvh@gmail.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>net/9p: set error to EREMOTEIO if trans-&gt;write returns zero</title>
<updated>2007-07-14T20:14:01+00:00</updated>
<author>
<name>Latchesar Ionkov</name>
<email>lucho@ionkov.net</email>
</author>
<published>2007-07-11T21:14:46+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/commit/?id=1d6b5602381524c339af2c2fdfe42ad0a01464a4'/>
<id>1d6b5602381524c339af2c2fdfe42ad0a01464a4</id>
<content type='text'>
If trans-&gt;write returns 0, p9_write_work goes through the error path, but
sets the error code to zero.

This patch sets the error code to EREMOTEIO if trans-&gt;write returns zero
value.

Signed-off-by: Latchesar Ionkov &lt;lucho@ionkov.net&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
If trans-&gt;write returns 0, p9_write_work goes through the error path, but
sets the error code to zero.

This patch sets the error code to EREMOTEIO if trans-&gt;write returns zero
value.

Signed-off-by: Latchesar Ionkov &lt;lucho@ionkov.net&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>net/9p: change net/9p module name to 9pnet</title>
<updated>2007-07-14T20:13:50+00:00</updated>
<author>
<name>Latchesar Ionkov</name>
<email>lucho@ionkov.net</email>
</author>
<published>2007-07-11T21:13:54+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/commit/?id=e46662be7fddde3464bf208317542c2f8df13d0b'/>
<id>e46662be7fddde3464bf208317542c2f8df13d0b</id>
<content type='text'>
Change module name of net/9p module from 9p.ko to 9pnet.ko. fs/9p module
already uses 9p.ko name.

Signed-off-by: Latchesar Ionkov &lt;lucho@ionkov.net&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Change module name of net/9p module from 9p.ko to 9pnet.ko. fs/9p module
already uses 9p.ko name.

Signed-off-by: Latchesar Ionkov &lt;lucho@ionkov.net&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>9p: Reorganization of 9p file system code</title>
<updated>2007-07-14T20:13:40+00:00</updated>
<author>
<name>Latchesar Ionkov</name>
<email>lucho@ionkov.net</email>
</author>
<published>2007-07-10T22:57:28+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/commit/?id=bd238fb431f31989898423c8b6496bc8c4204a86'/>
<id>bd238fb431f31989898423c8b6496bc8c4204a86</id>
<content type='text'>
This patchset moves non-filesystem interfaces of v9fs from fs/9p to net/9p.
It moves the transport, packet marshalling and connection layers to net/9p
leaving only the VFS related files in fs/9p.  This work is being done in
preparation for in-kernel 9p servers as well as alternate 9p clients (other
than VFS).

Signed-off-by: Latchesar Ionkov &lt;lucho@ionkov.net&gt;
Signed-off-by: Eric Van Hensbergen &lt;ericvh@gmail.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This patchset moves non-filesystem interfaces of v9fs from fs/9p to net/9p.
It moves the transport, packet marshalling and connection layers to net/9p
leaving only the VFS related files in fs/9p.  This work is being done in
preparation for in-kernel 9p servers as well as alternate 9p clients (other
than VFS).

Signed-off-by: Latchesar Ionkov &lt;lucho@ionkov.net&gt;
Signed-off-by: Eric Van Hensbergen &lt;ericvh@gmail.com&gt;
</pre>
</div>
</content>
</entry>
</feed>
