At first glance this looks like a typo (probably not a serious one, but worth fixing)<br><br>-                       else if ((rc == -EINVAL) || (rc != -EOPNOTSUPP))<br><br>looks like it should be:<br><br>+                       else if ((rc == -EINVAL) || (rc == -EOPNOTSUPP))<br>
<br><div class="gmail_quote">On Sat, Jan 9, 2010 at 1:30 PM, Roel Kluin <span dir="ltr">&lt;<a href="mailto:roel.kluin@gmail.com">roel.kluin@gmail.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
rc != -EOPNOTSUPP is true as well if rc is -EINVAL.<br>
<br>
Signed-off-by: Roel Kluin &lt;<a href="mailto:roel.kluin@gmail.com">roel.kluin@gmail.com</a>&gt;<br>
---<br>
 fs/cifs/dir.c   |    2 +-<br>
 fs/cifs/inode.c |    2 +-<br>
 2 files changed, 2 insertions(+), 2 deletions(-)<br>
<br>
Or maybe there is ==/!= or &amp;&amp;/|| confusion?<br>
<br>
diff --git a/fs/cifs/dir.c b/fs/cifs/dir.c<br>
index 6ccf726..f5cfdd6 100644<br>
--- a/fs/cifs/dir.c<br>
+++ b/fs/cifs/dir.c<br>
@@ -690,7 +690,7 @@ cifs_lookup(struct inode *parent_dir_inode, struct dentry *direntry,<br>
                         */<br>
                        if ((rc == 0) || (rc == -ENOENT))<br>
                                posix_open = true;<br>
-                       else if ((rc == -EINVAL) || (rc != -EOPNOTSUPP))<br>
+                       else if (rc != -EOPNOTSUPP)<br>
                                pTcon-&gt;broken_posix_open = true;<br>
                }<br>
                if (!posix_open)<br>
diff --git a/fs/cifs/inode.c b/fs/cifs/inode.c<br>
index cf18ee7..aec2928 100644<br>
--- a/fs/cifs/inode.c<br>
+++ b/fs/cifs/inode.c<br>
@@ -1264,7 +1264,7 @@ cifs_do_rename(int xid, struct dentry *from_dentry, const char *fromPath,<br>
         * source Note that cross directory moves do not work with<br>
         * rename by filehandle to various Windows servers.<br>
         */<br>
-       if (rc == 0 || rc != -ETXTBSY)<br>
+       if (rc != -ETXTBSY)<br>
                return rc;<br>
<br>
        /* open the file to be renamed -- we need DELETE perms */<br>
</blockquote></div><br><br clear="all"><br>-- <br>Thanks,<br><br>Steve<br>