From ff388611f461b4a4ee044ce86aedda9ce18dfc97 Mon Sep 17 00:00:00 2001 From: Andrew Tridgell Date: Tue, 4 Aug 2009 16:01:11 +1000 Subject: fixed support for readx greater than 64k This fixes bug 6547, where smbclient in S3 reads more than 64k at a time with readx. --- source4/ntvfs/posix/pvfs_read.c | 4 ++-- source4/smb_server/smb/reply.c | 3 ++- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/source4/ntvfs/posix/pvfs_read.c b/source4/ntvfs/posix/pvfs_read.c index d9080d632d..75fba3c90d 100644 --- a/source4/ntvfs/posix/pvfs_read.c +++ b/source4/ntvfs/posix/pvfs_read.c @@ -59,8 +59,8 @@ NTSTATUS pvfs_read(struct ntvfs_module_context *ntvfs, } maxcnt = rd->readx.in.maxcnt; - if (maxcnt > UINT16_MAX && req->ctx->protocol < PROTOCOL_SMB2) { - maxcnt = 0; + if (maxcnt > 2*UINT16_MAX && req->ctx->protocol < PROTOCOL_SMB2) { + return NT_STATUS_INVALID_PARAMETER; } status = pvfs_check_lock(pvfs, f, req->smbpid, diff --git a/source4/smb_server/smb/reply.c b/source4/smb_server/smb/reply.c index ec7b6783fd..0433d3582f 100644 --- a/source4/smb_server/smb/reply.c +++ b/source4/smb_server/smb/reply.c @@ -831,7 +831,8 @@ static void reply_read_and_X_send(struct ntvfs_request *ntvfs) SMBSRV_VWV_RESERVED(4, 1); SSVAL(req->out.vwv, VWV(5), io->readx.out.nread); SSVAL(req->out.vwv, VWV(6), PTR_DIFF(io->readx.out.data, req->out.hdr)); - SMBSRV_VWV_RESERVED(7, 5); + SSVAL(req->out.vwv, VWV(7), (io->readx.out.nread>>16)); + SMBSRV_VWV_RESERVED(8, 4); smbsrv_chain_reply(req); } -- cgit