Render the IP address for the "Ubik: Lost contact with sync-site" log
authorRod Widdowson <rdw@steadingsoftware.com>
Wed, 24 Mar 2010 16:59:15 +0000 (16:59 +0000)
committerDerrick Brashear <shadow@dementia.org>
Mon, 29 Mar 2010 04:34:54 +0000 (21:34 -0700)
message in the same way that all other IP addresses are (via
afs_inet_ntoa, rather than stripping the buytes out in a manner which
assumes a specific endianism).

Done more as a way to test my understanding of the GIT/GERRIT
technology with a small non-important checkin.

Change-Id: I177e1288e8e23087aeebb7abe4ed63a7c2b88ccb
Reviewed-on: http://gerrit.openafs.org/1649
Tested-by: Rod Widdowson <rdw@steadingsoftware.com>
Reviewed-by: Rod Widdowson <rdw@steadingsoftware.com>
Reviewed-by: Simon Wilkinson <sxw@inf.ed.ac.uk>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: Derrick Brashear <shadow@dementia.org>

src/ubik/vote.c

index 4635c63..c1ec6f5 100644 (file)
@@ -276,9 +276,8 @@ SVOTE_Beacon(register struct rx_call * rxcall, afs_int32 astate,
     } else if (syncTime + BIGTIME < now) {
        if (syncHost) {
            ubik_dprint
-               ("Ubik: Lost contact with sync-site %d.%d.%d.%d (NOT in quorum)\n",
-                ((syncHost >> 24) & 0xff), ((syncHost >> 16) & 0xff),
-                ((syncHost >> 8) & 0xff), (syncHost & 0xff));
+               ("Ubik: Lost contact with sync-site %s (NOT in quorum)\n",
+                afs_inet_ntoa(syncHost));
        }
        syncHost = 0;
     }