bcdb-listdumps-argument-fix-20090122
[openafs.git] / src / bucoord / ubik_db_if.c
index ae5ec21..8e72148 100644 (file)
@@ -1,9 +1,20 @@
-/* Copyright (C) 1998 Transarc Corporation - All rights reserved */
+/*
+ * 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
+ */
 
 /* Interface and supporting routines for the backup system's ubik database */
 
-#include <afs/param.h>
-#include <stdio.h>
+#include <afsconfig.h>
+#include <afs/stds.h>
+
+RCSID
+    ("$Header$");
+
 #include <sys/types.h>
 #include <fcntl.h>
 #ifdef AFS_NT40_ENV
@@ -11,7 +22,7 @@
 #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 <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"
 
+/* protos */
+afs_int32 bcdb_AddVolume(register struct budb_volumeEntry *);
+afs_int32 bcdb_AddVolumes(register struct budb_volumeEntry *, afs_int32 );
+afs_int32 bcdb_CreateDump(register struct budb_dumpEntry *) ;
+afs_int32 bcdb_deleteDump(afs_int32, afs_int32, afs_int32, budb_dumpsList *);
+/*note the pinter to the function comes from ubik/ubikclient ubik_Call function.*/
+afs_int32 bcdb_listDumps (afs_int32, afs_int32,afs_int32,afs_int32, budb_dumpsList *,
+ budb_dumpsList *);
+afs_int32 bcdb_DeleteVDP(char *, char *, afs_int32 );
+afs_int32 bcdb_FindClone(afs_int32, char *, afs_int32 *);
+
 extern char *whoami;
 
 /* -------------------------------------
@@ -40,64 +63,55 @@ 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;
 
@@ -109,24 +123,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
@@ -143,15 +159,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
@@ -177,8 +191,10 @@ bcdb_FindDump(volumeName, beforeDate, deptr)
      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
@@ -192,8 +208,8 @@ bcdb_FindDumpByID(dumpID, deptr)
      register 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 */
@@ -201,40 +217,33 @@ 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;
     }
 
-    bcopy(dl.budb_dumpList_val, deptr, sizeof(*deptr));
+    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:
-    bzero(deptr, sizeof(*deptr));
+  error:
+    memset(deptr, 0, sizeof(*deptr));
     goto exit;
 }
 
@@ -263,11 +272,11 @@ bcdb_FindLastVolClone(volSetName, dumpName, volName, clonetime)
      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
@@ -293,9 +302,10 @@ bcdb_FindLatestDump(volSetName, dumpPath, deptr)
      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);
 }
 
 
@@ -316,26 +326,27 @@ bcdb_FindTape(dumpid, tapeName, teptr)
     afs_int32 dbTime;
     afs_int32 code = 0;
 
-    bzero(teptr, sizeof(*teptr));
+    memset(teptr, 0, sizeof(*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 */
 
-    bcopy(tl.budb_tapeList_val, teptr, sizeof(*teptr));
+    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)
@@ -348,25 +359,26 @@ bcdb_FindTapeSeq(dumpid, tapeSeq, teptr)
     afs_int32 dbTime;
     afs_int32 code = 0;
 
-    bzero(teptr, sizeof(*teptr));
+    memset(teptr, 0, sizeof(*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 */
 
-    bcopy(tl.budb_tapeList_val, teptr, sizeof(*teptr));
+    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
@@ -400,37 +412,33 @@ 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;
 {
-    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;
 {
-    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
@@ -453,23 +461,18 @@ 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);
 }
 
 
@@ -477,9 +480,9 @@ bcdb_UseTape(teptr, newFlag)
      register 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);
 }
 
 
@@ -489,7 +492,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.
@@ -500,7 +503,7 @@ bcdb_UseTape(teptr, newFlag)
 bcdb_GetTextFile(register udbClientTextP ctPtr)
 {
     afs_int32 bufferSize;
-    afs_int32 offset, nextOffset, chunkSize; 
+    afs_int32 offset, nextOffset;
     charListT charList;
     afs_int32 code = 0;
 
@@ -509,64 +512,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,14 +590,14 @@ bcdb_SaveTextFile(ctPtr)
 
     /* 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);
 
@@ -602,64 +606,65 @@ bcdb_SaveTextFile(ctPtr)
     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;
+     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;
+     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));
 }
 
 
@@ -676,10 +681,10 @@ filesize(stream)
     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);
 }
 
 
@@ -704,21 +709,25 @@ bc_LockText(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
@@ -729,8 +738,9 @@ bc_LockText(ctPtr)
     }
 
     /* cleanup */
-    if (code) ctPtr->lockHandle = 0;
-    return(code);
+    if (code)
+       ctPtr->lockHandle = 0;
+    return (code);
 }
 
 /* bc_UnlockText
@@ -747,14 +757,15 @@ bc_UnlockText(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
@@ -769,18 +780,19 @@ bc_CheckTextVersion(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
  * -------------------------------------
@@ -790,123 +802,115 @@ bc_CheckTextVersion(ctPtr)
  *      Initialize a client for the vl ubik database.
  */
 vldbClientInit(noAuthFlag, localauth, cellName, cstruct, ttoken)
-     int                noAuthFlag;
-     int                localauth;
-     char               *cellName;
+     int noAuthFlag;
+     int localauth;
+     char *cellName;
      struct ubik_client **cstruct;
-     struct ktc_token   *ttoken;
+     struct ktc_token *ttoken;
 {
-    afs_int32                   code = 0;
-    struct afsconf_dir      *acdir;
+    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 i, scIndex = 0;  /* Index of Rx security object - noauth */
+    struct afsconf_cell info;
+    struct ktc_principal sname;
+    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");
+    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;
        }
-    }
-    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), (char *)0);
-           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, 0,
+                       "; Can't get AFS tokens - running unauthenticated");
+           } else {
+               if ((ttoken->kvno < 0) || (ttoken->kvno > 255))
+                   afs_com_err(whoami, 0,
+                           "Funny kvno (%d) in ticket, proceeding",
+                           ttoken->kvno);
+
                scIndex = 2;
            }
        }
-    
-       switch (scIndex) 
-       {
-           case 0:
-             sc = (struct rx_securityClass *) 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);
     }
@@ -914,31 +918,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
@@ -947,168 +951,162 @@ error_exit:
 
 afs_int32
 udbClientInit(noAuthFlag, localauth, cellName)
-     int  noAuthFlag;
-     int  localauth;
+     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), (char *)0);
-           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",
+           code = ktc_GetToken(&principal, &token, sizeof(token), NULL);
+           if (code) {
+               afs_com_err(whoami, code,
+                       "; Can't get tokens - running unauthenticated");
+           } else {
+               if ((token.kvno < 0) || (token.kvno > 255))
+                   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 = (struct rx_securityClass *) 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",
+    if (!udbHandle.uh_secobj) {
+       afs_com_err(whoami, 0,
+               "Can't create a security object with security index %d",
                udbHandle.uh_secobj);
        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);
 }
 
 /* -------------------------------------
@@ -1129,12 +1127,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;
@@ -1150,11 +1147,13 @@ 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(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;
 {
     register afs_int32 code;
     afs_int32 someCode, newHost, thisHost;
@@ -1165,30 +1164,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 */
-               bzero(&uServer, sizeof(uServer));
+               memset(&uServer, 0, sizeof(uServer));
            }
-           return(code);
-       }
-       else
-       if ( (aflags & UF_END_SINGLESERVER) != 0 )
-       {
-           bzero(&uServer, sizeof(uServer));
-           return(0);
+           return (code);
+       } else if ((aflags & UF_END_SINGLESERVER) != 0) {
+           memset(&uServer, 0, sizeof(uServer));
+           return (0);
        }
     }
 
@@ -1196,26 +1190,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
@@ -1223,53 +1217,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)
@@ -1278,22 +1266,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,52 +1305,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
@@ -1380,38 +1363,42 @@ abort:
  *     !0: error code
  *     0: Success.
  */
-int bc_openTextFile(udbClientTextP ctPtr, char *tmpFileName)
+int
+bc_openTextFile(udbClientTextP ctPtr, char *tmpFileName)
 {
-  int code = 0;
-
-  if ( ctPtr->textStream != NULL )
-    fclose(ctPtr->textStream);
-  
-  sprintf(tmpFileName, "%s/bu_XXXXXX", gettmpdir());
-  mktemp(tmpFileName);
-  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);
+    int code = 0;
+
+    if (ctPtr->textStream != NULL)
+       fclose(ctPtr->textStream);
+
+    sprintf(tmpFileName, "%s/bu_XXXXXX", gettmpdir());
+#ifdef AFS_LINUX20_ENV
+    mkstemp(tmpFileName);
+#else
+    mktemp(tmpFileName);
 #endif
-  dprintf(("file is %s\n", tmpFileName));
+    ctPtr->textStream = fopen(tmpFileName, "w+");
+    if (ctPtr->textStream == NULL)
+       ERROR(BUDB_INTERNALERROR);
 
-normal_exit:
-  return code;
+#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
 
-error_exit:
-  if ( ctPtr->textStream != NULL )
-    {
-      fclose(ctPtr->textStream);
-      ctPtr->textStream = NULL;
+    dprintf(("file is %s\n", tmpFileName));
+
+  normal_exit:
+    return code;
+
+  error_exit:
+    if (ctPtr->textStream != NULL) {
+       fclose(ctPtr->textStream);
+       ctPtr->textStream = NULL;
     }
-  goto normal_exit;
+    goto normal_exit;
 }
 
 
@@ -1423,17 +1410,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;
 }