Add printf format checks to afs_com_err()
[openafs.git] / src / bucoord / ubik_db_if.c
index dad67e1..2588867 100644 (file)
 /* Interface and supporting routines for the backup system's ubik database */
 
 #include <afsconfig.h>
-#include <afs/param.h>
+#include <afs/stds.h>
 
-RCSID("$Header$");
+#ifdef IGNORE_SOME_GCC_WARNINGS
+# pragma GCC diagnostic warning "-Wstrict-prototypes"
+#endif
 
-#include <stdio.h>
 #include <sys/types.h>
 #include <fcntl.h>
 #ifdef AFS_NT40_ENV
@@ -22,20 +23,24 @@ RCSID("$Header$");
 #elif defined(AFS_SUN5_ENV)
 #include <netdb.h>
 #else
-#include <sys/param.h>                  /* for hostnames etc */
+#include <sys/param.h>         /* for hostnames etc */
 #endif
 #include <afs/auth.h>
 #include <afs/cellconfig.h>
 #include <ubik.h>
 #include <afs/volser.h>
+#include <afs/volser_prototypes.h>
 #include <afs/afsutil.h>
 #include <afs/bubasics.h>
 #include <afs/budb_client.h>
 #include <afs/budb.h>
+#include <afs/com_err.h>
 #include <errno.h>
 
 #include "bc.h"
 #include "error_macros.h"
+#include "bucoord_internal.h"
+#include "bucoord_prototypes.h"
 
 extern char *whoami;
 
@@ -51,64 +56,57 @@ struct udbHandleS udbHandle;
  * -------------------------------------
  */
 
-bcdb_AddVolume(veptr)
-     register struct budb_volumeEntry *veptr;
+afs_int32 bcdb_AddVolume(register struct budb_volumeEntry *veptr)
 {
     afs_int32 code;
 
-    code = ubik_Call( BUDB_AddVolume,  udbHandle.uh_client, 0, veptr);
-    return(code);
+    code = ubik_BUDB_AddVolume(udbHandle.uh_client, 0, veptr);
+    return (code);
 }
 
-bcdb_AddVolumes(veptr, count)
-  register struct budb_volumeEntry *veptr;
-  afs_int32 count;
+afs_int32 bcdb_AddVolumes(register struct budb_volumeEntry *veptr, afs_int32 count)
 {
-  struct budb_volumeList volumeList;
-  afs_int32 code;
+    struct budb_volumeList volumeList;
+    afs_int32 code;
 
-  volumeList.budb_volumeList_len = count;
-  volumeList.budb_volumeList_val = veptr;
-  code = ubik_Call(BUDB_AddVolumes,  udbHandle.uh_client, 0, &volumeList);
-  return(code);
+    volumeList.budb_volumeList_len = count;
+    volumeList.budb_volumeList_val = veptr;
+    code = ubik_BUDB_AddVolumes(udbHandle.uh_client, 0, &volumeList);
+    return (code);
 }
 
 
-bcdb_CreateDump(deptr)
-     register struct budb_dumpEntry *deptr;
+afs_int32 bcdb_CreateDump(register struct budb_dumpEntry *deptr)
 {
     afs_int32 code;
 
-    code = ubik_Call(BUDB_CreateDump, udbHandle.uh_client, 0, deptr);
-    return(code);    
+    code = ubik_BUDB_CreateDump(udbHandle.uh_client, 0, deptr);
+    return (code);
 }
 
-bcdb_deleteDump(dumpID, fromTime, toTime, dumps)
-    afs_int32 dumpID;
-    afs_int32 fromTime;
-    afs_int32 toTime;
-    budb_dumpsList *dumps;
+afs_int32 bcdb_deleteDump(afs_int32 dumpID, afs_int32 fromTime, afs_int32 toTime, 
+  budb_dumpsList *dumps)
 {
     afs_int32 code;
     budb_dumpsList dumpsList, *dumpsPtr;
 
     dumpsList.budb_dumpsList_len = 0;
     dumpsList.budb_dumpsList_val = 0;
-    dumpsPtr = ( dumps ? dumps : &dumpsList );
+    dumpsPtr = (dumps ? dumps : &dumpsList);
 
-    code = ubik_Call(BUDB_DeleteDump, udbHandle.uh_client, 0, dumpID, fromTime, toTime, dumpsPtr);
-    if (dumpsList.budb_dumpsList_val) free(dumpsList.budb_dumpsList_val);
-    return(code);    
+    code =
+       ubik_BUDB_DeleteDump(udbHandle.uh_client, 0, dumpID, fromTime,
+                 toTime, dumpsPtr);
+    if (dumpsList.budb_dumpsList_val)
+       free(dumpsList.budb_dumpsList_val);
+    return (code);
 }
 
-bcdb_listDumps(sflags, groupId, fromTime, toTime, dumps, flags)
-    afs_int32 groupId;
-    afs_int32 fromTime;
-    afs_int32 toTime;
-    budb_dumpsList *dumps;
-    budb_dumpsList *flags;
+afs_int32 bcdb_listDumps (afs_int32 sflags, afs_int32 groupId,
+                         afs_int32 fromTime, afs_int32 toTime,
+                         budb_dumpsList *dumps, budb_dumpsList *flags)
 {
-    afs_int32 code, sflag=0;
+    afs_int32 code;
     budb_dumpsList dumpsList, *dumpsPtr;
     budb_dumpsList flagsList, *flagsPtr;
 
@@ -120,24 +118,26 @@ bcdb_listDumps(sflags, groupId, fromTime, toTime, dumps, flags)
     flagsList.budb_dumpsList_val = 0;
     flagsPtr = (flags ? flags : &flagsList);
 
-    code = ubik_Call(BUDB_ListDumps, udbHandle.uh_client, 0, 
-                    sflags, "", groupId, fromTime, toTime, dumpsPtr, flagsPtr);
+    code =
+       ubik_BUDB_ListDumps(udbHandle.uh_client, 0, sflags, "", groupId,
+                 fromTime, toTime, dumpsPtr, flagsPtr);
 
-    if (dumpsList.budb_dumpsList_val) free(dumpsList.budb_dumpsList_val);
-    if (flagsList.budb_dumpsList_val) free(flagsList.budb_dumpsList_val);
-    return(code);    
+    if (dumpsList.budb_dumpsList_val)
+       free(dumpsList.budb_dumpsList_val);
+    if (flagsList.budb_dumpsList_val)
+       free(flagsList.budb_dumpsList_val);
+    return (code);
 }
 
 
-bcdb_DeleteVDP(dumpSetName, dumpPath, dumpID)
-     char *dumpSetName;
-     char *dumpPath;
-     afs_int32 dumpID;
+afs_int32 bcdb_DeleteVDP(char *dumpSetName, char *dumpPath, afs_int32 dumpID)
 {
     afs_int32 code;
 
-    code = ubik_Call( BUDB_DeleteVDP,  udbHandle.uh_client, 0, dumpSetName, dumpPath, dumpID);
-    return(code);    
+    code =
+       ubik_BUDB_DeleteVDP(udbHandle.uh_client, 0, dumpSetName,
+                 dumpPath, dumpID);
+    return (code);
 }
 
 /* bcdb_FindClone
@@ -154,15 +154,13 @@ bcdb_DeleteVDP(dumpSetName, dumpPath, dumpID)
  *      -2 - no clone times found, clone time set to 0
  */
 
-afs_int32
-bcdb_FindClone(dumpID, volName, clonetime)
-     afs_int32 dumpID;
-     char *volName;
-     afs_int32 *clonetime;
+afs_int32 bcdb_FindClone(afs_int32 dumpID, char *volName, afs_int32 *clonetime)
 {
     afs_int32 code;
-    code = ubik_Call( BUDB_FindClone,  udbHandle.uh_client, 0, dumpID, volName, clonetime);
-    return(code);    
+    code =
+       ubik_BUDB_FindClone(udbHandle.uh_client, 0, dumpID, volName,
+                 clonetime);
+    return (code);
 }
 
 /* bcdb_FindDump
@@ -182,14 +180,15 @@ bcdb_FindClone(dumpID, volName, clonetime)
  *      the current multiple bcdb_ call algorithm.
  */
 
-bcdb_FindDump(volumeName, beforeDate, deptr)
-     char *volumeName;
-     afs_int32 beforeDate;
-     struct budb_dumpEntry *deptr;
+int
+bcdb_FindDump(char *volumeName, afs_int32 beforeDate, 
+             struct budb_dumpEntry *deptr)
 {
     afs_int32 code;
-    code = ubik_Call( BUDB_FindDump,  udbHandle.uh_client, 0, volumeName, beforeDate, deptr);
-    return(code);    
+    code =
+       ubik_BUDB_FindDump(udbHandle.uh_client, 0, volumeName,
+                 beforeDate, deptr);
+    return (code);
 }
 
 /* bcdb_FindDumpByID
@@ -198,13 +197,12 @@ bcdb_FindDump(volumeName, beforeDate, deptr)
  *     dumpID - id to lookup
  * exit:
  */
-bcdb_FindDumpByID(dumpID, deptr)
-     afs_int32 dumpID;
-     register struct budb_dumpEntry *deptr;
+int
+bcdb_FindDumpByID(afs_int32 dumpID, struct budb_dumpEntry *deptr)
 {
     register afs_int32 code;
-    afs_int32  nextindex;
-    afs_int32  dbTime;
+    afs_int32 nextindex;
+    afs_int32 dbTime;
     budb_dumpList dl;
 
     /* initialize the dump list */
@@ -212,39 +210,32 @@ bcdb_FindDumpByID(dumpID, deptr)
     dl.budb_dumpList_val = 0;
 
     /* outline algorithm */
-    code = ubik_Call (BUDB_GetDumps,  udbHandle.uh_client, 0,
-                     BUDB_MAJORVERSION,
-                     BUDB_OP_DUMPID,
-                     "",                       /* no name */
-                     dumpID,                   /* start */
-                     0,                        /* end */
-                     0,                        /* index */
-                     &nextindex,
-                     &dbTime,
-                     &dl);
-
-    if ( (code != 0)
-    ||   (dl.budb_dumpList_len != 1)           /* single retn val expected */
-       )
-    {
+    code = ubik_BUDB_GetDumps(udbHandle.uh_client, 0, BUDB_MAJORVERSION, BUDB_OP_DUMPID, "",   /* no name */
+                    dumpID,    /* start */
+                    0,         /* end */
+                    0,         /* index */
+                    &nextindex, &dbTime, &dl);
+
+    if ((code != 0)
+       || (dl.budb_dumpList_len != 1)  /* single retn val expected */
+       ) {
 /*     printf("bcdb_FindDumpByID: code %d, nvalues %d\n",
               code, dl.budb_dumpList_len); */
-       if ( code == 0 )
-               code = 1;                       /* multiple id's */
+       if (code == 0)
+           code = 1;           /* multiple id's */
        goto error;
     }
 
     memcpy(deptr, dl.budb_dumpList_val, sizeof(*deptr));
 
-exit:
-    if ( dl.budb_dumpList_val )
-    {
+  exit:
+    if (dl.budb_dumpList_val) {
        /* free any allocated structures */
        free(dl.budb_dumpList_val);
     }
-    return(code);
+    return (code);
 
-error:
+  error:
     memset(deptr, 0, sizeof(*deptr));
     goto exit;
 }
@@ -267,18 +258,15 @@ error:
  */
 
 afs_int32
-bcdb_FindLastVolClone(volSetName, dumpName, volName, clonetime)
-     char *volSetName;
-     char *dumpName;
-     char *volName;
-     afs_int32 *clonetime;
+bcdb_FindLastVolClone(char *volSetName, char *dumpName, char *volName, 
+                     afs_int32 *clonetime)
 {
     /* server notes
-       search by dumpName
-       match on volumeset and dump path
-       search for the volume name
+     * search by dumpName
+     * match on volumeset and dump path
+     * search for the volume name
      */
-    return(0);
+    return (0);
 }
 
 /* bcdb_FindLatestDump
@@ -298,15 +286,15 @@ bcdb_FindLastVolClone(volSetName, dumpName, volName, clonetime)
  *      Need new routine since params are two strings
  */
 
-bcdb_FindLatestDump(volSetName, dumpPath, deptr)
-     char *volSetName;
-     char *dumpPath;
-     struct budb_dumpEntry *deptr;
+int
+bcdb_FindLatestDump(char *volSetName, char *dumpPath, 
+                   struct budb_dumpEntry *deptr)
 {
     afs_int32 code;
-    code = ubik_Call( BUDB_FindLatestDump,  udbHandle.uh_client, 0,
-                     volSetName, dumpPath, deptr);
-    return(code);
+    code =
+       ubik_BUDB_FindLatestDump(udbHandle.uh_client, 0, volSetName,
+                 dumpPath, deptr);
+    return (code);
 }
 
 
@@ -317,10 +305,9 @@ bcdb_FindLatestDump(volSetName, dumpPath, deptr)
  *     tapeName: name of tape
  */
 
-bcdb_FindTape(dumpid, tapeName, teptr)
-     afs_int32 dumpid;
-     char *tapeName;
-     struct budb_tapeEntry *teptr;
+int
+bcdb_FindTape(afs_int32 dumpid, char *tapeName, 
+             struct budb_tapeEntry *teptr)
 {
     budb_tapeList tl;
     afs_int32 next;
@@ -331,28 +318,28 @@ bcdb_FindTape(dumpid, tapeName, teptr)
     tl.budb_tapeList_len = 0;
     tl.budb_tapeList_val = 0;
 
-    code = ubik_Call( BUDB_GetTapes, udbHandle.uh_client, 0,
-                     BUDB_MAJORVERSION,
-                     BUDB_OP_TAPENAME|BUDB_OP_DUMPID,
-                     tapeName, dumpid, 0, 0, &next, &dbTime, &tl);
+    code =
+       ubik_BUDB_GetTapes(udbHandle.uh_client, 0, BUDB_MAJORVERSION,
+                 BUDB_OP_TAPENAME | BUDB_OP_DUMPID, tapeName, dumpid, 0, 0,
+                 &next, &dbTime, &tl);
 
-    if (code) ERROR(code);
+    if (code)
+       ERROR(code);
 
     if (tl.budb_tapeList_len != 1)
        ERROR(BC_NOTUNIQUE);    /* expecting a single descriptor */
 
     memcpy(teptr, tl.budb_tapeList_val, sizeof(*teptr));
 
-error_exit:                 
-    if ( tl.budb_tapeList_val )
+  error_exit:
+    if (tl.budb_tapeList_val)
        free(tl.budb_tapeList_val);
-    return(code);
+    return (code);
 }
 
-bcdb_FindTapeSeq(dumpid, tapeSeq, teptr)
-     afs_int32 dumpid;
-     afs_int32 tapeSeq;
-     struct budb_tapeEntry *teptr;
+int
+bcdb_FindTapeSeq(afs_int32 dumpid, afs_int32 tapeSeq, 
+                struct budb_tapeEntry *teptr)
 {
     budb_tapeList tl;
     afs_int32 next;
@@ -363,21 +350,22 @@ bcdb_FindTapeSeq(dumpid, tapeSeq, teptr)
     tl.budb_tapeList_len = 0;
     tl.budb_tapeList_val = 0;
 
-    code = ubik_Call( BUDB_GetTapes, udbHandle.uh_client, 0,
-                     BUDB_MAJORVERSION,
-                     BUDB_OP_TAPESEQ|BUDB_OP_DUMPID,
-                     "", dumpid, tapeSeq, 0, &next, &dbTime, &tl);
-    if (code) ERROR(code);
+    code =
+       ubik_BUDB_GetTapes(udbHandle.uh_client, 0, BUDB_MAJORVERSION,
+                 BUDB_OP_TAPESEQ | BUDB_OP_DUMPID, "", dumpid, tapeSeq, 0,
+                 &next, &dbTime, &tl);
+    if (code)
+       ERROR(code);
 
     if (tl.budb_tapeList_len != 1)
        ERROR(BC_NOTUNIQUE);    /* expecting a single descriptor */
 
     memcpy(teptr, tl.budb_tapeList_val, sizeof(*teptr));
 
-error_exit:                 
-    if ( tl.budb_tapeList_val )
+  error_exit:
+    if (tl.budb_tapeList_val)
        free(tl.budb_tapeList_val);
-    return(code);
+    return (code);
 }
 
 /* bcdb_FindVolumes
@@ -394,14 +382,10 @@ error_exit:
  */
 
 afs_int32
-bcdb_FindVolumes(dumpID, volumeName, returnArray, last, next, maxa, nEntries)
-     afs_int32 dumpID;
-     char *volumeName;
-     struct budb_volumeEntry *returnArray;
-     afs_int32 last;
-     afs_int32 *next;
-     afs_int32 maxa;
-     afs_int32 *nEntries;
+bcdb_FindVolumes(afs_int32 dumpID, char *volumeName, 
+                struct budb_volumeEntry *returnArray, 
+                afs_int32 last, afs_int32 *next, afs_int32 maxa, 
+                afs_int32 *nEntries)
 {
     budb_volumeList vl;
     afs_int32 dbTime;
@@ -411,50 +395,41 @@ bcdb_FindVolumes(dumpID, volumeName, returnArray, last, next, maxa, nEntries)
     vl.budb_volumeList_val = returnArray;
 
     /* outline algorithm */
-    code = ubik_Call (BUDB_GetVolumes,  udbHandle.uh_client, 0,
-                      BUDB_MAJORVERSION,
-                      BUDB_OP_VOLUMENAME|BUDB_OP_DUMPID,
-                      volumeName,               /* name */
-                      dumpID,                  /* start */
-                      0,                        /* end */
-                     last,                     /* index */
-                      next,                    /* nextindex */
-                      &dbTime,
-                      &vl);
-
-    *nEntries  = vl.budb_volumeList_len;
-    return(code);
-}
+    code = ubik_BUDB_GetVolumes(udbHandle.uh_client, 0, BUDB_MAJORVERSION, BUDB_OP_VOLUMENAME | BUDB_OP_DUMPID, volumeName,    /* name */
+                    dumpID,    /* start */
+                    0,         /* end */
+                    last,      /* index */
+                    next,      /* nextindex */
+                    &dbTime, &vl);
 
+    *nEntries = vl.budb_volumeList_len;
+    return (code);
+}
 
-bcdb_FinishDump(deptr)
-     register struct budb_dumpEntry *deptr;
+int
+bcdb_FinishDump(register struct budb_dumpEntry *deptr)
 {
-    afs_int32 code;    
-    code = ubik_Call (BUDB_FinishDump,  udbHandle.uh_client, 0, deptr);
-    return(code);    
+    afs_int32 code;
+    code = ubik_BUDB_FinishDump(udbHandle.uh_client, 0, deptr);
+    return (code);
 }
 
-bcdb_FinishTape(teptr)
-     register struct budb_tapeEntry *teptr;
+int
+bcdb_FinishTape(register struct budb_tapeEntry *teptr)
 {
-    afs_int32 code;    
-    code = ubik_Call (BUDB_FinishTape,  udbHandle.uh_client, 0, teptr);
-    return(code);    
-    
+    afs_int32 code;
+    code = ubik_BUDB_FinishTape(udbHandle.uh_client, 0, teptr);
+    return (code);
+
 }
 
 /* bcdb_LookupVolumes
  */
 
 afs_int32
-bcdb_LookupVolume(volumeName, returnArray, last, next, maxa, nEntries)
-     char *volumeName;
-     struct budb_volumeEntry *returnArray;
-     afs_int32 last;
-     afs_int32 *next;
-     afs_int32 maxa;
-     afs_int32 *nEntries;
+bcdb_LookupVolume(char *volumeName, struct budb_volumeEntry *returnArray, 
+                 afs_int32 last, afs_int32 *next, afs_int32 maxa, 
+                 afs_int32 *nEntries)
 {
     budb_volumeList vl;
     afs_int32 dbTime;
@@ -464,33 +439,26 @@ bcdb_LookupVolume(volumeName, returnArray, last, next, maxa, nEntries)
     vl.budb_volumeList_val = returnArray;
 
     /* outline algorithm */
-    code = ubik_Call (BUDB_GetVolumes,  udbHandle.uh_client, 0,
-                      BUDB_MAJORVERSION,
-                      BUDB_OP_VOLUMENAME,
-                      volumeName,               /* name */
-                      0,                       /* start */
-                      0,                        /* end */
-                     last,                     /* index */
-                      next,                    /* nextindex */
-                      &dbTime,
-                      &vl);
-    if ( code )
-    {
+    code = ubik_BUDB_GetVolumes(udbHandle.uh_client, 0, BUDB_MAJORVERSION, BUDB_OP_VOLUMENAME, volumeName,     /* name */
+                    0,         /* start */
+                    0,         /* end */
+                    last,      /* index */
+                    next,      /* nextindex */
+                    &dbTime, &vl);
+    if (code) {
        *nEntries = 0;
-       return(code);
+       return (code);
     }
     *nEntries = vl.budb_volumeList_len;
-    return(0);
+    return (0);
 }
 
-
-bcdb_UseTape(teptr, newFlag)
-     register struct budb_tapeEntry *teptr;
-     afs_int32 *newFlag;
+int
+bcdb_UseTape(struct budb_tapeEntry *teptr, afs_int32 *newFlag)
 {
-    afs_int32 code;    
-    code = ubik_Call (BUDB_UseTape,  udbHandle.uh_client, 0, teptr, newFlag);
-    return(code);    
+    afs_int32 code;
+    code = ubik_BUDB_UseTape(udbHandle.uh_client, 0, teptr, newFlag);
+    return (code);
 }
 
 
@@ -500,7 +468,7 @@ bcdb_UseTape(teptr, newFlag)
  *     The caller should pass in/out a fid for an unlinked, open file to prevent
  *     tampering with the files contents; 
  */
+
 /* bcdb_GetTextFile
  *     extract the specified textType and put it in a temporary, local
  *     file.
@@ -508,10 +476,11 @@ bcdb_UseTape(teptr, newFlag)
  *     ctPtr - ptr to client structure with all the required information
  */
 
+int
 bcdb_GetTextFile(register udbClientTextP ctPtr)
 {
     afs_int32 bufferSize;
-    afs_int32 offset, nextOffset, chunkSize; 
+    afs_int32 offset, nextOffset;
     charListT charList;
     afs_int32 code = 0;
 
@@ -520,64 +489,65 @@ bcdb_GetTextFile(register udbClientTextP ctPtr)
     memset((void *)&charList, 0, sizeof(charList));
 
     /* check params and cleanup any previous state */
-    if ( ctPtr->lockHandle == 0 )
-        ERROR(BUDB_INTERNALERROR);
+    if (ctPtr->lockHandle == 0)
+       ERROR(BUDB_INTERNALERROR);
 
-    if ( ctPtr->textStream == NULL ) /* Should have an open stream */
-        ERROR(BUDB_INTERNALERROR);
+    if (ctPtr->textStream == NULL)     /* Should have an open stream */
+       ERROR(BUDB_INTERNALERROR);
 
     /* allocate a buffer */
     bufferSize = 1024;
-    charList.charListT_val = (char *) malloc(bufferSize);
-    if ( charList.charListT_val == 0 )
-       ERROR(BUDB_INTERNALERROR);
+    charList.charListT_val = (char *)malloc(bufferSize);
+    if (charList.charListT_val == 0)
+       ERROR(BUDB_INTERNALERROR);
     charList.charListT_len = bufferSize;
 
     offset = 0;
     nextOffset = 0;
     ctPtr->textSize = 0;
-    while ( nextOffset != -1 )
-    {
+    while (nextOffset != -1) {
        offset = nextOffset;
        charList.charListT_len = bufferSize;
-       code = ubik_Call(BUDB_GetText,  udbHandle.uh_client, 0,
-                        ctPtr->lockHandle, ctPtr->textType, bufferSize,
-                        offset, &nextOffset, &charList);
+       code =
+           ubik_BUDB_GetText(udbHandle.uh_client, 0, ctPtr->lockHandle,
+                     ctPtr->textType, bufferSize, offset, &nextOffset,
+                     &charList);
 
-       if ( code )
-               ERROR(code);
+       if (code)
+           ERROR(code);
 
-       code = fwrite(charList.charListT_val, sizeof(char), 
-                     charList.charListT_len, ctPtr->textStream);
-       if ( ferror(ctPtr->textStream) )
-               ERROR(BUDB_INTERNALERROR);
+       code =
+           fwrite(charList.charListT_val, sizeof(char),
+                  charList.charListT_len, ctPtr->textStream);
+       if (ferror(ctPtr->textStream))
+           ERROR(BUDB_INTERNALERROR);
 
-        ctPtr->textSize += charList.charListT_len;
+       ctPtr->textSize += charList.charListT_len;
     }
 
     /* get text version */
-    code = ubik_Call(BUDB_GetTextVersion, udbHandle.uh_client, 0,
-                    ctPtr->textType, &ctPtr->textVersion);
-    if ( code )
-       ERROR(code);
+    code =
+       ubik_BUDB_GetTextVersion(udbHandle.uh_client, 0,
+                 ctPtr->textType, &ctPtr->textVersion);
+    if (code)
+       ERROR(code);
 
-normal_exit:
-    fflush(ctPtr->textStream);                         /* debug */
+  normal_exit:
+    fflush(ctPtr->textStream); /* debug */
 
     /* exit, leaving the configuration text file open */
-    if ( charList.charListT_val )
-       free(charList.charListT_val);
-    return(code);
-
-error_exit:
-    if ( ctPtr->textStream != NULL )
-    {
-       fclose(ctPtr->textStream);
+    if (charList.charListT_val)
+       free(charList.charListT_val);
+    return (code);
+
+  error_exit:
+    if (ctPtr->textStream != NULL) {
+       fclose(ctPtr->textStream);
        ctPtr->textStream = NULL;
     }
     goto normal_exit;
 }
-    
+
 
 /* bcdb_SaveTextFile
  *     save the text file in ubik database
@@ -586,91 +556,89 @@ error_exit:
  *     filename - where to get the text from
  */
 
-bcdb_SaveTextFile(ctPtr)
-     udbClientTextP ctPtr;
+int
+bcdb_SaveTextFile(udbClientTextP ctPtr)
 {
     afs_int32 bufferSize;
     afs_int32 offset, chunkSize, fileSize;
     charListT charList;
     afs_int32 code = 0;
-    afs_int32 filesize();
 
     /* allocate a buffer */
     bufferSize = 1024;
-    charList.charListT_val = (char *) malloc(bufferSize);
-    if ( charList.charListT_val == 0 )
-       ERROR(BUDB_INTERNALERROR);
+    charList.charListT_val = (char *)malloc(bufferSize);
+    if (charList.charListT_val == 0)
+       ERROR(BUDB_INTERNALERROR);
     charList.charListT_len = bufferSize;
 
-    if ( ctPtr->textStream == NULL )
-       ERROR(BUDB_INTERNALERROR);
-    rewind(ctPtr->textStream); 
+    if (ctPtr->textStream == NULL)
+       ERROR(BUDB_INTERNALERROR);
+    rewind(ctPtr->textStream);
 
     fileSize = (afs_int32) filesize(ctPtr->textStream);
 
-    dprintf(("filesize is %d\n", fileSize));
+    afs_dprintf(("filesize is %d\n", fileSize));
 
     rewind(ctPtr->textStream);
 
     /* special case empty files */
-    if ( fileSize == 0 )
-    {
-       charList.charListT_len = 0;     
-       code = ubik_Call(BUDB_SaveText,  udbHandle.uh_client, 0,
-                        ctPtr->lockHandle, ctPtr->textType, 0,
-                        BUDB_TEXT_COMPLETE, &charList);
+    if (fileSize == 0) {
+       charList.charListT_len = 0;
+       code =
+           ubik_BUDB_SaveText(udbHandle.uh_client, 0,
+                     ctPtr->lockHandle, ctPtr->textType, 0,
+                     BUDB_TEXT_COMPLETE, &charList);
        goto error_exit;
     }
 
     offset = 0;
-    while ( fileSize != 0 )
-    {
+    while (fileSize != 0) {
        chunkSize = MIN(fileSize, bufferSize);
-       code = fread(charList.charListT_val, sizeof(char), chunkSize,
-                    ctPtr->textStream);
+       code =
+           fread(charList.charListT_val, sizeof(char), chunkSize,
+                 ctPtr->textStream);
 
-       if ( code != chunkSize )
-               printf("code = %d\n", code );
-       if ( ferror(ctPtr->textStream ) )
-               ERROR(BUDB_INTERNALERROR);
+       if (code != chunkSize)
+           printf("code = %d\n", code);
+       if (ferror(ctPtr->textStream))
+           ERROR(BUDB_INTERNALERROR);
 
        charList.charListT_len = chunkSize;
-       code = ubik_Call(BUDB_SaveText,  udbHandle.uh_client, 0,
-                        ctPtr->lockHandle, ctPtr->textType, offset, 
-                        (chunkSize == fileSize) ? BUDB_TEXT_COMPLETE: 0,
-                        &charList);
-       if ( code )
-               ERROR(code);
+       code =
+           ubik_BUDB_SaveText(udbHandle.uh_client, 0,
+                     ctPtr->lockHandle, ctPtr->textType, offset,
+                     (chunkSize == fileSize) ? BUDB_TEXT_COMPLETE : 0,
+                     &charList);
+       if (code)
+           ERROR(code);
 
        fileSize -= chunkSize;
        offset += chunkSize;
     }
-    
-error_exit:
+
+  error_exit:
     /* if ( ctPtr->textStream >= 0 )
-       close(ctPtr->textStream); */
-    if ( charList.charListT_val )
-       free(charList.charListT_val);
-    return(code);
+     * close(ctPtr->textStream); */
+    if (charList.charListT_val)
+       free(charList.charListT_val);
+    return (code);
 }
 
-bcdb_FindLastTape(dumpID, dumpEntry, tapeEntry, volEntry)
-    afs_int32                    dumpID;
-    struct budb_dumpEntry   *dumpEntry;
-    struct budb_tapeEntry   *tapeEntry;
-    struct budb_volumeEntry *volEntry;
+int
+bcdb_FindLastTape(afs_int32 dumpID, struct budb_dumpEntry *dumpEntry,
+                 struct budb_tapeEntry *tapeEntry,
+                 struct budb_volumeEntry *volEntry)
 {
-    return( ubik_Call(BUDB_FindLastTape,udbHandle.uh_client,0,
-                     dumpID,dumpEntry,tapeEntry,volEntry) );
+    return (ubik_BUDB_FindLastTape(udbHandle.uh_client, 0, dumpID, dumpEntry,
+            tapeEntry, volEntry));
 }
 
-bcdb_MakeDumpAppended(appendedDumpID, initialDumpID, startTapeSeq)
-    afs_int32 appendedDumpID;
-    afs_int32 initialDumpID;
-    afs_int32 startTapeSeq;
+int
+bcdb_MakeDumpAppended(afs_int32 appendedDumpID, afs_int32 initialDumpID,
+                     afs_int32 startTapeSeq)
 {
-    return( ubik_Call(BUDB_MakeDumpAppended,udbHandle.uh_client,0,
-                     appendedDumpID,initialDumpID,startTapeSeq) );
+    return (ubik_BUDB_MakeDumpAppended(udbHandle.uh_client, 0, appendedDumpID,
+            initialDumpID, startTapeSeq));
 }
 
 
@@ -680,17 +648,16 @@ bcdb_MakeDumpAppended(appendedDumpID, initialDumpID, startTapeSeq)
  */
 
 afs_int32
-filesize(stream)
-     FILE *stream;
+filesize(FILE *stream)
 {
     afs_int32 offset;
     afs_int32 size;
 
     offset = ftell(stream);
-    fseek(stream, (afs_int32) 0, 2);           /* end of file */
+    fseek(stream, (afs_int32) 0, 2);   /* end of file */
     size = ftell(stream);
     fseek(stream, offset, 0);
-    return(size);
+    return (size);
 }
 
 
@@ -709,27 +676,31 @@ filesize(stream)
  *     n - fail
  */
 
-bc_LockText(ctPtr)
-     register udbClientTextP ctPtr;
+int
+bc_LockText(udbClientTextP ctPtr)
 {
     afs_int32 code;
     afs_int32 timeout, j = 0;
 
-    if (ctPtr->lockHandle != 0) return(1);     /* already locked */
+    if (ctPtr->lockHandle != 0)
+       return (1);             /* already locked */
 
-    timeout = ( (ctPtr->textSize == 0) ? 30 : ((ctPtr->textSize/50000) + 10) );
+    timeout =
+       ((ctPtr->textSize == 0) ? 30 : ((ctPtr->textSize / 50000) + 10));
 
     while (1) {
-       code = ubik_Call(BUDB_GetLock,  udbHandle.uh_client, 0,
-                        udbHandle.uh_instanceId, ctPtr->textType, timeout,
-                        &ctPtr->lockHandle);
-       if ( (code != BUDB_LOCKED) && (code != BUDB_SELFLOCKED) ) {
+       code =
+           ubik_BUDB_GetLock(udbHandle.uh_client, 0,
+                     udbHandle.uh_instanceId, ctPtr->textType, timeout,
+                     &ctPtr->lockHandle);
+       if ((code != BUDB_LOCKED) && (code != BUDB_SELFLOCKED)) {
            break;
        }
 
        /* Mention something every 30 seconds */
        if (++j >= 30) {
-           com_err(whoami, code,"; Waiting for db configuration text unlock");
+           afs_com_err(whoami, code,
+                   "; Waiting for db configuration text unlock");
            j = 0;
        }
 #ifdef AFS_PTHREAD_ENV
@@ -740,8 +711,9 @@ bc_LockText(ctPtr)
     }
 
     /* cleanup */
-    if (code) ctPtr->lockHandle = 0;
-    return(code);
+    if (code)
+       ctPtr->lockHandle = 0;
+    return (code);
 }
 
 /* bc_UnlockText
@@ -753,19 +725,20 @@ bc_LockText(ctPtr)
  *     n - fail
  */
 
-bc_UnlockText(ctPtr)
-     register udbClientTextP ctPtr;
+int
+bc_UnlockText(udbClientTextP ctPtr)
 {
     afs_int32 code = 0;
 
-    if ( ctPtr->lockHandle == 0 )
-       return(0);
+    if (ctPtr->lockHandle == 0)
+       return (0);
 
-    code = ubik_Call(BUDB_FreeLock, udbHandle.uh_client, 0, ctPtr->lockHandle);
+    code =
+       ubik_BUDB_FreeLock(udbHandle.uh_client, 0, ctPtr->lockHandle);
     ctPtr->lockHandle = 0;
-    
+
     /* Don't try to analyse the error. Let the lock timeout */
-    return(code);
+    return (code);
 }
 
 /* bc_CheckTextVersion
@@ -774,24 +747,25 @@ bc_UnlockText(ctPtr)
  *     n - out of date or error
  */
 
-bc_CheckTextVersion(ctPtr)
-     register udbClientTextP ctPtr;
+int
+bc_CheckTextVersion(udbClientTextP ctPtr)
 {
     afs_int32 code;
     afs_uint32 tversion;
 
-    if ( ctPtr->textVersion == -1 )
-       return(BC_VERSIONMISMATCH);
-
-    code = ubik_Call(BUDB_GetTextVersion,  udbHandle.uh_client, 0, 
-                    ctPtr->textType, &tversion);
-    if ( code )
-       return(code);
-    if ( tversion != ctPtr->textVersion )
-       return(BC_VERSIONMISMATCH);
-    return(0);
+    if (ctPtr->textVersion == -1)
+       return (BC_VERSIONMISMATCH);
+
+    code =
+       ubik_BUDB_GetTextVersion(udbHandle.uh_client, 0,
+                 ctPtr->textType, &tversion);
+    if (code)
+       return (code);
+    if (tversion != ctPtr->textVersion)
+       return (BC_VERSIONMISMATCH);
+    return (0);
 }
-     
+
 /* -------------------------------------
  * initialization routines
  * -------------------------------------
@@ -800,124 +774,115 @@ bc_CheckTextVersion(ctPtr)
 /* vldbClientInit 
  *      Initialize a client for the vl ubik database.
  */
-vldbClientInit(noAuthFlag, localauth, cellName, cstruct, ttoken)
-     int                noAuthFlag;
-     int                localauth;
-     char               *cellName;
-     struct ubik_client **cstruct;
-     struct ktc_token   *ttoken;
+int
+vldbClientInit(int noAuthFlag, int localauth, char *cellName, 
+              struct ubik_client **cstruct, 
+              time_t *expires)
 {
-    afs_int32                   code = 0;
-    struct afsconf_dir      *acdir;
-    struct rc_securityClass *sc;
-    afs_int32                   i, scIndex = 0;     /* Index of Rx security object - noauth */
-    struct afsconf_cell     info;
-    struct ktc_principal    sname;
-    struct rx_connection    *serverconns[VLDB_MAXSERVERS];
+    afs_int32 code = 0;
+    struct afsconf_dir *acdir;
+    struct rx_securityClass *sc;
+    afs_int32 i, scIndex = 0;  /* Index of Rx security object - noauth */
+    struct afsconf_cell info;
+    struct ktc_principal sname;
+    struct ktc_token *ttoken = NULL;
+    struct rx_connection *serverconns[VLDB_MAXSERVERS];
 
 
     /* Find out about the given cell */
-    acdir = afsconf_Open((localauth ? AFSDIR_SERVER_ETC_DIRPATH  :
-                         AFSDIR_CLIENT_ETC_DIRPATH));
-    if (!acdir) 
-    {
-       com_err(whoami,0,"Can't open configuration directory '%s'", 
-               (localauth ? AFSDIR_SERVER_ETC_DIRPATH  : 
+    acdir =
+       afsconf_Open((localauth ? AFSDIR_SERVER_ETC_DIRPATH :
+                     AFSDIR_CLIENT_ETC_DIRPATH));
+    if (!acdir) {
+       afs_com_err(whoami, 0, "Can't open configuration directory '%s'",
+               (localauth ? AFSDIR_SERVER_ETC_DIRPATH :
                 AFSDIR_CLIENT_ETC_DIRPATH));
        ERROR(BC_NOCELLCONFIG);
     }
 
-    if (!cellName[0])
-    {
-        char cname[64];
+    if (!cellName[0]) {
+       char cname[64];
 
-        code = afsconf_GetLocalCell(acdir, cname, sizeof(cname));
-       if (code) 
-       {
-           com_err(whoami, code, "; Can't get the local cell name - check %s/%s",  (localauth ? AFSDIR_SERVER_ETC_DIRPATH : AFSDIR_CLIENT_ETC_DIRPATH),
-                   AFSDIR_THISCELL_FILE);
+       code = afsconf_GetLocalCell(acdir, cname, sizeof(cname));
+       if (code) {
+           afs_com_err(whoami, code,
+                   "; Can't get the local cell name - check %s/%s",
+                   (localauth ? AFSDIR_SERVER_ETC_DIRPATH :
+                    AFSDIR_CLIENT_ETC_DIRPATH), AFSDIR_THISCELL_FILE);
            ERROR(code);
        }
        strcpy(cellName, cname);
     }
 
     code = afsconf_GetCellInfo(acdir, cellName, AFSCONF_VLDBSERVICE, &info);
-    if (code) 
-    {
-      com_err(whoami, code, "; Can't find cell %s's hosts in %s/%s", 
-             cellName, (localauth ? AFSDIR_SERVER_ETC_DIRPATH  :
-                        AFSDIR_CLIENT_ETC_DIRPATH),
-             AFSDIR_CELLSERVDB_FILE);
-      ERROR(BC_NOCELLCONFIG);
+    if (code) {
+       afs_com_err(whoami, code, "; Can't find cell %s's hosts in %s/%s",
+               cellName,
+               (localauth ? AFSDIR_SERVER_ETC_DIRPATH :
+                AFSDIR_CLIENT_ETC_DIRPATH), AFSDIR_CELLSERVDB_FILE);
+       ERROR(BC_NOCELLCONFIG);
     }
 
     /*
      * Grab tickets if we care about authentication.
      */
-    ttoken->endTime = 0;
-    if (localauth)
-    {
-        code = afsconf_GetLatestKey (acdir, 0, 0);
-       if (code)
-       {
-           com_err(whoami, code, "; Can't get key from local key file");
+    *expires = 0;
+    if (localauth) {
+       code = afsconf_GetLatestKey(acdir, 0, 0);
+       if (code) {
+           afs_com_err(whoami, code, "; Can't get key from local key file");
            ERROR(code);
-       }
-       else
-       {
+       } else {
            code = afsconf_ClientAuth(acdir, &sc, &scIndex);
-           if (code) 
-           {
-               com_err(whoami, code, "; Calling ClientAuth");
+           if (code) {
+               afs_com_err(whoami, code, "; Calling ClientAuth");
                ERROR(code);
            }
 
-           ttoken->endTime = NEVERDATE;
+           *expires = NEVERDATE;
        }
-    }
-    else
-    {
-       if (!noAuthFlag)
-       {
+    } else {
+       if (!noAuthFlag) {
            strcpy(sname.cell, info.name);
            sname.instance[0] = 0;
            strcpy(sname.name, "afs");
 
-           code = ktc_GetToken(&sname, ttoken, sizeof(struct ktc_token), NULL);
-           if (code) 
-           {
-               com_err(whoami,code, 0,"; Can't get AFS tokens - running unauthenticated");
-           }
-           else 
-           {
-               if ( (ttoken->kvno < 0) || (ttoken->kvno > 255) )
-                 com_err(whoami, 0, "Funny kvno (%d) in ticket, proceeding", ttoken->kvno);
-               
+           code =
+               ktc_GetToken(&sname, ttoken, sizeof(struct ktc_token), NULL);
+           if (code) {
+               afs_com_err(whoami, code,
+                           "; Can't get AFS tokens - running unauthenticated");
+           } else {
+               if ((ttoken->kvno < 0) || (ttoken->kvno > 256))
+                   afs_com_err(whoami, 0,
+                           "Funny kvno (%d) in ticket, proceeding",
+                           ttoken->kvno);
+               *expires = ttoken->endTime;
                scIndex = 2;
            }
        }
-    
-       switch (scIndex) 
-       {
-           case 0:
-             sc = rxnull_NewClientSecurityObject();
-             break;
-           case 2:
-             sc = (struct rx_securityClass *) 
-               rxkad_NewClientSecurityObject(rxkad_clear, &ttoken->sessionKey, 
-                                             ttoken->kvno, ttoken->ticketLen, 
+
+       switch (scIndex) {
+       case 0:
+           sc = rxnull_NewClientSecurityObject();
+           break;
+       case 2:
+           sc = (struct rx_securityClass *)
+               rxkad_NewClientSecurityObject(rxkad_clear,
+                                             &ttoken->sessionKey,
+                                             ttoken->kvno, ttoken->ticketLen,
                                              ttoken->ticket);
-             break;
-           default:
-             com_err(whoami, 0, "Unsupported authentication type %d", scIndex);
-             ERROR(-1);
-             break;
+           break;
+       default:
+           afs_com_err(whoami, 0, "Unsupported authentication type %d", scIndex);
+           ERROR(-1);
+           break;
        }
     }
 
-    if (!sc)
-    {
-        com_err(whoami, 0, "Can't create a security object with security index %d",
+    if (!sc) {
+       afs_com_err(whoami, 0,
+               "Can't create a security object with security index %d",
                scIndex);
        ERROR(-1);
     }
@@ -925,31 +890,31 @@ vldbClientInit(noAuthFlag, localauth, cellName, cstruct, ttoken)
     /* tell UV module about default authentication */
     UV_SetSecurity(sc, scIndex);
 
-    if (info.numServers > VLDB_MAXSERVERS) 
-    {
-        com_err(whoami, 0, 
-               "Warning: %d VLDB servers exist for cell '%s', can only remember the first %d", 
+    if (info.numServers > VLDB_MAXSERVERS) {
+       afs_com_err(whoami, 0,
+               "Warning: %d VLDB servers exist for cell '%s', can only remember the first %d",
                info.numServers, cellName, VLDB_MAXSERVERS);
        info.numServers = VLDB_MAXSERVERS;
     }
 
-    for (i=0; i<info.numServers; i++) 
-       serverconns[i] = rx_NewConnection(info.hostAddr[i].sin_addr.s_addr, 
-                                         info.hostAddr[i].sin_port, 
-                                         USER_SERVICE_ID, sc, scIndex);
+    for (i = 0; i < info.numServers; i++)
+       serverconns[i] =
+           rx_NewConnection(info.hostAddr[i].sin_addr.s_addr,
+                            info.hostAddr[i].sin_port, USER_SERVICE_ID, sc,
+                            scIndex);
     serverconns[i] = 0;
 
     *cstruct = 0;
     code = ubik_ClientInit(serverconns, cstruct);
-    if (code) 
-    {
-       com_err(whoami,code,"; Can't initialize ubik connection to vldb");
+    if (code) {
+       afs_com_err(whoami, code, "; Can't initialize ubik connection to vldb");
        ERROR(code);
     }
 
-error_exit:
-    if (acdir) afsconf_Close(acdir);
-    return(code);
+  error_exit:
+    if (acdir)
+       afsconf_Close(acdir);
+    return (code);
 }
 
 /* udbClientInit
@@ -957,169 +922,160 @@ error_exit:
  */
 
 afs_int32
-udbClientInit(noAuthFlag, localauth, cellName)
-     int  noAuthFlag;
-     int  localauth;
-     char *cellName;
+udbClientInit(int noAuthFlag, int localauth, char *cellName)
 {
-    afs_int32 serverList[MAXSERVERS];
-    char hostname[256];
     struct ktc_principal principal;
     struct ktc_token token;
     struct afsconf_cell info;
     struct afsconf_dir *acdir;
     int i;
-    afs_int32  code = 0;
-    
-    acdir = afsconf_Open((localauth ? AFSDIR_SERVER_ETC_DIRPATH  :
-                         AFSDIR_CLIENT_ETC_DIRPATH));   
-    if (!acdir) 
-    {
-      com_err(whoami,0,"Can't open configuration directory '%s'", 
-             (localauth ? AFSDIR_SERVER_ETC_DIRPATH  :
-              AFSDIR_CLIENT_ETC_DIRPATH));
-      ERROR(BC_NOCELLCONFIG);
+    afs_int32 code = 0;
+
+    acdir =
+       afsconf_Open((localauth ? AFSDIR_SERVER_ETC_DIRPATH :
+                     AFSDIR_CLIENT_ETC_DIRPATH));
+    if (!acdir) {
+       afs_com_err(whoami, 0, "Can't open configuration directory '%s'",
+               (localauth ? AFSDIR_SERVER_ETC_DIRPATH :
+                AFSDIR_CLIENT_ETC_DIRPATH));
+       ERROR(BC_NOCELLCONFIG);
     }
-    
-    if (!cellName[0])
-    {
-        char cname[64];
-
-        code = afsconf_GetLocalCell(acdir, cname, sizeof(cname));
-       if (code) 
-       {
-         com_err(whoami, code, "; Can't get the local cell name - check %s/%s",    (localauth ? AFSDIR_SERVER_ETC_DIRPATH : AFSDIR_CLIENT_ETC_DIRPATH),
-                 AFSDIR_THISCELL_FILE);
-         ERROR(code);
+
+    if (!cellName[0]) {
+       char cname[64];
+
+       code = afsconf_GetLocalCell(acdir, cname, sizeof(cname));
+       if (code) {
+           afs_com_err(whoami, code,
+                   "; Can't get the local cell name - check %s/%s",
+                   (localauth ? AFSDIR_SERVER_ETC_DIRPATH :
+                    AFSDIR_CLIENT_ETC_DIRPATH), AFSDIR_THISCELL_FILE);
+           ERROR(code);
        }
        strcpy(cellName, cname);
     }
 
     code = afsconf_GetCellInfo(acdir, cellName, 0, &info);
-    if (code) 
-    {
-      com_err(whoami, code, "; Can't find cell %s's hosts in %s/%s", 
-             cellName, (localauth ? AFSDIR_SERVER_ETC_DIRPATH :
-                        AFSDIR_CLIENT_ETC_DIRPATH),
-             AFSDIR_CELLSERVDB_FILE);
+    if (code) {
+       afs_com_err(whoami, code, "; Can't find cell %s's hosts in %s/%s",
+               cellName,
+               (localauth ? AFSDIR_SERVER_ETC_DIRPATH :
+                AFSDIR_CLIENT_ETC_DIRPATH), AFSDIR_CELLSERVDB_FILE);
        ERROR(BC_NOCELLCONFIG);
     }
-       
+
     udbHandle.uh_scIndex = RX_SCINDEX_NULL;
 
-    if (localauth)
-    {
-        code = afsconf_GetLatestKey (acdir, 0, 0);
-       if (code)
-       {
-           com_err(whoami, code, "; Can't get key from local key file");
+    if (localauth) {
+       code = afsconf_GetLatestKey(acdir, 0, 0);
+       if (code) {
+           afs_com_err(whoami, code, "; Can't get key from local key file");
            ERROR(-1);
-       }
-       else
-       {
-           code = afsconf_ClientAuth(acdir, &udbHandle.uh_secobj, &udbHandle.uh_scIndex);
-           if (code) 
-           {
-               com_err(whoami, code, "; Calling ClientAuth");
+       } else {
+           code =
+               afsconf_ClientAuth(acdir, &udbHandle.uh_secobj,
+                                  &udbHandle.uh_scIndex);
+           if (code) {
+               afs_com_err(whoami, code, "; Calling ClientAuth");
                ERROR(-1);
            }
        }
-    }
-    else
-    {
-        if (!noAuthFlag)
-       {
+    } else {
+       if (!noAuthFlag) {
            /* setup principal */
            strcpy(principal.cell, info.name);
            principal.instance[0] = 0;
            strcpy(principal.name, "afs");
-       
+
            /* get token */
            code = ktc_GetToken(&principal, &token, sizeof(token), NULL);
-           if (code)
-           {
-               com_err(whoami, code, "; Can't get tokens - running unauthenticated");
-           }
-           else
-           {
-               if ((token.kvno < 0) || (token.kvno > 255))
-                   com_err(whoami, 0, "Unexpected kvno (%d) in ticket - proceeding",
+           if (code) {
+               afs_com_err(whoami, code,
+                       "; Can't get tokens - running unauthenticated");
+           } else {
+               if ((token.kvno < 0) || (token.kvno > 256))
+                   afs_com_err(whoami, 0,
+                           "Unexpected kvno (%d) in ticket - proceeding",
                            token.kvno);
-               udbHandle.uh_scIndex = RX_SCINDEX_KAD;          /* Kerberos */
+               udbHandle.uh_scIndex = RX_SCINDEX_KAD;  /* Kerberos */
            }
        }
 
-       switch (udbHandle.uh_scIndex)
-       {
-           case 0:
-             udbHandle.uh_secobj = rxnull_NewClientSecurityObject();
-             break;
-       
-           case 2:
-             udbHandle.uh_secobj = (struct rx_securityClass *)
+       switch (udbHandle.uh_scIndex) {
+       case 0:
+           udbHandle.uh_secobj = rxnull_NewClientSecurityObject();
+           break;
+
+       case 2:
+           udbHandle.uh_secobj = (struct rx_securityClass *)
                rxkad_NewClientSecurityObject(rxkad_clear, &token.sessionKey,
                                              token.kvno, token.ticketLen,
                                              token.ticket);
-             break;
+           break;
 
-           default:
-             com_err(whoami, 0, "Unsupported authentication type %d", udbHandle.uh_scIndex );
-             ERROR(-1);
-             break;
+       default:
+           afs_com_err(whoami, 0, "Unsupported authentication type %d",
+                   udbHandle.uh_scIndex);
+           ERROR(-1);
+           break;
        }
     }
 
-    if (!udbHandle.uh_secobj)
-    {  
-        com_err(whoami, 0, "Can't create a security object with security index %d",
-               udbHandle.uh_secobj);
+    if (!udbHandle.uh_secobj) {
+       afs_com_err(whoami, 0,
+               "Can't create a security object with security index %d",
+               udbHandle.uh_scIndex);
        ERROR(-1);
     }
 
-    if (info.numServers > MAXSERVERS)
-    {
-       com_err(whoami,0,
-               "Warning: %d BDB servers exist for cell '%s', can only remember the first %d", 
+    if (info.numServers > MAXSERVERS) {
+       afs_com_err(whoami, 0,
+               "Warning: %d BDB servers exist for cell '%s', can only remember the first %d",
                info.numServers, cellName, MAXSERVERS);
        info.numServers = MAXSERVERS;
     }
 
     /* establish connections to the servers. Check for failed connections? */
     for (i = 0; i < info.numServers; i++) {
-        udbHandle.uh_serverConn[i] = rx_NewConnection(info.hostAddr[i].sin_addr.s_addr,
-                                                  htons(AFSCONF_BUDBPORT),
-                                                  BUDB_SERVICE, 
-                                                  udbHandle.uh_secobj, 
-                                                  udbHandle.uh_scIndex);
+       udbHandle.uh_serverConn[i] =
+           rx_NewConnection(info.hostAddr[i].sin_addr.s_addr,
+                            htons(AFSCONF_BUDBPORT), BUDB_SERVICE,
+                            udbHandle.uh_secobj, udbHandle.uh_scIndex);
     }
     udbHandle.uh_serverConn[i] = 0;
 
     code = ubik_ClientInit(udbHandle.uh_serverConn, &udbHandle.uh_client);
     if (code) {
-       com_err(whoami,code,"; Can't initialize ubik connection to backup database");
+       afs_com_err(whoami, code,
+               "; Can't initialize ubik connection to backup database");
        ERROR(code);
     }
 
     /* Try to quickly find a good site by setting deadtime low */
     for (i = 0; i < info.numServers; i++)
        rx_SetConnDeadTime(udbHandle.uh_client->conns[i], 1);
-    code = ubik_Call(BUDB_GetInstanceId, udbHandle.uh_client, 0, &udbHandle.uh_instanceId);
+    code =
+       ubik_BUDB_GetInstanceId(udbHandle.uh_client, 0,
+                 &udbHandle.uh_instanceId);
 
     /* Reset dead time back up to default */
     for (i = 0; i < info.numServers; i++)
-       rx_SetConnDeadTime(udbHandle.uh_client->conns[i], 60);
+       rx_SetConnDeadTime(udbHandle.uh_client->conns[i], 60);
 
     /* If did not find a site on first quick pass, try again */
     if (code == -1)
-       code = ubik_Call(BUDB_GetInstanceId, udbHandle.uh_client, 0, &udbHandle.uh_instanceId);
+       code =
+           ubik_BUDB_GetInstanceId(udbHandle.uh_client, 0,
+                     &udbHandle.uh_instanceId);
     if (code) {
-       com_err(whoami,code,"; Can't access backup database");
+       afs_com_err(whoami, code, "; Can't access backup database");
        ERROR(code);
     }
 
-error_exit:
-    if (acdir) afsconf_Close(acdir);
-    return(code);
+  error_exit:
+    if (acdir)
+       afsconf_Close(acdir);
+    return (code);
 }
 
 /* -------------------------------------
@@ -1140,12 +1096,11 @@ error_exit:
  *     4) if the vanilla ubik_Call is ever modified, the END_SINGLESERVER
  *        flag can be discarded. The first call without SINGLESERVER set
  *        can clean up the state.
- */ 
+ */
 
-struct ubikCallState
-{
-    afs_int32 ucs_flags;                       /* state flags */
-    afs_int32 ucs_selectedServer;              /* which server selected */
+struct ubikCallState {
+    afs_int32 ucs_flags;       /* state flags */
+    afs_int32 ucs_selectedServer;      /* which server selected */
 };
 
 static struct ubikCallState uServer;
@@ -1161,11 +1116,11 @@ static struct ubikCallState uServer;
  */
 
 afs_int32
-ubik_Call_SingleServer(aproc, aclient, aflags, p1, p2, p3, p4, p5, p6, p7, p8, p9, p10, p11, p12, p13, p14, p15, p16)
-    register struct ubik_client *aclient;
-    int (*aproc)();
-    afs_int32 aflags;
-    char *p1, *p2, *p3, *p4, *p5, *p6, *p7, *p8, *p9, *p10, *p11, *p12, *p13, *p14, *p15, *p16;
+ubik_Call_SingleServer(int (*aproc) (), struct ubik_client *aclient, 
+                      afs_int32 aflags, char *p1, char *p2, char *p3, 
+                      char *p4, char *p5, char *p6, char *p7, char *p8,
+                      char *p9, char *p10, char *p11, char *p12, char *p13, 
+                      char *p14, char *p15, char *p16)
 {
     register afs_int32 code;
     afs_int32 someCode, newHost, thisHost;
@@ -1176,30 +1131,25 @@ ubik_Call_SingleServer(aproc, aclient, aflags, p1, p2, p3, p4, p5, p6, p7, p8, p
     struct rx_connection *tc;
     struct rx_peer *rxp;
 
-    if ( (aflags & (UF_SINGLESERVER | UF_END_SINGLESERVER)) != 0 )
-    {
-       if ( ((aflags & UF_SINGLESERVER) != 0 )
-       &&   ((uServer.ucs_flags & UF_SINGLESERVER ) != 0)
-          )
-       {
-           
+    if ((aflags & (UF_SINGLESERVER | UF_END_SINGLESERVER)) != 0) {
+       if (((aflags & UF_SINGLESERVER) != 0)
+           && ((uServer.ucs_flags & UF_SINGLESERVER) != 0)
+           ) {
+
            /* have a selected server */
            tc = aclient->conns[uServer.ucs_selectedServer];
-           
-           code = (*aproc)(tc, p1, p2, p3, p4, p5, p6, p7, p8, p9, p10, p11,
-                           p12, p13, p14, p15, p16);
-           if ( code )
-           {
+
+           code =
+               (*aproc) (tc, p1, p2, p3, p4, p5, p6, p7, p8, p9, p10, p11,
+                         p12, p13, p14, p15, p16);
+           if (code) {
                /* error. Clean up single server state */
                memset(&uServer, 0, sizeof(uServer));
            }
-           return(code);
-       }
-       else
-       if ( (aflags & UF_END_SINGLESERVER) != 0 )
-       {
+           return (code);
+       } else if ((aflags & UF_END_SINGLESERVER) != 0) {
            memset(&uServer, 0, sizeof(uServer));
-           return(0);
+           return (0);
        }
     }
 
@@ -1207,26 +1157,26 @@ ubik_Call_SingleServer(aproc, aclient, aflags, p1, p2, p3, p4, p5, p6, p7, p8, p
     chaseCount = 0;
     pass = 0;
     count = 0;
-    while(1)
-    { /*w*/
+    while (1) {                        /*w */
 
        /* tc is the next conn to try */
        tc = aclient->conns[count];
-       if ( tc == 0 )
-       {
-           if (pass == 0) {
+       if (tc == 0) {
+           if (pass == 0) {
                pass = 1;       /* in pass 1, we look at down hosts, too */
-               count = 0;
+               count = 0;
                continue;
-           }
-           else break;                 /* nothing left to try */
+           } else
+               break;          /* nothing left to try */
        }
        if (pass == 0 && (aclient->states[count] & CFLastFailed)) {
            count++;
-           continue;   /* this guy's down, try someone else first */
+           continue;           /* this guy's down, try someone else first */
        }
 
-       code = (*aproc)(tc, p1, p2, p3, p4, p5, p6, p7, p8, p9, p10, p11, p12, p13, p14, p15, p16);
+       code =
+           (*aproc) (tc, p1, p2, p3, p4, p5, p6, p7, p8, p9, p10, p11, p12,
+                     p13, p14, p15, p16);
 
        /* note that getting a UNOTSYNC error code back does *not* guarantee
         * that there is a sync site yet elected.  However, if there is a
@@ -1234,53 +1184,47 @@ ubik_Call_SingleServer(aproc, aclient, aflags, p1, p2, p3, p4, p5, p6, p7, p8, p
         * requires a sync site, ubik will return UNOTSYNC, indicating the
         * operation won't work until you find a sync site
         */
-       if (code == UNOTSYNC) 
-       { /*ns*/
+       if (code == UNOTSYNC) { /*ns */
            /* means that this requires a sync site to work */
            someCode = code;    /* remember an error, if this fails */
 
            /* now see if we can find the sync site host */
            code = VOTE_GetSyncSite(tc, &newHost);
            if (code == 0 && newHost != 0) {
-               newHost = htonl(newHost); /* convert back to network order */
+               newHost = htonl(newHost);       /* convert back to network order */
 
                /* position count at the appropriate slot in the client
                 * structure and retry. If we can't find in slot, we'll just
                 * continue through the whole list
-                */
-               for(i=0;i<MAXSERVERS;i++)
-               { /*f*/
+                */
+               for (i = 0; i < MAXSERVERS; i++) {      /*f */
                    rxp = rx_PeerOf(aclient->conns[i]);
                    if (!(thisHost = rx_HostOf(rxp))) {
-                       count++;    /* host not found, try the next dude */
+                       count++;        /* host not found, try the next dude */
                        break;
                    }
-                   if (thisHost == newHost) 
-                   {   
+                   if (thisHost == newHost) {
                        /* avoid asking in a loop */
-                       if (chaseCount++ > 2) 
-                           break; 
-                       count = i;  /* we were told to use this one */
+                       if (chaseCount++ > 2)
+                           break;
+                       count = i;      /* we were told to use this one */
                        break;
                    }
-               } /*f*/
-           }
-           else count++;   /* not directed, keep looking for a sync site */
+               }               /*f */
+           } else
+               count++;        /* not directed, keep looking for a sync site */
            continue;
-       } /*ns*/
-       else if (code == UNOQUORUM) {   /* this guy is still recovering */
+       } /*ns */
+       else if (code == UNOQUORUM) {   /* this guy is still recovering */
            someCode = code;
            count++;
            continue;
-       }
-       else if (code < 0) {    /* network errors */
+       } else if (code < 0) {  /* network errors */
            someCode = code;
            aclient->states[count] |= CFLastFailed;
            count++;
            continue;
-       }
-       else
-       {
+       } else {
            /* ok, operation worked */
            aclient->states[count] &= ~CFLastFailed;
            /* either misc ubik code, or misc application code (incl success)
@@ -1289,22 +1233,21 @@ ubik_Call_SingleServer(aproc, aclient, aflags, p1, p2, p3, p4, p5, p6, p7, p8, p
            /* if the call succeeded, setup connection state for subsequent
             * calls
             */
-           if ( (code == 0)
-           &&   (aflags & UF_SINGLESERVER != 0)
-              )
-           {
+           if ((code == 0)
+               && ((aflags & UF_SINGLESERVER) != 0)
+               ) {
                /* need to save state */
                uServer.ucs_flags = UF_SINGLESERVER;
                uServer.ucs_selectedServer = count;
            }
-              
+
            return code;
-       } 
-    } /*w*/
+       }
+    }                          /*w */
     return someCode;
 }
 
-     
+
 /* -------------------------------------
  * debug and test routines 
  * -------------------------------------
@@ -1318,7 +1261,8 @@ ubik_Call_SingleServer(aproc, aclient, aflags, p1, p2, p3, p4, p5, p6, p7, p8, p
  *     n - error.
  */
 
-udbLocalInit()
+int
+udbLocalInit(void)
 {
     afs_int32 serverList[MAXSERVERS];
     char hostname[256];
@@ -1329,52 +1273,48 @@ udbLocalInit()
     /* get our host name */
     gethostname(hostname, sizeof(hostname));
     /* strcpy(hostname, "hops"); */
-    
+
     args[0] = "";
     args[1] = "-servers";
     args[2] = hostname;
 
     code = ubik_ParseClientList(3, args, serverList);
-    if ( code )
-    {
-       com_err(whoami, code, "; udbLocalInit: parsing ubik server list");
-       return(-1);
+    if (code) {
+       afs_com_err(whoami, code, "; udbLocalInit: parsing ubik server list");
+       return (-1);
     }
 
     udbHandle.uh_scIndex = RX_SCINDEX_NULL;
-    udbHandle.uh_secobj = (struct rx_securityClass *) 
-                                       rxnull_NewClientSecurityObject();
-
-    for ( i = 0; serverList[i] != 0; i++ )
-    {
-       udbHandle.uh_serverConn[i] = rx_NewConnection (serverList[i], 
-                                                   htons(AFSCONF_BUDBPORT),
-                                                   BUDB_SERVICE,
-                                                   udbHandle.uh_secobj,
-                                                   udbHandle.uh_scIndex);
-       if ( udbHandle.uh_serverConn[i] == 0 )
-       {
-           com_err(whoami, 0, "connection %d failed", i);
+    udbHandle.uh_secobj = (struct rx_securityClass *)
+       rxnull_NewClientSecurityObject();
+
+    for (i = 0; serverList[i] != 0; i++) {
+       udbHandle.uh_serverConn[i] =
+           rx_NewConnection(serverList[i], htons(AFSCONF_BUDBPORT),
+                            BUDB_SERVICE, udbHandle.uh_secobj,
+                            udbHandle.uh_scIndex);
+       if (udbHandle.uh_serverConn[i] == 0) {
+           afs_com_err(whoami, 0, "connection %d failed", i);
            continue;
        }
     }
     udbHandle.uh_serverConn[i] = 0;
     code = ubik_ClientInit(udbHandle.uh_serverConn, &udbHandle.uh_client);
-    if ( code )
-    {
-       com_err(whoami, code, "; in ubik_ClientInit");
-       return(code);
+    if (code) {
+       afs_com_err(whoami, code, "; in ubik_ClientInit");
+       return (code);
     }
 
-    code = ubik_Call(BUDB_GetInstanceId,  udbHandle.uh_client, 0, &udbHandle.uh_instanceId);
-    if ( code )
-    {
-       com_err(whoami, code, "; Can't estblish instance Id");
-       return(code);
+    code =
+       ubik_BUDB_GetInstanceId(udbHandle.uh_client, 0,
+                 &udbHandle.uh_instanceId);
+    if (code) {
+       afs_com_err(whoami, code, "; Can't estblish instance Id");
+       return (code);
     }
 
-abort:
-    return(0);
+   /* abort: */
+    return (0);
 }
 
 /* bc_openTextFile - This function opens a temp file to read in the
@@ -1391,42 +1331,42 @@ abort:
  *     !0: error code
  *     0: Success.
  */
-int bc_openTextFile(udbClientTextP ctPtr, char *tmpFileName)
+int
+bc_openTextFile(udbClientTextP ctPtr, char *tmpFileName)
 {
-  int code = 0;
+    int code = 0;
 
-  if ( ctPtr->textStream != NULL )
-    fclose(ctPtr->textStream);
-  
-  sprintf(tmpFileName, "%s/bu_XXXXXX", gettmpdir());
+    if (ctPtr->textStream != NULL)
+       fclose(ctPtr->textStream);
+
+    sprintf(tmpFileName, "%s/bu_XXXXXX", gettmpdir());
 #ifdef AFS_LINUX20_ENV
-  mkstemp(tmpFileName);
+    mkstemp(tmpFileName);
 #else
-  mktemp(tmpFileName);
+    mktemp(tmpFileName);
 #endif
-  ctPtr->textStream = fopen(tmpFileName, "w+");
-  if ( ctPtr->textStream == NULL )
-    ERROR(BUDB_INTERNALERROR);
-  
-#ifndef AFS_NT40_ENV /* This can't be done on NT */
-  /* make the file invisible to others */
-  code = unlink(tmpFileName);
-  if ( code )
-    ERROR(errno);
+    ctPtr->textStream = fopen(tmpFileName, "w+");
+    if (ctPtr->textStream == NULL)
+       ERROR(BUDB_INTERNALERROR);
+
+#ifndef AFS_NT40_ENV           /* This can't be done on NT */
+    /* make the file invisible to others */
+    code = unlink(tmpFileName);
+    if (code)
+       ERROR(errno);
 #endif
-  dprintf(("file is %s\n", tmpFileName));
 
-normal_exit:
-  return code;
+    afs_dprintf(("file is %s\n", tmpFileName));
 
-error_exit:
-  if ( ctPtr->textStream != NULL )
-    {
-      fclose(ctPtr->textStream);
-      ctPtr->textStream = NULL;
+  normal_exit:
+    return code;
+
+  error_exit:
+    if (ctPtr->textStream != NULL) {
+       fclose(ctPtr->textStream);
+       ctPtr->textStream = NULL;
     }
-  goto normal_exit;
+    goto normal_exit;
 }
 
 
@@ -1438,17 +1378,18 @@ error_exit:
  *    0  - Success
  *    !0 - error code
  */
-int bc_closeTextFile(udbClientTextP ctPtr, char *tmpFileName)
+int
+bc_closeTextFile(udbClientTextP ctPtr, char *tmpFileName)
 {
-  int code = 0;
+    int code = 0;
 
-  if (ctPtr->textStream)
-    fclose(ctPtr->textStream);
+    if (ctPtr->textStream)
+       fclose(ctPtr->textStream);
 
-  if (*tmpFileName) { /* we have a valid name */
-    code = unlink(tmpFileName);
-    if (code < 0)
-      code = errno;
-  }
-  return code;
+    if (*tmpFileName) {                /* we have a valid name */
+       code = unlink(tmpFileName);
+       if (code < 0)
+           code = errno;
+    }
+    return code;
 }