[CIFS] Add write perm for usr to file on windows should remove r/o dos attr

Remove read only dos attribute on chmod when adding any write permission (ie on any of
user/group/other (not all of user/group/other ie  0222) when
mounted to windows.

Suggested by: Urs Fleisch

Signed-off-by: Urs Fleisch <urs.fleisch@gmail.com>
Signed-off-by: Steve French <sfrench@us.ibm.com>

1 files changed, 11 insertions, 11 deletions

diff --git a/fs/cifs/inode.c b/fs/cifs/inode.c
index f414526e476a..eeea33752e68 100644
--- a/fs/cifs/inode.c
+++ b/fs/cifs/inode.c
@@ -1339,17 +1339,17 @@ int cifs_setattr(struct dentry *direntry, struct iattr *attrs)
                                         cpu_to_le32(cifsInode->cifsAttrs |
                                                     ATTR_READONLY);
                         }
-                } else if ((mode & S_IWUGO) == S_IWUGO) {
-                        if (cifsInode->cifsAttrs & ATTR_READONLY) {
-                                set_dosattr = TRUE;
-                                time_buf.Attributes =
-                                        cpu_to_le32(cifsInode->cifsAttrs &
-                                                    (~ATTR_READONLY));
-                                /* Windows ignores set to zero */
-                                if(time_buf.Attributes == 0)
-                                        time_buf.Attributes |= 
-                                                cpu_to_le32(ATTR_NORMAL);
-                        }
+                } else if (cifsInode->cifsAttrs & ATTR_READONLY) {
+                        /* If file is readonly on server, we would
+                        not be able to write to it - so if any write
+                        bit is enabled for user or group or other we
+                        need to at least try to remove r/o dos attr */
+                        set_dosattr = TRUE;
+                        time_buf.Attributes = cpu_to_le32(cifsInode->cifsAttrs &
+                                            (~ATTR_READONLY));
+                        /* Windows ignores set to zero */
+                        if(time_buf.Attributes == 0)
+                                time_buf.Attributes |= cpu_to_le32(ATTR_NORMAL);
                 }
                 /* BB to be implemented -
                    via Windows security descriptors or streams */