diff mbox

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

Message ID 1239904129-13132-1-git-send-email-jlayton@redhat.com (mailing list archive)
State New, archived
Headers show

Commit Message

Jeff Layton April 16, 2009, 5:48 p.m. UTC
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@linux.vnet.ibm.com>
Signed-off-by: Jeff Layton <jlayton@redhat.com>
---
 fs/cifs/connect.c |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

Comments

Steve French April 16, 2009, 7:27 p.m. UTC | #1
merged

On Thu, Apr 16, 2009 at 12:48 PM, Jeff Layton <jlayton@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@linux.vnet.ibm.com>
> Signed-off-by: Jeff Layton <jlayton@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
>
>
diff mbox

Patch

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 {