1. 28 Aug, 2008 4 commits
    • Steve French's avatar
      [CIFS] Turn off Unicode during session establishment for plaintext authentication · c76da9da
      Steve French authored
      LANMAN session setup did not support Unicode (after session setup, unicode can
      still be used though).
      
      Fixes samba bug# 5319
      
      CC: Jeff Layton <jlayton@redhat.com>
      CC: Stable Kernel <stable@vger.kernel.org>
      Signed-off-by: default avatarSteve French <sfrench@us.ibm.com>
      c76da9da
    • Steve French's avatar
      [CIFS] update cifs change log · 2e655021
      Steve French authored
      Signed-off-by: default avatarSteve French <sfrench@us.ibm.com>
      2e655021
    • Jeff Layton's avatar
      cifs: fix O_APPEND on directio mounts · 838726c4
      Jeff Layton authored
      The direct I/O write codepath for CIFS is done through
      cifs_user_write(). That function does not currently call
      generic_write_checks() so the file position isn't being properly set
      when the file is opened with O_APPEND.  It's also not doing the other
      "normal" checks that should be done for a write call.
      
      The problem is currently that when you open a file with O_APPEND on a
      mount with the directio mount option, the file position is set to the
      beginning of the file. This makes any subsequent writes clobber the data
      in the file starting at the beginning.
      
      This seems to fix the problem in cursory testing. It is, however
      important to note that NFS disallows the combination of
      (O_DIRECT|O_APPEND). If my understanding is correct, the concern is
      races with multiple clients appending to a file clobbering each others'
      data. Since the write model for CIFS and NFS is pretty similar in this
      regard, CIFS is probably subject to the same sort of races. What's
      unclear to me is why this is a particular problem with O_DIRECT and not
      with buffered writes...
      
      Regardless, disallowing O_APPEND on an entire mount is probably not
      reasonable, so we'll probably just have to deal with it and reevaluate
      this flag combination when we get proper support for O_DIRECT. In the
      meantime this patch at least fixes the existing problem.
      Signed-off-by: default avatarJeff Layton <jlayton@redhat.com>
      Cc: Stable Tree <stable@kernel.org>
      Signed-off-by: default avatarSteve French <sfrench@us.ibm.com>
      838726c4
    • Steve French's avatar
  2. 27 Aug, 2008 26 commits
  3. 26 Aug, 2008 10 commits