From: Jeffrey Altman <jaltman@secure-endpoints.com>
Date: Tue, 21 Aug 2007 09:30:27 +0000 (+0000)
Subject: DEVEL15-viced-remove-asserts-20070821
X-Git-Tag: openafs-devel-1_5_23~16
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=6af7a56a2fc45cf1a5eb22e390fd4beb6d925def;p=packages%2Fo%2Fopenafs.git

DEVEL15-viced-remove-asserts-20070821


remove asserts testing the number of alternate interfaces.


(cherry picked from commit d0f62da48417989040986a0bd6539d9696616229)
---

diff --git a/src/viced/afsfileprocs.c b/src/viced/afsfileprocs.c
index bdbd142c6..f879b5dc6 100644
--- a/src/viced/afsfileprocs.c
+++ b/src/viced/afsfileprocs.c
@@ -7458,11 +7458,9 @@ SRXAFS_CallBackRxConnAddr (struct rx_call * acall, afs_int32 *addr)
     if ( !thost->interface ) 
 	goto Bad_CallBackRxConnAddr;
     
-    assert(thost->interface->numberOfInterfaces > 0 );
-    
     /* the only address is the primary interface */
     /* can't change when there's only 1 address, anyway */
-    if ( thost->interface->numberOfInterfaces == 1 ) 
+    if ( thost->interface->numberOfInterfaces <= 1 ) 
 	goto Bad_CallBackRxConnAddr;
     
     /* initialise a security object only once */
diff --git a/src/viced/callback.c b/src/viced/callback.c
index 152bb8224..0a8075a80 100644
--- a/src/viced/callback.c
+++ b/src/viced/callback.c
@@ -2991,11 +2991,9 @@ MultiProbeAlternateAddress_r(struct host *host)
     if (!host->interface)
 	return 1;		/* failure */
 
-    assert(host->interface->numberOfInterfaces > 0);
-
     /* the only address is the primary interface */
-    if (host->interface->numberOfInterfaces == 1)
-	return 1;		/* failure */
+    if (host->interface->numberOfInterfaces <= 1)
+        return 1;               /* failure */
 
     /* initialise a security object only once */
     if (!sc)