rconn: Tolerate negative argument in rconn_set_max_backoff().
authorBen Pfaff <blp@nicira.com>
Thu, 5 Mar 2009 20:26:07 +0000 (12:26 -0800)
committerBen Pfaff <blp@nicira.com>
Thu, 5 Mar 2009 20:26:07 +0000 (12:26 -0800)
This argument is likely to come directly from the user, so it helps callers
if they don't have to validate carefully.

lib/rconn.c

index 562f96f3162a2aad085ef31371872bfe67d8313a..9b4ae551da4eb0f8cd941ac3d52dd31369c3eea6 100644 (file)
@@ -221,7 +221,7 @@ rconn_create(int probe_interval, int max_backoff)
 void
 rconn_set_max_backoff(struct rconn *rc, int max_backoff)
 {
-    rc->max_backoff = max_backoff ? max_backoff : 60;
+    rc->max_backoff = MAX(1, max_backoff);
     if (rc->state == S_BACKOFF && rc->backoff > max_backoff) {
         rc->backoff = max_backoff;
         if (rc->backoff_deadline > time_now() + max_backoff) {