server: FILE_WRITE_ATTRIBUTES should succeed for readonly files.

dlls/ntdll/tests/file.c

@@ -4303,7 +4303,7 @@ static void test_readonly(void)
 
     status = pNtOpenFile(&handle, FILE_WRITE_ATTRIBUTES,  &attr, &io,
                          FILE_SHARE_READ|FILE_SHARE_WRITE|FILE_SHARE_DELETE, FILE_OPEN_FOR_BACKUP_INTENT);
-    todo_wine ok(status == STATUS_SUCCESS, "got %#x\n", status);
+    ok(status == STATUS_SUCCESS, "got %#x\n", status);
     CloseHandle(handle);
 
     status = pNtOpenFile(&handle, DELETE,  &attr, &io,

server/fd.c

@@ -2125,7 +2125,7 @@ struct fd *open_fd( struct fd *root, const char *name, struct unicode_str nt_nam
         else if (errno == EACCES)
         {
             /* try to change permissions temporarily to open a file descriptor */
-            if (!(access & (FILE_UNIX_WRITE_ACCESS | FILE_UNIX_READ_ACCESS | DELETE)) &&
+            if (!(access & ((FILE_UNIX_WRITE_ACCESS | FILE_UNIX_READ_ACCESS | DELETE) & ~FILE_WRITE_ATTRIBUTES)) &&
                 !stat( name, &st ) && st.st_uid == getuid() &&
                 !chmod( name, st.st_mode | S_IRUSR ))
             {