Re: [PATCH] CIFS: Fix SMB2 readdir error handling

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



2014-10-03 20:58 GMT+04:00 Greg KH <greg@xxxxxxxxx>:
> On Tue, Sep 16, 2014 at 09:27:44AM +0400, Pavel Shilovsky wrote:
>> This is a backported version for v3.14.
>>
>> Commit 52755808d4525f4d5b86d112d36ffc7a46f3fb48 upstream.
>>
>> SMB2 servers indicates the end of a directory search with
>> STATUS_NO_MORE_FILE error code that is not processed now.
>> This causes generic/257 xfstest to fail. Fix this by triggering
>> the end of search by this error code in SMB2_query_directory.
>>
>> Also when negotiating CIFS protocol we tell the server to close
>> the search automatically at the end and there is no need to do
>> it itself. In the case of SMB2 protocol, we need to close it
>> explicitly - separate close directory checks for different
>> protocols.
>>
>> Cc: <stable@xxxxxxxxxxxxxxx>
>> Signed-off-by: Pavel Shilovsky <pshilovsky@xxxxxxxxx>
>> Signed-off-by: Steve French <smfrench@xxxxxxxxx>
>> ---
>>  fs/cifs/cifsglob.h     |    2 ++
>>  fs/cifs/file.c         |    2 +-
>>  fs/cifs/readdir.c      |    2 +-
>>  fs/cifs/smb1ops.c      |    7 +++++++
>>  fs/cifs/smb2maperror.c |    2 +-
>>  fs/cifs/smb2ops.c      |    9 +++++++++
>>  fs/cifs/smb2pdu.c      |    9 ++++-----
>>  7 files changed, 25 insertions(+), 8 deletions(-)
>
> This doesn't apply at all!
>
> What went wrong?
>
> greg k-h

Seems strange. Reposted the rebased version to stable@.

-- 
Best regards,
Pavel Shilovsky.
--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]