From: Andrew Deason <adeason@sinenomine.net>
Date: Wed, 30 Jun 2010 17:53:13 +0000 (-0500)
Subject: vlclient: Remove incorrect whitespace fix
X-Git-Tag: openafs-devel-1_5_75~36
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=e10e29bc5bf53cb6fa30fa8580d8b25d00b5aed7;p=packages%2Fo%2Fopenafs.git

vlclient: Remove incorrect whitespace fix

Commit e59a0ff9934f8ea44ecb02e9fdaa81a29bd04b61 generated a warning
since we try to assign NULL into a character. Just remove this code,
since removing trailing whitespace was already handled by commit
df18c9e895e75e77ed466dea3b06ef2a0a07260a.

Change-Id: I915c0678a2f6893e09cf049bf71c519be554369f
Reviewed-on: http://gerrit.openafs.org/2303
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: Derrick Brashear <shadow@dementia.org>
---

diff --git a/src/vlserver/vlclient.c b/src/vlserver/vlclient.c
index e15f94bde..7186d68a7 100644
--- a/src/vlserver/vlclient.c
+++ b/src/vlserver/vlclient.c
@@ -243,13 +243,6 @@ handleit(struct cmd_syndesc *as, void *arock)
 	    char *oper, *vname;
 	    register char **argp = args;
 
-	    /* Eliminate terminating CR */
-	    while(strlen(line) > 0 &&
-		 (line[strlen(line)-1] == '\n' ||
-		  line[strlen(line)-1] == '\r')) {
-	      line[strlen(line)-1] = NULL;
-	    }
-
 	    GetArgs(line, argp, &nargs);
 	    oper = &argp[0][0];
 	    ++argp, --nargs;