lkml.org 
[lkml]   [2010]   [Apr]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    Subject[043/139] CIFS: initialize nbytes at the beginning of CIFSSMBWrite()
    2.6.33-stable review patch.  If anyone has any objections, please let us know.

    ------------------

    From: Steve French <sfrench@us.ibm.com>

    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@gmail.com>
    Reviewed-by: Jeff Layton <jlayton@samba.org>
    Signed-off-by: Suresh Jayaraman <sjayaraman@suse.de>
    Signed-off-by: Steve French <sfrench@us.ibm.com>
    Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

    ---
    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)
    return -ECONNABORTED;
    @@ -1512,7 +1514,6 @@ CIFSSMBWrite(const int xid, struct cifsT
    cifs_stats_inc(&tcon->num_writes);
    if (rc) {
    cFYI(1, ("Send error in write = %d", rc));
    - *nbytes = 0;
    } else {
    *nbytes = le16_to_cpu(pSMBr->CountHigh);
    *nbytes = (*nbytes) << 16;



    \
     
     \ /
      Last update: 2010-04-22 22:25    [W:4.029 / U:0.060 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site