[linux-cifs-client] Re: [PATCH] cifs: remove unneeded bcc_ptr update in CIFSTCon

Steve French smfrench at gmail.com
Thu Apr 16 19:27:27 GMT 2009


merged

On Thu, Apr 16, 2009 at 12:48 PM, Jeff Layton <jlayton at redhat.com> wrote:
> This pointer isn't used again after this point. It's also not updated in
> the ascii case, so there's no need to update it here.
>
> Pointed-out-by: Dave Kleikamp <shaggy at linux.vnet.ibm.com>
> Signed-off-by: Jeff Layton <jlayton at redhat.com>
> ---
>  fs/cifs/connect.c |    1 -
>  1 files changed, 0 insertions(+), 1 deletions(-)
>
> diff --git a/fs/cifs/connect.c b/fs/cifs/connect.c
> index 4a04ecd..bacdef1 100644
> --- a/fs/cifs/connect.c
> +++ b/fs/cifs/connect.c
> @@ -3765,7 +3765,6 @@ CIFSTCon(unsigned int xid, struct cifsSesInfo *ses,
>                                        cFYI(1, ("nativeFileSystem=%s",
>                                                tcon->nativeFileSystem));
>                                }
> -                               bcc_ptr += (2 * length) + 2;
>                        }
>                        /* else do not bother copying these information fields*/
>                } else {
> --
> 1.5.5.6
>
>



-- 
Thanks,

Steve


More information about the linux-cifs-client mailing list