From: Greg KH on
2.6.33-stable review patch. If anyone has any objections, please let us know.


From: Steve French <sfrench(a)>

commit a24e2d7d8f512340991ef0a59cb5d08d491b8e98 upstream.

By doing this we always overwrite nbytes value that is being passed on to
CIFSSMBWrite() and need not rely on the callers to initialize. CIFSSMBWrite2 is
doing this already.

Reviewed-by: Shirish Pargaonkar <shirishpargaonkar(a)>
Reviewed-by: Jeff Layton <jlayton(a)>
Signed-off-by: Suresh Jayaraman <sjayaraman(a)>
Signed-off-by: Steve French <sfrench(a)>
Signed-off-by: Greg Kroah-Hartman <gregkh(a)>

fs/cifs/cifssmb.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)

--- a/fs/cifs/cifssmb.c
+++ b/fs/cifs/cifssmb.c
@@ -1430,6 +1430,8 @@ CIFSSMBWrite(const int xid, struct cifsT
__u32 bytes_sent;
__u16 byte_count;

+ *nbytes = 0;
/* cFYI(1, ("write at %lld %d bytes", offset, count));*/
if (tcon->ses == NULL)
@@ -1512,7 +1514,6 @@ CIFSSMBWrite(const int xid, struct cifsT
if (rc) {
cFYI(1, ("Send error in write = %d", rc));
- *nbytes = 0;
} else {
*nbytes = le16_to_cpu(pSMBr->CountHigh);
*nbytes = (*nbytes) << 16;

To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)
More majordomo info at
Please read the FAQ at