fsprobe: Get rid of unused hostname buffer

When we copy the user supplied hostname into the fixed length
hnamebuf array, we might overflow it. As we never use this buffer,
just get rid of it.

Caught by coverity (#985913)

Change-Id: I4cda6279cd667343856deeb425dceaf00f7904b6
Reviewed-on: http://gerrit.openafs.org/9394
Reviewed-by: Marc Dionne <marc.c.dionne@gmail.com>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Derrick Brashear <shadow@your-file-system.com>

diff --git a/src/viced/fsprobe.c b/src/viced/fsprobe.c
index fa2e80d24e25435a89fb78489417cda4c2316eba..b2360230190a19fc4f85f6d90e45bf43b6d89dac 100644 (file)
--- a/src/viced/fsprobe.c
+++ b/src/viced/fsprobe.c
@@ -59,7 +59,6 @@ main(int argc, char **argv)
     struct sockaddr_in host;
     afs_int32 code;
     struct hostent *hp;
-    char hnamebuf[200];
     struct timeval tv;
     int noAuth = 1;            /* Default is authenticated connections */
 
@@ -74,9 +73,7 @@ main(int argc, char **argv)
 #ifdef STRUCT_SOCKADDR_HAS_SA_LEN
     host.sin_len = sizeof(struct sockaddr_in);
 #endif
-    if (host.sin_addr.s_addr != -1) {
-       strcpy(hnamebuf, av[0]);
-    } else {
+    if (host.sin_addr.s_addr == -1) {
        hp = gethostbyname(av[0]);
        if (hp) {
            host.sin_family = hp->h_addrtype;