Log binding ip address and port during startup
[openafs.git] / src / vlserver / vldb_check.c
index 166ae6e..5e454d2 100644 (file)
@@ -1,12 +1,28 @@
 /*
  * Copyright 2000, International Business Machines Corporation and others.
  * All Rights Reserved.
- * 
+ *
  * This software has been released under the terms of the IBM Public
  * License.  For details, see the LICENSE file in the top-level source
  * directory or online at http://www.openafs.org/dl/license10.html
  */
 
+#include <afsconfig.h>
+#include <afs/param.h>
+
+#include <roken.h>
+
+#ifdef AFS_NT40_ENV
+#include <WINNT/afsevent.h>
+#endif
+
+#include <ubik.h>
+#include <afs/afsutil.h>
+#include <afs/cmd.h>
+
+#include "vlserver.h"
+#include "vldbint.h"
+
 /* Read a VLDB file and verify it for correctness */
 
 #define VL  0x001              /* good volume entry */
 #define vldbread(x,y,z) vldbio(x,y,z,0)
 #define vldbwrite(x,y,z) vldbio(x,y,z,1)
 
-#include <afsconfig.h>
-#include <afs/param.h>
-
-
-#include <stdlib.h>
-#include <sys/types.h>
-#include <sys/stat.h>
-#include <fcntl.h>
-#include <errno.h>
-#include <string.h>
-#ifdef AFS_NT40_ENV
-#include <winsock2.h>
-#include <WINNT/afsevent.h>
-#include <io.h>
-#else
-#include <sys/socket.h>
-#include <netdb.h>
-#include <netinet/in.h>
-#endif
-
-#include "vlserver.h"
-#include "vldbint.h"
-#include <ubik.h>
-#include <afs/afsutil.h>
-#include <afs/cmd.h>
-
-#define ADDR(x) (x/sizeof(struct nvlentry))
+#define HDRSIZE 64
+#define ADDR(x) ((x)/sizeof(struct nvlentry))
+#define OFFSET(addr) ((addr) + HDRSIZE)
 
 int fd;
 int listentries, listservers, listheader, listuheader, verbose, quiet;
 
 int fix = 0;
-int fixed = 0;
 int passes = 0;
 /* if quiet, don't send anything to stdout */
-int quiet = 0; 
+int quiet = 0;
 /*  error level. 0 = no error, 1 = warning, 2 = error, 4 = fatal */
-int error_level  = 0; 
+int error_level  = 0;
 
 struct er {
     long addr;
@@ -88,41 +79,40 @@ struct er {
 } *record;
 afs_int32 maxentries;
 int serveraddrs[MAXSERVERID + 2];
+u_char serverxref[MAXSERVERID + 2];  /**< to resolve cross-linked mh entries */
+int serverref[MAXSERVERID + 2];      /**< which addrs are referenced by vl entries */
+
+struct mhinfo {
+    afs_uint32 addr;                   /**< vldb file record */
+    char orphan[VL_MHSRV_PERBLK];      /**< unreferenced mh enties */
+} mhinfo[VL_MAX_ADDREXTBLKS];
+
 
 /*  Used to control what goes to stdout based on quiet flag */
-void 
+void
 quiet_println(const char *fmt,...) {
-    va_list args;                                             
+    va_list args;
     if (!quiet) {
-        va_start(args, fmt);                                      
-        vfprintf(stdout, fmt, args);                              
-        va_end(args);                                             
+        va_start(args, fmt);
+        vfprintf(stdout, fmt, args);
+        va_end(args);
     }
 }
 
 /*  Used to set the error level and ship messages to stderr */
-void                                                   
-log_error(int eval, const char *fmt, ...)                          
-{                                                             
-    va_list args;                                             
+void
+log_error(int eval, const char *fmt, ...)
+{
+    va_list args;
     if (error_level < eval) error_level  = eval ;  /*  bump up the severity */
-    va_start(args, fmt);                                      
-    vfprintf(stderr, fmt, args);                              
-    va_end(args);                                             
+    va_start(args, fmt);
+    vfprintf(stderr, fmt, args);
+    va_end(args);
 
     if (error_level  == VLDB_CHECK_FATAL) exit(VLDB_CHECK_FATAL);
-}  
-
-
-#if 0
-int
-writeUbikHeader()
-{
-    /* Bump the version number?? We could cheat and push a new db... */
 }
-#endif
 
-#define HDRSIZE 64
+
 int
 readUbikHeader(void)
 {
@@ -144,7 +134,7 @@ readUbikHeader(void)
     }
 
     uheader.magic = ntohl(uheader.magic);
-    uheader.size = ntohl(uheader.size);
+    uheader.size = ntohs(uheader.size);
     uheader.version.epoch = ntohl(uheader.version.epoch);
     uheader.version.counter = ntohl(uheader.version.counter);
 
@@ -172,16 +162,16 @@ vldbio(int position, void *buffer, int size, int rdwr)
     int offset, r, p;
 
     /* seek to the correct spot. skip ubik stuff */
-    p = position + HDRSIZE;
+    p = OFFSET(position);
     offset = lseek(fd, p, 0);
     if (offset != p) {
        log_error(VLDB_CHECK_FATAL,"error: lseek to %d failed: %d %d\n", p, offset, errno);
        return (-1);
     }
 
-    if (rdwr == 1) 
+    if (rdwr == 1)
        r = write(fd, buffer, size);
-    else 
+    else
        r = read(fd, buffer, size);
 
     if (r != size) {
@@ -223,7 +213,7 @@ NameHash(char *volname)
 afs_int32
 IdHash(afs_uint32 volid)
 {
-    return ((abs(volid)) % HASHSIZE);
+    return (volid % HASHSIZE);
 }
 
 #define LEGALCHARS ".ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789-_"
@@ -237,6 +227,8 @@ InvalidVolname(char *volname)
     slen = strlen(volname);
     if (slen >= VL_MAXNAMELEN)
        return 1;
+    if (slen == 0)
+       return 1;
     return (slen != strspn(volname, map));
 }
 
@@ -278,7 +270,7 @@ readheader(struct vlheader *headerp)
            ntohl(headerp->vital_header.totalEntries[1]);
 
     headerp->SIT = ntohl(headerp->SIT);
-    for (i = 0; i < MAXSERVERID; i++)
+    for (i = 0; i <= MAXSERVERID; i++)
        headerp->IpMappedAddr[i] = ntohl(headerp->IpMappedAddr[i]);
     for (i = 0; i < HASHSIZE; i++)
        headerp->VolnameHash[i] = ntohl(headerp->VolnameHash[i]);
@@ -342,7 +334,7 @@ writeheader(struct vlheader *headerp)
            htonl(headerp->vital_header.totalEntries[1]);
 
     headerp->SIT = htonl(headerp->SIT);
-    for (i = 0; i < MAXSERVERID; i++)
+    for (i = 0; i <= MAXSERVERID; i++)
        headerp->IpMappedAddr[i] = htonl(headerp->IpMappedAddr[i]);
     for (i = 0; i < HASHSIZE; i++)
        headerp->VolnameHash[i] = htonl(headerp->VolnameHash[i]);
@@ -354,18 +346,27 @@ writeheader(struct vlheader *headerp)
 }
 
 void
-readMH(afs_int32 addr, struct extentaddr *mhblockP)
+readMH(afs_uint32 addr, int block, struct extentaddr *mhblockP)
 {
     int i, j;
     struct extentaddr *e;
 
     vldbread(addr, (char *)mhblockP, VL_ADDREXTBLK_SIZE);
 
-    mhblockP->ex_count = ntohl(mhblockP->ex_count);
-    mhblockP->ex_flags = ntohl(mhblockP->ex_flags);
-    for (i = 0; i < VL_MAX_ADDREXTBLKS; i++)
-       mhblockP->ex_contaddrs[i] = ntohl(mhblockP->ex_contaddrs[i]);
-
+    /* Every mh block has the VLCONTBLOCK flag set in the header to
+     * indicate the entry is an 8192 byte extended block. The
+     * VLCONTBLOCK flag is always clear in regular vl entries. The
+     * vlserver depends on the VLCONTBLOCK flag to correctly traverse
+     * the vldb. The flags field is in network byte order. */
+    mhblockP->ex_hdrflags = ntohl(mhblockP->ex_hdrflags);
+
+    if (block == 0) {
+        /* These header fields are only used in the first mh block. */
+        mhblockP->ex_count = ntohl(mhblockP->ex_count);
+        for (i = 0; i < VL_MAX_ADDREXTBLKS; i++) {
+           mhblockP->ex_contaddrs[i] = ntohl(mhblockP->ex_contaddrs[i]);
+        }
+    }
     for (i = 1; i < VL_MHSRV_PERBLK; i++) {
        e = &(mhblockP[i]);
 
@@ -409,7 +410,7 @@ readentry(afs_int32 addr, struct nvlentry *vlentryp, afs_int32 *type)
     }
 
     if (listentries) {
-       quiet_println("address %u: ", addr);
+       quiet_println("address %u (offset 0x%0x): ", addr, OFFSET(addr));
        if (vlentryp->flags == VLCONTBLOCK) {
            quiet_println("mh extension block\n");
        } else if (vlentryp->flags == VLFREE) {
@@ -426,7 +427,19 @@ readentry(afs_int32 addr, struct nvlentry *vlentryp, afs_int32 *type)
                quiet_println(" ro");
            if (vlentryp->flags & VLF_BACKEXISTS)
                quiet_println(" bk");
-           if (vlentryp->flags & 0xffff8fff)
+           if (vlentryp->flags & VLOP_MOVE)
+               quiet_println(" lock_move");
+           if (vlentryp->flags & VLOP_RELEASE)
+               quiet_println(" lock_release");
+           if (vlentryp->flags & VLOP_BACKUP)
+               quiet_println(" lock_backup");
+           if (vlentryp->flags & VLOP_DELETE)
+               quiet_println(" lock_delete");
+           if (vlentryp->flags & VLOP_DUMP)
+               quiet_println(" lock_dump");
+
+           /* all bits not covered by VLF_* and VLOP_* constants */
+           if (vlentryp->flags & 0xffff8e0f)
                quiet_println(" errorflag(0x%x)", vlentryp->flags);
            quiet_println("\n");
            quiet_println("   LockAfsId     = %d\n", vlentryp->LockAfsId);
@@ -458,12 +471,42 @@ readentry(afs_int32 addr, struct nvlentry *vlentryp, afs_int32 *type)
 }
 
 void
+writeMH(afs_int32 addr, int block, struct extentaddr *mhblockP)
+{
+    int i, j;
+    struct extentaddr *e;
+
+    if (verbose) {
+       quiet_println("Writing back MH block % at addr %u\n", block,  addr);
+    }
+    mhblockP->ex_hdrflags = htonl(mhblockP->ex_hdrflags);
+    if (block == 0) {
+       /*
+        * These header fields are only used in the first mh block, so were
+        * converted to host byte order only when the first mh block was read.
+        */
+       mhblockP->ex_count = htonl(mhblockP->ex_count);
+       for (i = 0; i < VL_MAX_ADDREXTBLKS; i++) {
+           mhblockP->ex_contaddrs[i] = htonl(mhblockP->ex_contaddrs[i]);
+       }
+    }
+    for (i = 1; i < VL_MHSRV_PERBLK; i++) {
+       e = &(mhblockP[i]);
+       /* hostuuid was not converted */
+       e->ex_uniquifier = htonl(e->ex_uniquifier);
+       for (j = 0; j < VL_MAXIPADDRS_PERMH; j++) {
+           e->ex_addrs[j] = htonl(e->ex_addrs[j]);
+       }
+    }
+    vldbwrite(addr, (char *)mhblockP, VL_ADDREXTBLK_SIZE);
+}
+
+void
 writeentry(afs_int32 addr, struct nvlentry *vlentryp)
 {
     int i;
 
     if (verbose) quiet_println("Writing back entry at addr %u\n", addr);
-    fixed++;
     for (i = 0; i < MAXTYPES; i++)
        vlentryp->volumeId[i] = htonl(vlentryp->volumeId[i]);
     vlentryp->flags = htonl(vlentryp->flags);
@@ -482,61 +525,6 @@ writeentry(afs_int32 addr, struct nvlentry *vlentryp)
     vldbwrite(addr, (char *)vlentryp, sizeof(*vlentryp));
 }
 
-void
-readSIT(int base, int addr)
-{
-    int i, j, a;
-    char sitbuf[VL_ADDREXTBLK_SIZE];
-    struct extentaddr *extent;
-
-    if (!addr)
-       return;
-    vldbread(addr, sitbuf, VL_ADDREXTBLK_SIZE);
-    extent = (struct extentaddr *)sitbuf;
-
-    quiet_println("multihome info block: base %d\n", base);
-    if (base == 0) {
-       quiet_println("   count = %u\n", ntohl(extent->ex_count));
-       quiet_println("   flags = %u\n", ntohl(extent->ex_flags));
-       for (i = 0; i < VL_MAX_ADDREXTBLKS; i++) {
-           quiet_println("   contaddrs[%d] = %u\n", i,
-                  ntohl(extent->ex_contaddrs[i]));
-       }
-    }
-    for (i = 1; i < VL_MHSRV_PERBLK; i++) {
-       /* should we skip this entry */
-       for (j = 0; j < VL_MAX_ADDREXTBLKS; j++) {
-           if (extent[i].ex_addrs[j])
-               break;
-       }
-       if (j >= VL_MAX_ADDREXTBLKS)
-           continue;
-
-       quiet_println("   base %d index %d:\n", base, i);
-
-       quiet_println("       afsuuid    = (%x %x %x /%d/%d/ /%x/%x/%x/%x/%x/%x/)\n",
-              ntohl(extent[i].ex_hostuuid.time_low),
-              ntohl(extent[i].ex_hostuuid.time_mid),
-              ntohl(extent[i].ex_hostuuid.time_hi_and_version),
-              ntohl(extent[i].ex_hostuuid.clock_seq_hi_and_reserved),
-              ntohl(extent[i].ex_hostuuid.clock_seq_low),
-              ntohl(extent[i].ex_hostuuid.node[0]),
-              ntohl(extent[i].ex_hostuuid.node[1]),
-              ntohl(extent[i].ex_hostuuid.node[2]),
-              ntohl(extent[i].ex_hostuuid.node[3]),
-              ntohl(extent[i].ex_hostuuid.node[4]),
-              ntohl(extent[i].ex_hostuuid.node[5]));
-       quiet_println("       uniquifier = %u\n", ntohl(extent[i].ex_uniquifier));
-       for (j = 0; j < VL_MAXIPADDRS_PERMH; j++) {
-           a = ntohl(extent[i].ex_addrs[j]);
-           if (a) {
-               quiet_println("       %d.%d.%d.%d\n", (a >> 24) & 0xff,
-                      (a >> 16) & 0xff, (a >> 8) & 0xff, (a) & 0xff);
-           }
-       }
-    }
-}
-
 /*
  * Read each entry in the database:
  * Record what type of entry it is and its address in the record array.
@@ -560,9 +548,6 @@ ReadAllEntries(struct vlheader *header)
        /* Remember the highest volume id */
        readentry(addr, &vlentry, &type);
        if (type == VL) {
-           if (!(vlentry.flags & VLF_RWEXISTS))
-               log_error(VLDB_CHECK_WARNING,"VLDB_CHECK_WARNING: VLDB entry '%s' has no RW volume\n",
-                      vlentry.name);
 
            for (i = 0; i < MAXTYPES; i++)
                if (maxvolid < vlentry.volumeId[i])
@@ -587,9 +572,9 @@ ReadAllEntries(struct vlheader *header)
                    continue;
                }
                if (e) {
-                  log_error 
-                       (VLDB_CHECK_ERROR,"VLDB entry '%s' contains an unknown RW/RO index serverFlag\n",
-                        vlentry.name);
+                  log_error
+                       (VLDB_CHECK_ERROR,"address %u (offset 0x%0x): VLDB entry '%s' contains an unknown RW/RO index serverFlag\n",
+                       addr, OFFSET(addr), vlentry.name);
                    e = 0;
                }
                quiet_println
@@ -619,7 +604,7 @@ ReadAllEntries(struct vlheader *header)
            entrysize = VL_ADDREXTBLK_SIZE;
            mhcount++;
        } else {
-           log_error(VLDB_CHECK_ERROR, "Unknown entry at %u. Aborting\n", addr);
+           log_error(VLDB_CHECK_ERROR, "address %u (offset 0x%0x): Unknown entry. Aborting\n", addr, OFFSET(addr));
            break;
        }
     }
@@ -637,57 +622,6 @@ ReadAllEntries(struct vlheader *header)
             header->vital_header.MaxVolumeId, maxvolid);
 }
 
-
-void
-SetHashEnd(long addr, int type, long new)
-{
-    struct nvlentry vlentry;
-    afs_int32 type2, next = -1;
-
-    for (; addr; addr = next) {
-       readentry(addr, &vlentry, &type2);
-       switch(type & 0xf0) {
-       case RWH:
-           next = vlentry.nextIdHash[0];
-           break;
-       case ROH:
-           next = vlentry.nextIdHash[1];
-           break;
-       case BKH:
-           next = vlentry.nextIdHash[2];
-           break;
-       case NH:
-           next = vlentry.nextNameHash;
-           break;
-       default:
-           next = -1;
-       }
-
-       if (next < 1) {
-           switch(type & 0xf0) {
-           case RWH:
-             if (vlentry.nextIdHash[0] != 0) {quiet_println("bwoop\n");}
-               vlentry.nextIdHash[0] = new;
-               break;
-           case ROH:
-             if (vlentry.nextIdHash[1] != 0) {quiet_println("bwoop\n");}
-               vlentry.nextIdHash[1] = new;
-               break;
-           case BKH:
-             if (vlentry.nextIdHash[2] != 0) {quiet_println("bwoop\n");}
-               vlentry.nextIdHash[2] = new;
-               break;
-           case NH:
-             if (vlentry.nextNameHash != 0) {quiet_println("bwoop\n");}
-               vlentry.nextNameHash = new;
-               break;
-           }
-           writeentry(addr, &vlentry);
-           return;
-       }
-    }
-}
-
 /*
  * Follow each Name hash bucket marking it as read in the record array.
  * Record we found it in the name hash within the record array.
@@ -707,7 +641,7 @@ FollowNameHash(struct vlheader *header)
        chainlength = 0;
 
        if (!validVolumeAddr(header->VolnameHash[i])) {
-           log_error(VLDB_CHECK_ERROR,"Name Hash %d: Bad entry %u is out of range\n",
+           log_error(VLDB_CHECK_ERROR,"Name Hash index %d is out of range: %u\n",
                      i, header->VolnameHash[i]);
            continue;
        }
@@ -715,8 +649,8 @@ FollowNameHash(struct vlheader *header)
        for (addr = header->VolnameHash[i]; addr; addr = vlentry.nextNameHash) {
            readentry(addr, &vlentry, &type);
            if (type != VL) {
-               log_error(VLDB_CHECK_ERROR,"Name Hash %d: Bad entry at %u: Not a valid vlentry\n",
-                      i, addr);
+               log_error(VLDB_CHECK_ERROR,"address %u (offset 0x%0x): Name Hash %d: Not a vlentry\n",
+                      addr, OFFSET(addr), i);
                continue;
            }
 
@@ -727,21 +661,21 @@ FollowNameHash(struct vlheader *header)
             * checked it either above or below
             */
            if (record[rindex].addr != addr && record[rindex].addr) {
-               log_error       
+               log_error
                    (VLDB_CHECK_ERROR,"INTERNAL VLDB_CHECK_ERROR: addresses %ld and %u use same record slot %d\n",
                     record[rindex].addr, addr, rindex);
            }
            if (record[rindex].type & NH) {
-               log_error       
-                   (VLDB_CHECK_ERROR,"Name Hash %d: Bad entry '%s': Already in the name hash\n",
-                    i, vlentry.name);
+               log_error
+                   (VLDB_CHECK_ERROR,"address %u (offset 0x%0x): Name Hash %d: volume name '%s' is already in the name hash\n",
+                    addr, OFFSET(addr), i, vlentry.name);
                record[rindex].type |= MULTN;
                break;
            }
 
            if (!validVolumeAddr(vlentry.nextNameHash)) {
-               log_error(VLDB_CHECK_ERROR,"Name Hash forward link of '%s' is out of range\n",
-                         vlentry.name);
+               log_error(VLDB_CHECK_ERROR,"address %u (offset 0x%0x): Name Hash forward link of '%s' is out of range\n",
+                         addr, OFFSET(addr), vlentry.name);
                record[rindex].type |= MULTN;
                break;
            }
@@ -754,9 +688,9 @@ FollowNameHash(struct vlheader *header)
 
            /* Hash the name and check if in correct hash table */
            if (NameHash(vlentry.name) != i) {
-               log_error       
-                   (VLDB_CHECK_ERROR,"Name Hash %d: Bad entry '%s': Incorrect name hash chain (should be in %d)\n",
-                    i, vlentry.name, NameHash(vlentry.name));
+               log_error
+                   (VLDB_CHECK_ERROR,"address %u (offset 0x%0x): Name Hash %d: volume name '%s': Incorrect name hash chain (should be in %d)\n",
+                    addr, OFFSET(addr), i, vlentry.name, NameHash(vlentry.name));
                record[rindex].type |= MULTN;
            }
        }
@@ -799,7 +733,7 @@ FollowIdHash(struct vlheader *header)
        for (j = 0; j < HASHSIZE; j++) {
            chainlength = 0;
            if (!validVolumeAddr(header->VolidHash[i][j])) {
-               log_error(VLDB_CHECK_ERROR,"%s Hash %d: Bad entry %u is out of range\n",
+               log_error(VLDB_CHECK_ERROR,"%s Hash index %d is out of range: %u\n",
                          vtype(i), j, header->VolidHash[i][j]);
                continue;
            }
@@ -809,8 +743,8 @@ FollowIdHash(struct vlheader *header)
                readentry(addr, &vlentry, &type);
                if (type != VL) {
                    log_error
-                       (VLDB_CHECK_ERROR,"%s Id Hash %d: Bad entry at %u: Not a valid vlentry\n",
-                        vtype(i), j, addr);
+                       (VLDB_CHECK_ERROR,"address %u (offset 0x%0x): %s Id Hash %d: Not a vlentry\n",
+                        addr, OFFSET(addr), vtype(i), j);
                    continue;
                }
 
@@ -822,15 +756,15 @@ FollowIdHash(struct vlheader *header)
                }
                if (record[rindex].type & hash) {
                    log_error
-                       (VLDB_CHECK_ERROR,"%s Id Hash %d: Bad entry '%s': Already in the hash table\n",
-                        vtype(i), j, vlentry.name);
+                       (VLDB_CHECK_ERROR,"address %u (offset 0x%0x): %s Id Hash %d: volume name '%s': Already in the hash table\n",
+                        addr, OFFSET(addr), vtype(i), j, vlentry.name);
                    record[rindex].type |= badref;
                    break;
                }
 
                if (!validVolumeAddr(vlentry.nextIdHash[i])) {
-                   log_error(VLDB_CHECK_ERROR,"%s Id Hash forward link of '%s' is out of range\n",
-                             vtype(i), vlentry.name);
+                   log_error(VLDB_CHECK_ERROR,"address %u (offset 0x%0x): %s Id Hash forward link of '%s' is out of range\n",
+                             addr, OFFSET(addr), vtype(i), vlentry.name);
                    record[rindex].type |= badref;
                    break;
                }
@@ -843,9 +777,9 @@ FollowIdHash(struct vlheader *header)
 
                /* Hash the id and check if in correct hash table */
                if (IdHash(vlentry.volumeId[i]) != j) {
-                  log_error 
-                       (VLDB_CHECK_ERROR,"%s Id Hash %d: Bad entry '%s': Incorrect Id hash chain (should be in %d)\n",
-                        vtype(i), j, vlentry.name,
+                  log_error
+                       (VLDB_CHECK_ERROR,"address %u (offset 0x%0x): %s Id Hash %d: volume name '%s': Incorrect Id hash chain (should be in %d)\n",
+                        addr, OFFSET(addr), vtype(i), j, vlentry.name,
                         IdHash(vlentry.volumeId[i]));
                    record[rindex].type |= badhash;
                }
@@ -883,21 +817,21 @@ FollowFreeChain(struct vlheader *header)
         addr = vlentry.nextIdHash[0]) {
        readentry(addr, &vlentry, &type);
        if (type != FR) {
-          log_error 
-               (VLDB_CHECK_ERROR,"Free Chain %d: Bad entry at %u: Not a valid free vlentry (0x%x)\n",
-                count, addr, type);
+          log_error
+               (VLDB_CHECK_ERROR,"address %u (offset 0%0x): Free Chain %d: Not a free vlentry (0x%x)\n",
+                addr, OFFSET(addr), count, type);
            continue;
        }
 
        rindex = addr / sizeof(vlentry);
        if (record[rindex].addr != addr && record[rindex].addr) {
-          log_error 
-               (VLDB_CHECK_ERROR,"INTERNAL VLDB_CHECK_ERROR: addresses %u and %ld use same record slot %d\n",
-                record[rindex].addr, addr, rindex);
+          log_error
+               (VLDB_CHECK_ERROR,"INTERNAL VLDB_CHECK_ERROR: addresses %u (0x%0x) and %ld (0x%0x) use same record slot %d\n",
+                record[rindex].addr, OFFSET(record[rindex].addr), addr, OFFSET(addr), rindex);
        }
        if (record[rindex].type & FRC) {
-           log_error(VLDB_CHECK_ERROR,"Free Chain: Bad entry at %u: Already in the free chain\n",
-                  addr);
+           log_error(VLDB_CHECK_ERROR,"address %u (offset 0x%0x): Free Chain: Already in the free chain\n",
+                  addr, OFFSET(addr));
            break;
        }
        record[rindex].type |= FRC;
@@ -913,16 +847,16 @@ FollowFreeChain(struct vlheader *header)
  * Read each multihomed block and mark it as found in the record.
  * Read each entry in each multihomed block and mark the serveraddrs
  * array with the number of ip addresses found for this entry.
- * 
+ *
  * Then read the IpMappedAddr array in the header.
  * Verify that multihomed entries base and index are valid and points to
  * a good multhomed entry.
  * Mark the serveraddrs array with 1 ip address for regular entries.
- * 
- * By the end, the severaddrs array will have a 0 if the entry has no 
+ *
+ * By the end, the severaddrs array will have a 0 if the entry has no
  * IP addresses in it or the count of the number of IP addresses.
  *
- * The code does not verify if there are duplicate IP addresses in the 
+ * The code does not verify if there are duplicate IP addresses in the
  * list. The vlserver does this when a fileserver registeres itself.
  */
 void
@@ -931,7 +865,6 @@ CheckIpAddrs(struct vlheader *header)
     int mhblocks = 0;
     afs_int32 i, j, m, rindex;
     afs_int32 mhentries, regentries;
-    afs_int32 caddrs[VL_MAX_ADDREXTBLKS];
     char mhblock[VL_ADDREXTBLK_SIZE];
     struct extentaddr *MHblock = (struct extentaddr *)mhblock;
     struct extentaddr *e;
@@ -944,82 +877,83 @@ CheckIpAddrs(struct vlheader *header)
        quiet_println("Check Multihomed blocks\n");
 
     if (header->SIT) {
-       /* Read the first MH block and from it, gather the 
+       /* Read the first MH block and from it, gather the
         * addresses of all the mh blocks.
         */
-       readMH(header->SIT, MHblock);
-       if (MHblock->ex_flags != VLCONTBLOCK) {
-          log_error 
-               (VLDB_CHECK_ERROR,"Multihomed Block 0: Bad entry at %u: Not a valid multihomed block\n",
-                header->SIT);
+       readMH(header->SIT, 0, MHblock);
+       if (MHblock->ex_hdrflags != VLCONTBLOCK) {
+          log_error
+               (VLDB_CHECK_ERROR,"address %u (offset 0x%0x): Multihomed Block 0: Not a multihomed block\n",
+                header->SIT, OFFSET(header->SIT));
        }
 
        for (i = 0; i < VL_MAX_ADDREXTBLKS; i++) {
-           caddrs[i] = MHblock->ex_contaddrs[i];
+           mhinfo[i].addr = MHblock->ex_contaddrs[i];
        }
 
-       if (header->SIT != caddrs[0]) {
-          log_error 
-               (VLDB_CHECK_ERROR,"MH block does not point to self %u in header, %u in block\n",
-                header->SIT, caddrs[0]);
+       if (header->SIT != mhinfo[0].addr) {
+          log_error
+               (VLDB_CHECK_ERROR,"address %u (offset 0x%0x): MH block does not point to self in header, %u in block\n",
+                header->SIT, OFFSET(header->SIT), mhinfo[0].addr);
        }
 
        /* Now read each MH block and record it in the record array */
        for (i = 0; i < VL_MAX_ADDREXTBLKS; i++) {
-           if (!caddrs[i])
+           if (!mhinfo[i].addr)
                continue;
 
-           readMH(caddrs[i], MHblock);
-           if (MHblock->ex_flags != VLCONTBLOCK) {
-               log_error       
-                   (VLDB_CHECK_ERROR,"Multihomed Block 0: Bad entry at %u: Not a valid multihomed block\n",
-                    header->SIT);
+           readMH(mhinfo[i].addr, i, MHblock);
+           if (MHblock->ex_hdrflags != VLCONTBLOCK) {
+               log_error
+                   (VLDB_CHECK_ERROR,"address %u (offset 0x%0x): Multihomed Block %d: Not a multihomed block\n",
+                    mhinfo[i].addr, OFFSET(mhinfo[i].addr), i);
            }
 
-           rindex = caddrs[i] / sizeof(vlentry);
-           if (record[rindex].addr != caddrs[i] && record[rindex].addr) {
-               log_error       
-                   (VLDB_CHECK_ERROR,"INTERNAL VLDB_CHECK_ERROR: addresses %u and %u use same record slot %d\n",
-                    record[rindex].addr, caddrs[i], rindex);
+           rindex = mhinfo[i].addr / sizeof(vlentry);
+           if (record[rindex].addr != mhinfo[i].addr && record[rindex].addr) {
+               log_error
+                   (VLDB_CHECK_ERROR,"INTERNAL VLDB_CHECK_ERROR: addresses %u (0x%0x) and %u (0x%0x) use same record slot %d\n",
+                    record[rindex].addr, OFFSET(record[rindex].addr), mhinfo[i].addr, OFFSET(mhinfo[i].addr), rindex);
            }
            if (record[rindex].type & FRC) {
-               log_error       
-                   (VLDB_CHECK_ERROR,"MH Blocks Chain %d: Bad entry at %ld: Already a MH block\n",
-                    i, record[rindex].addr);
+               log_error
+                   (VLDB_CHECK_ERROR,"address %u (offset 0x%0x): MH Blocks Chain %d: Already a MH block\n",
+                    record[rindex].addr, OFFSET(record[rindex].addr), i);
                break;
            }
            record[rindex].type |= MHC;
 
            mhblocks++;
 
-           /* Read each entry in a multihomed block. 
+           /* Read each entry in a multihomed block.
             * Find the pointer to the entry in the IpMappedAddr array and
             * verify that the entry is good (has IP addresses in it).
             */
            mhentries = 0;
            for (j = 1; j < VL_MHSRV_PERBLK; j++) {
+               int first_ipindex = -1;
                e = (struct extentaddr *)&(MHblock[j]);
 
-               /* Search the IpMappedAddr array for the reference to this entry */
-               for (ipindex = 0; ipindex < MAXSERVERID; ipindex++) {
-                   if (((header->IpMappedAddr[ipindex] & 0xff000000) ==
-                        0xff000000)
-                       &&
-                       (((header->
-                          IpMappedAddr[ipindex] & 0x00ff0000) >> 16) == i)
-                       && ((header->IpMappedAddr[ipindex] & 0x0000ffff) ==
-                           j)) {
-                       break;
+               /* Search the IpMappedAddr array for all the references to this entry. */
+               /* Use the first reference for checking the ip addresses of this entry. */
+               for (ipindex = 0; ipindex <= MAXSERVERID; ipindex++) {
+                   if (((header->IpMappedAddr[ipindex] & 0xff000000) == 0xff000000)
+                       && (((header-> IpMappedAddr[ipindex] & 0x00ff0000) >> 16) == i)
+                       && ((header->IpMappedAddr[ipindex] & 0x0000ffff) == j)) {
+                       if (first_ipindex == -1) {
+                           first_ipindex = ipindex;
+                       } else {
+                           serverxref[ipindex] = first_ipindex;
+                       }
                    }
                }
-               if (ipindex >= MAXSERVERID)
-                   ipindex = -1;
-               else
+               ipindex = first_ipindex;
+               if (ipindex != -1)
                    serveraddrs[ipindex] = -1;
 
                if (memcmp(&e->ex_hostuuid, &nulluuid, sizeof(afsUUID)) == 0) {
                    if (ipindex != -1) {
-                       log_error       
+                       log_error
                            (VLDB_CHECK_ERROR,"Server Addrs index %d references null MH block %d, index %d\n",
                             ipindex, i, j);
                        serveraddrs[ipindex] = 0;       /* avoids printing 2nd error below */
@@ -1038,7 +972,8 @@ CheckIpAddrs(struct vlheader *header)
                if (ipaddrs) {
                    mhentries++;
                    if (ipindex == -1) {
-                       log_error       
+                       mhinfo[i].orphan[j] = 1;
+                       log_error
                            (VLDB_CHECK_ERROR,"MH block %d, index %d: Not referenced by server addrs\n",
                             i, j);
                    } else {
@@ -1081,20 +1016,34 @@ CheckIpAddrs(struct vlheader *header)
                mhentries++;
                if (((header->IpMappedAddr[i] & 0x00ff0000) >> 16) >
                    VL_MAX_ADDREXTBLKS)
-                  log_error 
+                  log_error
                        (VLDB_CHECK_ERROR,"IP Addr for entry %d: Multihome block is bad (%d)\n",
                         i, ((header->IpMappedAddr[i] & 0x00ff0000) >> 16));
+               if (mhinfo[(header->IpMappedAddr[i] & 0x00ff0000) >> 16].addr == 0)
+                   log_error(VLDB_CHECK_ERROR,"IP Addr for entry %d: No such multihome block (%d)\n",
+                        i, ((header->IpMappedAddr[i] & 0x00ff0000) >> 16));
                if (((header->IpMappedAddr[i] & 0x0000ffff) > VL_MHSRV_PERBLK)
                    || ((header->IpMappedAddr[i] & 0x0000ffff) < 1))
-                   log_error 
+                   log_error
                        (VLDB_CHECK_ERROR,"IP Addr for entry %d: Multihome index is bad (%d)\n",
                         i, (header->IpMappedAddr[i] & 0x0000ffff));
                if (serveraddrs[i] == -1) {
-                   log_error 
+                   log_error
                        (VLDB_CHECK_WARNING,"warning: IP Addr for entry %d: Multihome entry has no ip addresses\n",
                         i);
                    serveraddrs[i] = 0;
                }
+               if (serverxref[i] != BADSERVERID) {
+                   log_error
+                       (VLDB_CHECK_WARNING,
+                       "warning: MH block %d, index %d is cross-linked by server numbers %d and %d.\n",
+                       (header->IpMappedAddr[i] & 0x00ff0000) >> 16,
+                       (header->IpMappedAddr[i] & 0x0000ffff),
+                       i, serverxref[i]);
+                   /* set addresses found/not found for this server number,
+                    * using the first index to the mh we found above. */
+                   serveraddrs[i] = serveraddrs[serverxref[i]];
+               }
                if (listservers) {
                    quiet_println("   Server ip addr %d = MH block %d, index %d\n",
                           i, (header->IpMappedAddr[i] & 0x00ff0000) >> 16,
@@ -1120,12 +1069,113 @@ CheckIpAddrs(struct vlheader *header)
     return;
 }
 
-void 
-FixBad(afs_uint32 idx, afs_uint32 addr, afs_uint32 type, afs_uint32 tmp, 
-       struct nvlentry *vlentry, afs_uint32 hash) {
-    SetHashEnd(addr, type, tmp);
-    quiet_println("linked unlinked chain %u (index %lu) to end of chain %d for %s hash\n", 
-          tmp, ADDR(tmp), hash, type==NH?"Name":(type==RWH?"RW":(type==ROH?"RO":"BK")));
+char *
+nameForAddr(afs_uint32 addr, int hashtype, afs_uint32 *hash, char *buffer)
+{
+    /*
+     * We need to simplify the reporting, while retaining
+     * legible messages.  This is a helper function.  The return address
+     * is either a fixed char or the provided buffer - so don't use the
+     * name after the valid lifetime of the buffer.
+     */
+    afs_int32 type;
+    struct nvlentry entry;
+    if (!addr) {
+       /* Distinguished, invalid, hash */
+       *hash = 0xFFFFFFFF;
+       return "empty";
+    } else if (!validVolumeAddr(addr)) {
+       /* Different, invalid, hash */
+       *hash = 0XFFFFFFFE;
+       return "invalid";
+    }
+    readentry(addr, &entry, &type);
+    if (VL != type) {
+       *hash = 0XFFFFFFFE;
+       return "invalid";
+    }
+    if (hashtype >= MAXTYPES) {
+       *hash = NameHash(entry.name);
+    } else {
+       *hash = IdHash(entry.volumeId[hashtype]);
+    }
+    sprintf(buffer, "for '%s'", entry.name);
+    return buffer;
+}
+
+void
+reportHashChanges(struct vlheader *header, afs_uint32 oldnamehash[HASHSIZE], afs_uint32 oldidhash[MAXTYPES][HASHSIZE])
+{
+    int i, j;
+    afs_uint32 oldhash, newhash;
+    char oldNameBuffer[10 + VL_MAXNAMELEN];
+    char newNameBuffer[10 + VL_MAXNAMELEN];
+    char *oldname, *newname;
+    /*
+     * report hash changes
+     */
+
+    for (i = 0; i < HASHSIZE; i++) {
+       if (oldnamehash[i] != header->VolnameHash[i]) {
+
+           oldname = nameForAddr(oldnamehash[i], MAXTYPES, &oldhash, oldNameBuffer);
+           newname = nameForAddr(header->VolnameHash[i], MAXTYPES, &newhash, newNameBuffer);
+           if (verbose || (oldhash != newhash)) {
+               quiet_println("FIX: Name hash header at %d was %s, is now %s\n", i, oldname, newname);
+           }
+       }
+       for (j = 0; j < MAXTYPES; j++) {
+           if (oldidhash[j][i] != header->VolidHash[j][i]) {
+
+               oldname = nameForAddr(oldidhash[j][i], j, &oldhash, oldNameBuffer);
+               newname = nameForAddr(header->VolidHash[j][i], j, &newhash, newNameBuffer);
+               if (verbose || (oldhash != newhash)) {
+                   quiet_println("FIX: %s hash header at %d was %s, is now %s\n", vtype(j), i, oldname, newname);
+               }
+           }
+       }
+    }
+}
+
+/**
+ * Remove unreferenced, duplicate multi-home address indices.
+ *
+ * Removes entries from IpMappedAddr which where found to be
+ * duplicates. Only entries which are not referenced by vl entries
+ * are removed on this pass.
+ *
+ * @param[inout] header the vldb header to be updated.
+ */
+void
+removeCrossLinkedAddresses(struct vlheader *header)
+{
+    int i;
+
+    for (i = 0; i <= MAXSERVERID; i++) {
+       if (serverref[i] == 0
+           && (header->IpMappedAddr[i] & 0xff000000) == 0xff000000
+           && serverxref[i] != BADSERVERID) {
+           if (serverxref[i] == i) {
+               log_error(VLDB_CHECK_ERROR,
+                         "INTERNAL VLDB_CHECK_ERROR: serverxref points to self; index %d\n",
+                         i);
+           } else if (header->IpMappedAddr[serverxref[i]] == 0) {
+               log_error(VLDB_CHECK_ERROR,
+                         "INTERNAL VLDB_CHECK_ERROR: serverxref points to empty addr; index %d, value %d\n",
+                         i, serverxref[i]);
+           } else if (header->IpMappedAddr[serverxref[i]] != header->IpMappedAddr[i]) {
+               log_error(VLDB_CHECK_ERROR,
+                         "INTERNAL VLDB_CHECK_ERROR: invalid serverxref; index %d, value %d\n",
+                         i, serverxref[i]);
+           } else {
+               quiet_println
+                   ("FIX: Removing unreferenced address index %d, which cross-links MH block %d, index %d\n",
+                    i, (header->IpMappedAddr[i] & 0x00ff0000) >> 16,
+                    (header->IpMappedAddr[i] & 0x0000ffff));
+               header->IpMappedAddr[i] = 0;
+           }
+       }
+    }
 }
 
 int
@@ -1135,7 +1185,9 @@ WorkerBee(struct cmd_syndesc *as, void *arock)
     afs_int32 type;
     struct vlheader header;
     struct nvlentry vlentry, vlentry2;
-    int i, j;
+    int i, j, k;
+    afs_uint32 oldnamehash[HASHSIZE];
+    afs_uint32 oldidhash[MAXTYPES][HASHSIZE];
 
     error_level = 0;  /*  start clean with no error status */
     dbfile = as->parms[0].items->data; /* -database */
@@ -1154,7 +1206,7 @@ WorkerBee(struct cmd_syndesc *as, void *arock)
         return VLDB_CHECK_FATAL;
     }
 
- restart:
+
     /* open the vldb database file */
     fd = open(dbfile, (fix > 0)?O_RDWR:O_RDONLY, 0);
     if (fd < 0) {
@@ -1171,9 +1223,13 @@ WorkerBee(struct cmd_syndesc *as, void *arock)
     }
 
     maxentries = (header.vital_header.eofPtr / sizeof(vlentry)) + 1;
-    record = (struct er *)malloc(maxentries * sizeof(struct er));
-    memset(record, 0, (maxentries * sizeof(struct er)));
+    record = calloc(maxentries, sizeof(struct er));
     memset(serveraddrs, 0, sizeof(serveraddrs));
+    memset(mhinfo, 0, sizeof(mhinfo));
+    memset(serverref, 0, sizeof(serverref));
+    for (i = 0; i <= MAXSERVERID; i++) {
+       serverxref[i] = BADSERVERID;
+    }
 
     /* Will fill in the record array of entries it found */
     ReadAllEntries(&header);
@@ -1192,17 +1248,14 @@ WorkerBee(struct cmd_syndesc *as, void *arock)
     FollowFreeChain(&header);
 
     /* Now check the record we have been keeping for inconsistencies
-     * For valid vlentries, also check that the server we point to is 
+     * For valid vlentries, also check that the server we point to is
      * valid (the serveraddrs array).
      */
     if (verbose)
        quiet_println("Verify each volume entry\n");
     for (i = 0; i < maxentries; i++) {
-       int nextp = 0;
-       int reft = 0;
        int hash = 0;
         int nexthash = 0;
-       int *nextpp = NULL;
        char *which = NULL;
 
        if (record[i].type == 0)
@@ -1218,45 +1271,41 @@ WorkerBee(struct cmd_syndesc *as, void *arock)
 
            readentry(record[i].addr, &vlentry, &type);
 
+           if (!(vlentry.flags & VLF_RWEXISTS))
+               log_error(VLDB_CHECK_ERROR,"address %u (offset 0x%0x): Volume '%s' (%u) has no RW volume\n",
+                      record[i].addr, OFFSET(record[i].addr), vlentry.name, vlentry.volumeId[0]);
+
            if (InvalidVolname(vlentry.name))
-               log_error(VLDB_CHECK_ERROR,"Volume '%s' at addr %ld has an invalid name\n",
-                      vlentry.name, record[i].addr);
+               log_error(VLDB_CHECK_ERROR,"address %u (offset 0x%0x): Volume '%s' (%u) has an invalid name\n",
+                      record[i].addr, OFFSET(record[i].addr), vlentry.name, vlentry.volumeId[0]);
+
+           if (vlentry.volumeId[0] == 0)
+               log_error(VLDB_CHECK_ERROR,"address %u (offset 0x%0x): Volume '%s' (%u) has an invalid volume id\n",
+                      record[i].addr, OFFSET(record[i].addr), vlentry.name, vlentry.volumeId[0]);
 
            if (!(record[i].type & NH)) {
-               nextp = ADDR(vlentry.nextNameHash);
-               reft = REFN;
                hash = NameHash(vlentry.name);
-               nextpp = &vlentry.nextNameHash;
                which = "name";
                volidbuf[0]='\0';
                foundbad = 1;
            }
 
            if (vlentry.volumeId[0] && !(record[i].type & RWH)) {
-               nextp = ADDR(vlentry.nextIdHash[0]);
-               reft = REFRW;
                hash = IdHash(vlentry.volumeId[0]);
-               nextpp = &(vlentry.nextIdHash[0]);
                which = "RW";
                sprintf(volidbuf, "id %u ", vlentry.volumeId[0]);
                foundbad = 1;
            }
 
            if (vlentry.volumeId[1] && !(record[i].type & ROH)) {
-               nextp = ADDR(vlentry.nextIdHash[1]);
-               reft = REFRO;
                hash = IdHash(vlentry.volumeId[1]);
-               nextpp = &(vlentry.nextIdHash[1]);
                which = "RO";
                sprintf(volidbuf, "id %u ", vlentry.volumeId[1]);
                foundbad = 1;
            }
 
            if (vlentry.volumeId[2] && !(record[i].type & BKH)) {
-               nextp = ADDR(vlentry.nextIdHash[2]);
-               reft = REFBK;
                hash = IdHash(vlentry.volumeId[2]);
-               nextpp = &(vlentry.nextIdHash[2]);
                which = "BK";
                sprintf(volidbuf, "id %u ", vlentry.volumeId[2]);
                foundbad = 1;
@@ -1264,10 +1313,7 @@ WorkerBee(struct cmd_syndesc *as, void *arock)
 
            if (!validVolumeAddr(vlentry.nextNameHash) ||
                record[ADDR(vlentry.nextNameHash)].type & MULTN) {
-               nextp = ADDR(vlentry.nextNameHash);
-               reft = REFN;
                hash = NameHash(vlentry.name);
-               nextpp = &vlentry.nextNameHash;
                which = "name";
                volidbuf[0]='\0';
                if (validVolumeAddr(vlentry.nextNameHash)) {
@@ -1282,10 +1328,7 @@ WorkerBee(struct cmd_syndesc *as, void *arock)
 
            if (!validVolumeAddr(vlentry.nextIdHash[0]) ||
                record[ADDR(vlentry.nextIdHash[0])].type & MULTRW) {
-               nextp = ADDR(vlentry.nextIdHash[0]);
-               reft = REFRW;
                hash = IdHash(vlentry.volumeId[0]);
-               nextpp = &(vlentry.nextIdHash[0]);
                which = "RW";
                sprintf(volidbuf, "id %u ", vlentry.volumeId[0]);
                if (validVolumeAddr(vlentry.nextIdHash[0])) {
@@ -1300,10 +1343,7 @@ WorkerBee(struct cmd_syndesc *as, void *arock)
 
            if (!validVolumeAddr(vlentry.nextIdHash[1]) ||
                record[ADDR(vlentry.nextIdHash[1])].type & MULTRO) {
-               nextp = ADDR(vlentry.nextIdHash[1]);
-               reft = REFRO;
                hash = IdHash(vlentry.volumeId[1]);
-               nextpp = &(vlentry.nextIdHash[1]);
                which = "RO";
                sprintf(volidbuf, "id %u ", vlentry.volumeId[1]);
                if (validVolumeAddr(vlentry.nextIdHash[1])) {
@@ -1318,10 +1358,7 @@ WorkerBee(struct cmd_syndesc *as, void *arock)
 
            if (!validVolumeAddr(vlentry.nextIdHash[2]) ||
                record[ADDR(vlentry.nextIdHash[2])].type & MULTBK) {
-               nextp = ADDR(vlentry.nextIdHash[2]);
-               reft = REFBK;
                hash = IdHash(vlentry.volumeId[2]);
-               nextpp = &(vlentry.nextIdHash[2]);
                which = "BK";
                sprintf(volidbuf, "id %u ", vlentry.volumeId[2]);
                if (validVolumeAddr(vlentry.nextIdHash[2])) {
@@ -1335,196 +1372,246 @@ WorkerBee(struct cmd_syndesc *as, void *arock)
            }
 
            if (foundbroken) {
-               log_error(VLDB_CHECK_ERROR, "%d: Volume '%s' %s forward link in %s hash chain is broken (hash %d != %d)", i,
-                         vlentry.name, volidbuf, which, hash, nexthash);
+               log_error(VLDB_CHECK_ERROR,
+                       "address %u (offset 0x%0x): Volume '%s' %s forward link in %s hash chain is broken (hash %d != %d)\n",
+                       record[i].addr, OFFSET(record[i].addr),
+                       vlentry.name, volidbuf, which, hash, nexthash);
            } else if (foundbad) {
-               log_error(VLDB_CHECK_ERROR, "%d: Volume '%s' %snot found in %s hash %d", i,
-                      vlentry.name, volidbuf, which, hash);
-           }
-
-           if (foundbad || foundbroken) {
-               if (nextp && fix) {
-                   log_error(VLDB_CHECK_ERROR, ", unchaining next (%d)\n", nextp);
-                   *nextpp = 0;
-                   writeentry(record[i].addr, &vlentry);
-               } else {
-                   log_error(VLDB_CHECK_ERROR, "\n");
-               }
-               
+               log_error(VLDB_CHECK_ERROR,
+                       "address %u (offset 0x%0x): Volume '%s' %snot found in %s hash %d\n",
+                       record[i].addr, OFFSET(record[i].addr),
+                       vlentry.name, volidbuf, which, hash);
            }
 
            for (j = 0; j < NMAXNSERVERS; j++) {
-               if ((vlentry.serverNumber[j] != 255)
-                   && (serveraddrs[vlentry.serverNumber[j]] == 0)) {
-                  log_error 
-                       (VLDB_CHECK_ERROR,"Volume '%s', index %d points to empty server entry %d\n",
-                        vlentry.name, j, vlentry.serverNumber[j]);
+               if (vlentry.serverNumber[j] != BADSERVERID) {
+                   serverref[vlentry.serverNumber[j]] = 1;
+                   if (serveraddrs[vlentry.serverNumber[j]] == 0) {
+                       log_error
+                           (VLDB_CHECK_ERROR,"address %u (offset 0x%0x): Volume '%s', index %d points to empty server entry %d\n",
+                            record[i].addr, OFFSET(record[i].addr), vlentry.name, j, vlentry.serverNumber[j]);
+                   } else if (serverxref[vlentry.serverNumber[j]] != BADSERVERID) {
+                           log_error
+                           (VLDB_CHECK_ERROR,"address %u (offset 0x%0x): Volume '%s', index %d points to server entry %d, which is cross-linked by %d\n",
+                            record[i].addr, OFFSET(record[i].addr), vlentry.name, j, vlentry.serverNumber[j], serverxref[vlentry.serverNumber[j]]);
+                   }
                }
            }
-       
+
            if (record[i].type & 0xffff0f00)
-               log_error       
-                   (VLDB_CHECK_ERROR,"Volume '%s' id %u also found on other chains (0x%x)\n",
-                    vlentry.name, vlentry.volumeId[0], record[i].type);
-           
+               log_error
+                   (VLDB_CHECK_ERROR,"address %u (offset 0x%0x): Volume '%s' id %u also found on other chains (0x%x)\n",
+                    record[i].addr, OFFSET(record[i].addr), vlentry.name, vlentry.volumeId[0], record[i].type);
+
            /* A free entry */
        } else if (record[i].type & FR) {
            if (!(record[i].type & FRC))
-               log_error(VLDB_CHECK_ERROR,"Free vlentry at %ld not on free chain\n",
-                      record[i].addr);
-           
+               log_error(VLDB_CHECK_ERROR,"address %u (offset 0x%0x): Free vlentry not on free chain\n",
+                      record[i].addr, OFFSET(record[i].addr));
+
            if (record[i].type & 0xfffffdf0)
-               log_error       
-                   (VLDB_CHECK_ERROR,"Free vlentry at %ld also found on other chains (0x%x)\n",
-                    record[i].addr, record[i].type);
-           
+               log_error
+                   (VLDB_CHECK_ERROR,"address %u (offset 0x%0x): Free vlentry also found on other chains (0x%x)\n",
+                    record[i].addr, OFFSET(record[i].addr), record[i].type);
+
            /* A multihomed entry */
        } else if (record[i].type & MH) {
            if (!(record[i].type & MHC))
-               log_error(VLDB_CHECK_ERROR,"Multihomed block at %ld is orphaned\n",
-                      record[i].addr);
-           
+               log_error(VLDB_CHECK_ERROR,"address %u (offset 0x%0x): Multihomed block is orphaned\n",
+                      record[i].addr, OFFSET(record[i].addr));
+
            if (record[i].type & 0xfffffef0)
-               log_error       
-                   (VLDB_CHECK_ERROR,"Multihomed block at %ld also found on other chains (0x%x)\n",
-                    record[i].addr, record[i].type);
-           
+               log_error
+                   (VLDB_CHECK_ERROR,"address %u (offset 0x%0x): Multihomed block also found on other chains (0x%x)\n",
+                    record[i].addr, OFFSET(record[i].addr), record[i].type);
+
        } else {
-           log_error(VLDB_CHECK_ERROR,"Unknown entry type at %u (0x%x)\n", record[i].addr,
-                  record[i].type);
+           log_error(VLDB_CHECK_ERROR,"address %u (offset 0x%0x): Unknown entry type 0x%x\n",
+                  record[i].addr, OFFSET(record[i].addr), record[i].type);
        }
     }
 
-    if (verbose)  quiet_println("Verify each chain head\n");
-    {
-       afs_uint32 addr;
-       int hash;
+    if (fix) {
+       /*
+        * If we are fixing we will rebuild the free and hash lists from the ground up.
+        */
+       header.vital_header.freePtr = 0;
+       memcpy(oldnamehash, header.VolnameHash, sizeof(oldnamehash));
+       memset(header.VolnameHash, 0, sizeof(header.VolnameHash));
 
-       for (j = 0; j < HASHSIZE; j++) {
-           for (addr = header.VolnameHash[j]; j < HASHSIZE; j++) {
-               if (record[ADDR(addr)].type & MULTN) {
-                   hash = NameHash(vlentry.name);
-                   if (hash != j) {
-                       header.VolnameHash[j] = vlentry.nextNameHash;
-                       vlentry.nextNameHash = 0;
-                       if (fix)
-                           writeentry(record[i].addr, &vlentry);
-                   }
-               }
+       memcpy(oldidhash, header.VolidHash, sizeof(oldidhash));
+       memset(header.VolidHash, 0, sizeof(header.VolidHash));
+       quiet_println("Rebuilding %u entries\n", maxentries);
+    } else {
+       quiet_println("Scanning %u entries for possible repairs\n", maxentries);
+    }
+    for (i = 0; i < maxentries; i++) {
+       afs_uint32 hash;
+       if (record[i].type & VL) {
+           readentry(record[i].addr, &vlentry, &type);
+           if (!(record[i].type & REFN)) {
+               log_error(VLDB_CHECK_ERROR,"address %u (offset 0x%0x): Record is not in a name chain (type 0x%0x)\n",
+                      record[i].addr, OFFSET(record[i].addr), record[i].type);
            }
-       }
-       for (i = 0; i <= 2; i++) {
-         for (j = 0, addr = header.VolidHash[i][j]; j < HASHSIZE; j++) {
-        if (verbose) quiet_println("got %d %d %d\n", i, j, ADDR(addr));
-               if (i == 0 && (record[ADDR(addr)].type & MULTRW)) {
-                   hash = IdHash(vlentry.volumeId[i]);
-                   if (hash != j) {
-                       header.VolidHash[i][j] = vlentry.nextIdHash[i];
-                       vlentry.nextIdHash[i] = 0;
-                       if (fix) {
-                         quiet_println("fix %d %d %d\n", i, j, ADDR(addr));
-                           writeentry(record[i].addr, &vlentry);
-                   }
+           if (vlentry.volumeId[0] && !(record[i].type & REFRW)) {
+               log_error(VLDB_CHECK_ERROR,"address %u (offset 0x%0x): Record not in a RW chain (type 0x%0x)\n",
+                      record[i].addr, OFFSET(record[i].addr), record[i].type);
+           }
+           if (vlentry.volumeId[1] && !(record[i].type & REFRO)) {
+               log_error(VLDB_CHECK_ERROR,"address %u (offset 0x%0x): Record not in a RO chain (type 0x%0x)\n",
+                      record[i].addr, OFFSET(record[i].addr), record[i].type);
+           }
+           if (vlentry.volumeId[2] && !(record[i].type & REFBK)) {
+               log_error(VLDB_CHECK_ERROR,"address %u (offset 0x%0x): Record not in a BK chain (type 0x%0x)\n",
+                      record[i].addr, OFFSET(record[i].addr), record[i].type);
+           }
+           if (fix) {
+               afs_uint32 oldhash, newhash;
+               char oldNameBuffer[10 + VL_MAXNAMELEN];
+               char newNameBuffer[10 + VL_MAXNAMELEN];
+               char *oldname, *newname;
+
+               /* Fix broken names and numbers so entries can be inspected and deleted. */
+               if (InvalidVolname(vlentry.name)) {
+                   char bogus[VL_MAXNAMELEN];
+                   memset(bogus, 0, sizeof(bogus));
+                   snprintf(bogus, sizeof(bogus)-1, ".bogus.%ld", record[i].addr);
+                   strcpy(vlentry.name, bogus);
+                   quiet_println("FIX: Record %ld invalid volume name set to '%s'\n", record[i].addr, bogus);
                }
+               if (vlentry.volumeId[0] == 0) {
+                   afs_uint32 next_volid = header.vital_header.MaxVolumeId++;
+                   vlentry.volumeId[0] = next_volid;
+                   quiet_println("FIX: Record %ld invalid volume id set to %ld. New max volid is %ld\n",
+                       record[i].addr, next_volid, header.vital_header.MaxVolumeId);
                }
 
-               if (i == 1 && (record[ADDR(addr)].type & MULTRO)) {
-                   hash = IdHash(vlentry.volumeId[i]);
-                   if (hash != j) {
-                       header.VolidHash[i][j] = vlentry.nextIdHash[i];
-                       vlentry.nextIdHash[i] = 0;
-                       if (fix) {
-                         quiet_println("fix %d %d %d\n", i, j, addr);
-                           writeentry(record[i].addr, &vlentry);
+               /*
+                * Put the current hash table contexts into our 'next'
+                * and our address into the hash table.
+                */
+               hash = NameHash(vlentry.name);
+
+               if (vlentry.nextNameHash != header.VolnameHash[hash]) {
+                   oldname = nameForAddr(vlentry.nextNameHash, MAXTYPES, &oldhash, oldNameBuffer);
+                   newname = nameForAddr(header.VolnameHash[hash], MAXTYPES, &newhash, newNameBuffer);
+                   if (verbose || ((oldhash != newhash) &&
+                                    (0 != vlentry.nextNameHash) &&
+                                    (0 != header.VolnameHash[hash]))) {
+                       /*
+                        * That is, only report if we are verbose
+                        * or the hash is changing (and one side wasn't NULL
+                        */
+                       quiet_println("FIX: Name hash link for '%s' was %s, is now %s\n",
+                              vlentry.name, oldname, newname);
                    }
                }
-               }
 
-               if (i == 2 && (record[ADDR(addr)].type & MULTBK)) {
-                   hash = IdHash(vlentry.volumeId[i]);
-                   if (hash != j) {
-                       header.VolidHash[i][j] = vlentry.nextIdHash[i];
-                       vlentry.nextIdHash[i] = 0;
-                       if (fix) {
-                         quiet_println("fix %d %d %d\n", i, j, addr);
-                           writeentry(record[i].addr, &vlentry);
+               vlentry.nextNameHash = header.VolnameHash[hash];
+               header.VolnameHash[hash] = record[i].addr;
+
+               for (j = 0; j < MAXTYPES; j++) {
+
+                   if (0 == vlentry.volumeId[j]) {
+                       /*
+                        * No volume of that type.  Continue
+                        */
+                       continue;
                    }
-               }
-           }
-       }
-    }
-    }
-    /* By the time we get here, unchained entries are really unchained */
-    quiet_println("Scanning %u entries for possible repairs\n", maxentries);
-    for (i = 0; i < maxentries; i++) {
-       int *nextpp;
-       if (record[i].type & VL) {
-           readentry(record[i].addr, &vlentry, &type);
-           if (!(record[i].type & REFN)) {
-               log_error(VLDB_CHECK_ERROR,"%d: Record %ld (type 0x%x) not in a name chain\n", i, 
-                      record[i].addr, record[i].type);
-               if (strlen(vlentry.name)>0) {
-                   if (fix) {
-                       if (header.VolnameHash[NameHash(vlentry.name)] == 0)
-                           header.VolnameHash[NameHash(vlentry.name)] = record[i].addr;
-                       else
-                           FixBad(i, header.VolnameHash[NameHash(vlentry.name)], NH, record[i].addr, &vlentry, NameHash(vlentry.name));
+                   hash = IdHash(vlentry.volumeId[j]);
+
+                   if (vlentry.nextIdHash[j] != header.VolidHash[j][hash]) {
+                       oldname = nameForAddr(vlentry.nextIdHash[j], j, &oldhash, oldNameBuffer);
+                       newname = nameForAddr(header.VolidHash[j][hash], j, &newhash, newNameBuffer);
+                       if (verbose || ((oldhash != newhash) &&
+                                       (0 != vlentry.nextIdHash[j]) &&
+                                       (0 != header.VolidHash[j][hash]))) {
+                           quiet_println("FIX: %s hash link for '%s' was %s, is now %s\n",
+                                         vtype(j), vlentry.name, oldname, newname);
+                       }
                    }
-               } else {
-                   nextpp = &vlentry.nextNameHash;
-                   if (fix && *nextpp) {
-                       printf(", unchaining");
-                       *nextpp = 0;
-                       writeentry(record[i].addr, &vlentry);
+
+                   /* Consolidate server numbers which point to the same mh entry.
+                    * The serverref flags are not reset here, since we want to make
+                    * sure the data is actually written before the server number is
+                    * considered unreferenced. */
+                   for (k = 0; k < NMAXNSERVERS; k++) {
+                       if (vlentry.serverNumber[k] != BADSERVERID
+                           && serverxref[vlentry.serverNumber[k]] != BADSERVERID) {
+                           u_char oldsn = vlentry.serverNumber[k];
+                           u_char newsn = serverxref[oldsn];
+                           if (newsn == oldsn) {
+                               log_error(VLDB_CHECK_ERROR,
+                                         "INTERNAL VLDB_CHECK_ERROR: serverxref points to self; index %d\n",
+                                         oldsn);
+                           } else if (header.IpMappedAddr[oldsn] == 0) {
+                               log_error(VLDB_CHECK_ERROR,
+                                         "INTERNAL VLDB_CHECK_ERROR: serverxref; points to empty address; index %d, value %d\n",
+                                         oldsn, newsn);
+                           } else if (header.IpMappedAddr[newsn] != header.IpMappedAddr[oldsn]) {
+                               log_error(VLDB_CHECK_ERROR,
+                                         "INTERNAL VLDB_CHECK_ERROR: invalid serverxref; index %d\n",
+                                         oldsn);
+                           } else {
+                               quiet_println
+                                   ("FIX: Volume '%s', index %d, server number was %d, is now %d\n",
+                                    vlentry.name, k, oldsn, newsn);
+                               vlentry.serverNumber[k] = newsn;
+                           }
+                       }
                    }
+
+                   vlentry.nextIdHash[j] = header.VolidHash[j][hash];
+                   header.VolidHash[j][hash] = record[i].addr;
                }
+               writeentry(record[i].addr, &vlentry);
            }
-           if (vlentry.volumeId[0] && !(record[i].type & REFRW)) {
-               log_error(VLDB_CHECK_ERROR,"%d: Record %ld (type 0x%x) not in a RW chain\n", i,
-                      record[i].addr, record[i].type);
-               if (fix) {
-                   if (header.VolidHash[0][IdHash(vlentry.volumeId[0])] == 0)
-                       header.VolidHash[0][IdHash(vlentry.volumeId[0])] = record[i].addr;
-                   else
-                       FixBad(i, header.VolidHash[0][IdHash(vlentry.volumeId[0])], RWH, record[i].addr, &vlentry, IdHash(vlentry.volumeId[0]));
+       }
+       else if (record[i].type & MH) {
+           int block, index;
+           char mhblock[VL_ADDREXTBLK_SIZE];
+           struct extentaddr *MHblock = (struct extentaddr *)mhblock;
+
+           if (fix) {
+               for (block = 0; block < VL_MAX_ADDREXTBLKS; block++) {
+                   if (mhinfo[block].addr == record[i].addr)
+                       break;
                }
-           }
-           if (vlentry.volumeId[1] && !(record[i].type & REFRO)) {
-               log_error(VLDB_CHECK_ERROR,"%d: Record %ld (type 0x%x) not in a RO chain\n", i, 
-                      record[i].addr, record[i].type);
-               if (fix) {
-                   if (header.VolidHash[1][IdHash(vlentry.volumeId[1])] == 0)
-                       header.VolidHash[1][IdHash(vlentry.volumeId[1])] = record[i].addr;
-                   else
-                       FixBad(i, header.VolidHash[1][IdHash(vlentry.volumeId[1])], ROH, record[i].addr, &vlentry, IdHash(vlentry.volumeId[1]));
+               if (block == VL_MAX_ADDREXTBLKS) {
+                   continue;  /* skip orphaned extent block */
+               }
+               readMH(record[i].addr, block, MHblock);
+               for (index = 0; index < VL_MHSRV_PERBLK; index++) {
+                   if (mhinfo[block].orphan[index]) {
+                       quiet_println("FIX: Removing unreferenced mh entry; block %d, index %d\n",
+                               block, index);
+                       memset(&(MHblock[index]), 0, sizeof(struct extentaddr));
+                   }
                }
+               writeMH(record[i].addr, block, MHblock);
            }
-           if (vlentry.volumeId[2] && !(record[i].type & REFBK)) {
-               log_error(VLDB_CHECK_ERROR,"%d: Record %ld (type 0x%x) not in a BK chain\n", i, 
-                      record[i].addr, record[i].type);
-               if (fix) {
-                   if (header.VolidHash[2][IdHash(vlentry.volumeId[2])] == 0)
-                       header.VolidHash[2][IdHash(vlentry.volumeId[2])] = record[i].addr;
-                   else
-                       FixBad(i, header.VolidHash[2][IdHash(vlentry.volumeId[2])], BKH, record[i].addr, &vlentry, IdHash(vlentry.volumeId[2]));
+       } else if (record[i].type & FR) {
+           if (fix) {
+               readentry(record[i].addr, &vlentry, &type);
+               vlentry.nextIdHash[0] = header.vital_header.freePtr;
+               header.vital_header.freePtr = record[i].addr;
+               if ((record[i].type & FRC) == 0) {
+                   quiet_println
+                       ("FIX: Putting free entry on the free chain: addr=%lu (offset 0x%0x)\n",
+                        record[i].addr, OFFSET(record[i].addr));
                }
-
+               writeentry(record[i].addr, &vlentry);
            }
        }
     }
-    if (fix) 
+    if (fix) {
+       reportHashChanges(&header, oldnamehash, oldidhash);
+       removeCrossLinkedAddresses(&header);
        writeheader(&header);
+    }
 
     close(fd);
 
-    if (fixed) {
-      fixed=0;
-      passes++;
-      if (passes < 20)
-       goto restart;
-      else
-       return 1;
-    }
     return error_level;
 }
 
@@ -1535,7 +1622,7 @@ main(int argc, char **argv)
 
     setlinebuf(stdout);
 
-    ts = cmd_CreateSyntax(NULL, WorkerBee, NULL, "vldb check");
+    ts = cmd_CreateSyntax(NULL, WorkerBee, NULL, 0, "vldb check");
     cmd_AddParm(ts, "-database", CMD_SINGLE, CMD_REQUIRED, "vldb_file");
     cmd_AddParm(ts, "-uheader", CMD_FLAG, CMD_OPTIONAL,
                "Display UBIK header");