<feed xmlns='http://www.w3.org/2005/Atom'>
<title>kernel.git/fs/cifs/ioctl.c, branch linux-3.3.y</title>
<subtitle>Hosts the 0x221E linux distro kernel.
</subtitle>
<id>https://git.0xinfinity.dev/distro/kernel.git/atom?h=linux-3.3.y</id>
<link rel='self' href='https://git.0xinfinity.dev/distro/kernel.git/atom?h=linux-3.3.y'/>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/'/>
<updated>2011-05-27T04:34:02Z</updated>
<entry>
<title>[CIFS] Rename three structures to avoid camel case</title>
<updated>2011-05-27T04:34:02Z</updated>
<author>
<name>Steve French</name>
</author>
<published>2011-05-27T04:34:02Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=96daf2b09178d8ebde2b0d56b027de917c17dfdf'/>
<id>urn:sha1:96daf2b09178d8ebde2b0d56b027de917c17dfdf</id>
<content type='text'>
secMode to sec_mode
and
cifsTconInfo to cifs_tcon
and
cifsSesInfo to cifs_ses

Signed-off-by: Steve French &lt;sfrench@us.ibm.com&gt;
</content>
</entry>
<entry>
<title>cifs: make cifs_ioctl handle NULL filp-&gt;private_data correctly</title>
<updated>2010-11-08T18:56:36Z</updated>
<author>
<name>Jeff Layton</name>
</author>
<published>2010-11-08T12:28:32Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=618763958b2291a09057dbfa553da6ded93dcfad'/>
<id>urn:sha1:618763958b2291a09057dbfa553da6ded93dcfad</id>
<content type='text'>
Commit 13cfb7334e made cifs_ioctl use the tlink attached to the
cifsFileInfo for a filp. This ignores the case of an open directory
however, which in CIFS can have a NULL private_data until a readdir
is done on it.

This patch re-adds the NULL pointer checks that were removed in commit
50ae28f01 and moves the setting of tcon and "caps" variables lower.

Long term, a better fix would be to establish a f_op-&gt;open routine for
directories that populates that field at open time, but that requires
some other changes to how readdir calls are handled.

Reported-by: Kjell Rune Skaaraas &lt;kjella79@yahoo.no&gt;
Reviewed-and-Tested-by: Suresh Jayaraman &lt;sjayaraman@suse.de&gt;
Signed-off-by: Jeff Layton &lt;jlayton@redhat.com&gt;
Signed-off-by: Steve French &lt;sfrench@us.ibm.com&gt;
</content>
</entry>
<entry>
<title>FS: cifs, remove unneeded NULL tests</title>
<updated>2010-11-02T03:47:21Z</updated>
<author>
<name>Jiri Slaby</name>
</author>
<published>2010-11-01T15:08:55Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=50ae28f0144a790fc63a5b89b9aca3ffa9f88522'/>
<id>urn:sha1:50ae28f0144a790fc63a5b89b9aca3ffa9f88522</id>
<content type='text'>
Stanse found that pSMBFile in cifs_ioctl and file-&gt;f_path.dentry in
cifs_user_write are dereferenced prior their test to NULL.

The alternative is not to dereference them before the tests. The patch is
to point out the problem, you have to decide.

While at it we cache the inode in cifs_user_write to a local variable
and use all over the function.

Signed-off-by: Jiri Slaby &lt;jslaby@suse.cz&gt;
Cc: Steve French &lt;sfrench@samba.org&gt;
Cc: linux-cifs@vger.kernel.org
Cc: Jeff Layton &lt;jlayton@redhat.com&gt;
Cc: Christoph Hellwig &lt;hch@infradead.org&gt;
Signed-off-by: Steve French &lt;sfrench@us.ibm.com&gt;
</content>
</entry>
<entry>
<title>cifs: have cifsFileInfo hold a reference to a tlink rather than tcon pointer</title>
<updated>2010-10-06T16:12:49Z</updated>
<author>
<name>Jeff Layton</name>
</author>
<published>2010-09-29T23:51:11Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=13cfb7334eb6fd0fc06da5589aea1e947791f1d6'/>
<id>urn:sha1:13cfb7334eb6fd0fc06da5589aea1e947791f1d6</id>
<content type='text'>
cifsFileInfo needs a pointer to a tcon, but it doesn't currently hold a
reference to it. Change it to keep a pointer to a tcon_link instead and
hold a reference to it.

That will keep the tcon from being freed until the file is closed.

Signed-off-by: Jeff Layton &lt;jlayton@redhat.com&gt;
Signed-off-by: Steve French &lt;sfrench@us.ibm.com&gt;
</content>
</entry>
<entry>
<title>cifs: make various routines use the cifsFileInfo-&gt;tcon pointer</title>
<updated>2010-09-29T19:04:32Z</updated>
<author>
<name>Jeff Layton</name>
</author>
<published>2010-09-20T23:01:31Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=ba00ba64cf0895e4c2ac507e56306363dc125a90'/>
<id>urn:sha1:ba00ba64cf0895e4c2ac507e56306363dc125a90</id>
<content type='text'>
...where it's available and appropriate.

Signed-off-by: Jeff Layton &lt;jlayton@redhat.com&gt;
Signed-off-by: Steve French &lt;sfrench@us.ibm.com&gt;
</content>
</entry>
<entry>
<title>fs/cifs: Remove unnecessary casts of private_data</title>
<updated>2010-08-02T12:40:33Z</updated>
<author>
<name>Joe Perches</name>
</author>
<published>2010-07-12T20:50:14Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=c21dfb699f35b6b5508fb808bb0ca211a865f2c9'/>
<id>urn:sha1:c21dfb699f35b6b5508fb808bb0ca211a865f2c9</id>
<content type='text'>
Signed-off-by: Joe Perches &lt;joe@perches.com&gt;
Signed-off-by: Steve French &lt;sfrench@us.ibm.com&gt;
</content>
</entry>
<entry>
<title>[CIFS] Neaten cERROR and cFYI macros, reduce text space</title>
<updated>2010-04-21T03:50:45Z</updated>
<author>
<name>Joe Perches</name>
</author>
<published>2010-04-21T03:50:45Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=b6b38f704a8193daba520493ebdaf7e819962fc8'/>
<id>urn:sha1:b6b38f704a8193daba520493ebdaf7e819962fc8</id>
<content type='text'>
Neaten cERROR and cFYI macros, reduce text space
~2.5K

Convert '__FILE__ ": " fmt' to '"%s: " fmt', __FILE__' to save text space
Surround macros with do {} while
Add parentheses to macros
Make statement expression macro from macro with assign
Remove now unnecessary parentheses from cFYI and cERROR uses

defconfig with CIFS support old
$ size fs/cifs/built-in.o
   text	   data	    bss	    dec	    hex	filename
 156012	   1760	    148	 157920	  268e0	fs/cifs/built-in.o

defconfig with CIFS support old
$ size fs/cifs/built-in.o
   text	   data	    bss	    dec	    hex	filename
 153508	   1760	    148	 155416	  25f18	fs/cifs/built-in.o

allyesconfig old:
$ size fs/cifs/built-in.o
   text	   data	    bss	    dec	    hex	filename
 309138	   3864	  74824	 387826	  5eaf2	fs/cifs/built-in.o

allyesconfig new
$ size fs/cifs/built-in.o
   text	   data	    bss	    dec	    hex	filename
 305655	   3864	  74824	 384343	  5dd57	fs/cifs/built-in.o

Signed-off-by: Joe Perches &lt;joe@perches.com&gt;
Signed-off-by: Steve French &lt;sfrench@us.ibm.com&gt;
</content>
</entry>
<entry>
<title>CRED: Wrap task credential accesses in the CIFS filesystem</title>
<updated>2008-11-13T23:38:47Z</updated>
<author>
<name>David Howells</name>
</author>
<published>2008-11-13T23:38:47Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=a001e5b558f25eb1e588522d73ac949b643b7a37'/>
<id>urn:sha1:a001e5b558f25eb1e588522d73ac949b643b7a37</id>
<content type='text'>
Wrap access to task credentials so that they can be separated more easily from
the task_struct during the introduction of COW creds.

Change most current-&gt;(|e|s|fs)[ug]id to current_(|e|s|fs)[ug]id().

Change some task-&gt;e?[ug]id to task_e?[ug]id().  In some places it makes more
sense to use RCU directly rather than a convenient wrapper; these will be
addressed by later patches.

Signed-off-by: David Howells &lt;dhowells@redhat.com&gt;
Reviewed-by: James Morris &lt;jmorris@namei.org&gt;
Acked-by: Serge Hallyn &lt;serue@us.ibm.com&gt;
Cc: Steve French &lt;sfrench@samba.org&gt;
Cc: linux-cifs-client@lists.samba.org
Signed-off-by: James Morris &lt;jmorris@namei.org&gt;
</content>
</entry>
<entry>
<title>[CIFS] BKL-removal: convert CIFS over to unlocked_ioctl</title>
<updated>2008-05-15T05:51:55Z</updated>
<author>
<name>Steve French</name>
</author>
<published>2008-05-15T05:51:55Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=f9ddcca4cf7d95238beb295484d1de7c0bf490dd'/>
<id>urn:sha1:f9ddcca4cf7d95238beb295484d1de7c0bf490dd</id>
<content type='text'>
cifs_ioctl doesn't seem to need the BKL for anything, so convert it over
to use unlocked_ioctl.

Signed-off-by: Andi Kleen &lt;andi@firstfloor.org&gt;
Signed-off-by: Andrew Morton &lt;akpm@linux-foundation.org&gt;
Signed-off-by: Steve French &lt;sfrench@us.ibm.com&gt;
</content>
</entry>
<entry>
<title>[CIFS] reduce checkpatch warnings</title>
<updated>2008-02-07T23:25:02Z</updated>
<author>
<name>Steve French</name>
</author>
<published>2008-02-07T23:25:02Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=ad7a2926b9e53cfb3020d15bdfacacc54e2b63da'/>
<id>urn:sha1:ad7a2926b9e53cfb3020d15bdfacacc54e2b63da</id>
<content type='text'>
Signed-off-by: Steve French &lt;sfrench@us.ibm.com&gt;
</content>
</entry>
</feed>
