Consolidate duplicate definitions
authorMichael Meffie <mmeffie@sinenomine.net>
Thu, 10 Dec 2009 16:37:34 +0000 (11:37 -0500)
committerDerrick Brashear <shadow|account-1000005@unknown>
Mon, 21 Dec 2009 21:36:05 +0000 (13:36 -0800)
Housekeeping change to consolidate some common definitions.
Create a new common header called afs_consts.h.  This allows
us to remove the afscp.c dependency on afs.h (so src/tests
will build again.) Rename the max filename definition in the
update server package to MAXFNSIZE to avoid a name conflict
with MAXSIZE.  The global defines will be prefixed with
AFS_ in a second patch.

Change-Id: I2b8d555a244cc92d889618de4eec4a99550d7c7f
Reviewed-on: http://gerrit.openafs.org/855
Reviewed-by: Mickey Lane <mlane@sinenomine.net>
Tested-by: Mickey Lane <mlane@sinenomine.net>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: Derrick Brashear <shadow@dementia.org>

24 files changed:
src/JAVA/libjafs/ACL.c
src/WINNT/afsd/fs.c
src/WINNT/afsd/symlink.c
src/WINNT/client_exp/gui2fs.cpp
src/WINNT/client_exp/make_symbolic_link_dlg.cpp
src/WINNT/client_exp/shell_ext.cpp
src/afs/Makefile.in
src/afs/NTMakefile
src/afs/afs.h
src/afs/afs_consts.h [new file with mode: 0644]
src/kauth/kkids.c
src/kauth/test/test_interim_ktc.c
src/sys/rmtsysnet.c
src/update/client.c
src/update/global.h
src/update/server.c
src/uss/uss_acl.c
src/venus/afsio.c
src/venus/fs.c
src/venus/livesys.c
src/venus/twiddle.c
src/venus/up.c
src/vlserver/cnvldb.c
src/vlserver/sascnvldb.c

index 20f0557..11fe67f 100644 (file)
@@ -31,6 +31,7 @@
 #include <afs/vice.h>
 #include <netinet/in.h>
 #include <afs/venus.h>
+#include <afs/afs_consts.h>
 #include <afs/afs_args.h>
 
 #include "GetNativeString.h"
 */
 
 /* just for debugging */
-#define        MAXHOSTS 13
-#define        OMAXHOSTS 8
 #define MAXNAME 100
-#define        MAXSIZE 2048
 #define MAXINSIZE 1300    /* pioctl complains if data is larger than this */
 #define VMSGSIZE 128      /* size of msg buf in volume hdr */
 
index 295c1cf..6cf53e5 100644 (file)
@@ -24,6 +24,7 @@
 
 #include <osi.h>
 #include <afsint.h>
+#include <afs/afs_consts.h>
 #include <afs/cellconfig.h>
 #include <afs/ptserver.h>
 #include <afs/ptuser.h>
 #include "afsd.h"
 #include "cm_ioctl.h"
 
-#define        MAXHOSTS 13
-#define        OMAXHOSTS 8
-#define MAXCELLHOSTS 8
 #define MAXNAME 100
-#define        MAXSIZE 2048
 #define MAXINSIZE 1300    /* pioctl complains if data is larger than this */
 #define VMSGSIZE 128      /* size of msg buf in volume hdr */
 #define CELL_MAXNAMELEN                256
index 758b8a6..ad85016 100644 (file)
@@ -10,6 +10,7 @@
 #include <afs/param.h>
 #include <afs/stds.h>
 #include <afs/com_err.h>
+#include <afs/afs_consts.h>
 
 #include <windows.h>
 #include <stdlib.h>
@@ -29,7 +30,6 @@
 #include "cmd.h"
 
 #define MAXNAME 100
-#define        MAXSIZE 2048
 #define MAXINSIZE 1300    /* pioctl complains if data is larger than this */
 
 static char space[MAXSIZE];
index 0c14029..9c6b527 100644 (file)
@@ -33,6 +33,7 @@ extern "C" {
 #include "fs.h"
 #include "fs_utils.h"
 #include <afsint.h>
+#include <afs/afs_consts.h>
 #include <afs/cellconfig.h>
 #include <afs/vldbint.h>
 #include <afs/volser.h>
@@ -52,10 +53,7 @@ extern "C" {
 #define PCCHAR(str)            ((char *)(const char *)(str))
 #define VL_NOENT                (363524L)
 
-#define        MAXHOSTS 13
-#define        OMAXHOSTS 8
 #define MAXNAME 100
-#define        MAXSIZE 2048
 #define MAXINSIZE 1300    /* pioctl complains if data is larger than this */
 #define VMSGSIZE 128      /* size of msg buf in volume hdr */
 
index 376da86..e35ac78 100644 (file)
@@ -20,7 +20,6 @@ extern "C" {
 #include "make_symbolic_link_dlg.h"
 #include "gui2fs.h"
 #include "msgs.h"
-#define MAXSIZE 2048 /* most I'll get back from PIOCTL */
 
 #ifdef _DEBUG
 #define new DEBUG_NEW
index bee75fa..3b7a82d 100644 (file)
@@ -15,6 +15,7 @@
 extern "C" {
 #include <afs/param.h>
 #include <afs/stds.h>
+#include <afs/afs_consts.h>
 }
 
 #include <atlconv.h>
@@ -46,7 +47,6 @@ ULONG nICRefCount=0;
 ULONG nTPRefCount=0;
 ULONG nXPRefCount=0;
 
-#define MAXSIZE 2048 /* most I'll get back from PIOCTL */
 #define PCCHAR(str)    ((char *)(const char *)str)
 static char space[MAXSIZE];
 
index 0841a6c..9f4864f 100644 (file)
@@ -41,6 +41,7 @@ afszcm.cat: afs_trace.msf
        esac
 
 depinstall: afszcm.cat AFS_component_version_number.c \
+       ${TOP_INCDIR}/afs/afs_consts.h \
        ${TOP_INCDIR}/afs/afs.h \
        ${TOP_INCDIR}/afs/osi_inode.h \
        ${TOP_INCDIR}/afs/afs_stats.h \
@@ -49,6 +50,9 @@ depinstall: afszcm.cat AFS_component_version_number.c \
        ${TOP_INCDIR}/afs/unified_afs.h \
        ${TOP_INCDIR}/afs/sysctl.h
 
+${TOP_INCDIR}/afs/afs_consts.h: afs_consts.h
+       ${INSTALL_DATA} $? $@
+
 ${TOP_INCDIR}/afs/afs.h: afs.h
        ${INSTALL_DATA} $? $@
 
@@ -73,6 +77,7 @@ ${TOP_INCDIR}/afs/unified_afs.h: unified_afs.h
 install: unified_afs.h
        ${INSTALL} -d ${DESTDIR}${includedir}/afs
        ${INSTALL_DATA} afs.h ${DESTDIR}${includedir}/afs/afs.h
+       ${INSTALL_DATA} afs_consts.h ${DESTDIR}${includedir}/afs/afs_consts.h
        ${INSTALL_DATA} afs_stats.h ${DESTDIR}${includedir}/afs/afs_stats.h
        ${INSTALL_DATA} exporter.h ${DESTDIR}${includedir}/afs/exporter.h
        ${INSTALL_DATA} nfsclient.h ${DESTDIR}${includedir}/afs/nfsclient.h
@@ -98,6 +103,7 @@ install: unified_afs.h
 dest: unified_afs.h
        ${INSTALL} -d ${DEST}/include/afs
        ${INSTALL_DATA} afs.h ${DEST}/include/afs/afs.h
+       ${INSTALL_DATA} afs_consts.h ${DEST}/include/afs/afs_consts.h
        ${INSTALL_DATA} afs_stats.h ${DEST}/include/afs/afs_stats.h
        ${INSTALL_DATA} exporter.h ${DEST}/include/afs/exporter.h
        ${INSTALL_DATA} nfsclient.h ${DEST}/include/afs/nfsclient.h
index eee44ca..4f688d0 100644 (file)
@@ -19,5 +19,5 @@ $(INCDIR)\afs\unified_afs.c $(INCDIR)\afs\unified_afs.h:   unified_afs.et  unifi
    $(COMPILE_ET) unified_afs -h unified_afs
    $(COPY) unified_afs.h $(DESTDIR)\include\afs
 
-install: $(INCDIR)\afs\unified_afs.c $(INCDIR)\afs\afs_stats.h
+install: $(INCDIR)\afs\unified_afs.c $(INCDIR)\afs\afs_stats.h $(INCDIR)\afs\afs_consts.h
 
index 7659983..3685d53 100644 (file)
  */
 #ifdef KDUMP_KERNEL
 #include <afs/afs_args.h>
+#include <afs/afs_consts.h>
 #else
 #include "afs/afs_args.h"
+#include "afs/afs_consts.h"
 #endif
 
 /*
@@ -69,9 +71,6 @@ extern int afs_shuttingdown;
 
 /* The basic defines for the Andrew file system
     better keep things powers of two so "& (foo-1)" hack works for masking bits */
-#define        MAXHOSTS        13      /* max hosts per single volume */
-#define        OMAXHOSTS        8      /* backwards compatibility */
-#define MAXCELLHOSTS    8      /* max vldb servers per cell */
 #define        NBRS            15      /* max number of queued daemon requests */
 #define        NUSERS          16      /* hash table size for unixuser table */
 #define        NSERVERS        16      /* hash table size for server table */
diff --git a/src/afs/afs_consts.h b/src/afs/afs_consts.h
new file mode 100644 (file)
index 0000000..14e2635
--- /dev/null
@@ -0,0 +1,19 @@
+/*
+ * 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
+ */
+
+#ifndef _AFS_CONSTS_H_
+#define _AFS_CONSTS_H_
+
+#define        MAXHOSTS        13      /* max hosts per single volume */
+#define        OMAXHOSTS        8      /* backwards compatibility */
+#define MAXCELLHOSTS    8      /* max vldb servers per cell */
+
+#define MAXSIZE                2048    /* max returned from PIOCTL */
+
+#endif /* _AFS_CONSTS_H_ */
index 295ebbd..10361ef 100644 (file)
 #include <stdlib.h>
 #include <string.h>
 #include <afs/sys_prototypes.h>
+#include <afs/afs_consts.h>
 
 #include "kkids.h"
 
 #define MAXNAME 100
-#define MAXSIZE 2048
 
 static int using_child = 0;
 static FILE *childin, *childout;       /* file pointers on pipe to kpwvalid */
index 2c7baad..e96d7e6 100644 (file)
@@ -23,6 +23,7 @@
 #include <afs/com_err.h>
 #include <afs/cellconfig.h>
 #include <afs/auth.h>
+#include <afs/afs_consts.h>
 #include "kautils.h"
 
 
@@ -296,7 +297,6 @@ CheckAuth2(server)
 /* Stolen from the "fs" command. */
 
 #define MAXNAME 100
-#define        MAXSIZE 2048
 
 static void
 ListCellsCmd()
index 9faf082..097e941 100644 (file)
@@ -9,7 +9,7 @@
 
 #include <afsconfig.h>
 #include <afs/param.h>
-
+#include <afs/afs_consts.h>
 
 #include <errno.h>
 #include <sys/param.h>
@@ -38,8 +38,6 @@
  */
 
 #define        MAXNAME         100
-#define        MAXSIZE         2048
-#define        MAXHOSTS        8       /* XXX HARD Limit limitation XXX */
 #define        MAXGCSIZE       16
 
 struct Acl {
index 333e1eb..74185e6 100644 (file)
@@ -111,7 +111,7 @@ main(int argc, char **argv)
 
     afs_uint32 mode;
     int error;
-    char hostname[MAXSIZE];
+    char hostname[MAXFNSIZE];
     FILE *stream;
     afs_int32 time, length, atime;
     struct filestr *df;
@@ -122,7 +122,7 @@ main(int argc, char **argv)
     int a, cnt;
     rxkad_level level;
 
-    char dirbuf[MAXSIZE], filename[MAXSIZE];
+    char dirbuf[MAXFNSIZE], filename[MAXFNSIZE];
     struct filestr *dirname, *ModFiles, *okhostfiles;
 #ifdef AFS_AIX32_ENV
     /*
@@ -347,7 +347,7 @@ main(int argc, char **argv)
            {                   /*delete all the redundant files on the client */
                DIR *dirp;
                struct dirent *dp;
-               char filename[MAXSIZE];
+               char filename[MAXFNSIZE];
 
                dirp = opendir(curDir);
                if (dirp == 0) {
@@ -602,7 +602,7 @@ NotOnHost(char *filename, struct filestr *okhostfiles)
 static int
 RenameNewFiles(struct filestr *modFiles)
 {
-    char newname[MAXSIZE];
+    char newname[MAXFNSIZE];
     char *fname;
     int errcode = 0;
     struct filestr *tf;
@@ -656,7 +656,7 @@ GetFileFromUpServer(struct rx_connection *conn,     /* handle for upserver */
 #else
     struct timeval tvp[2];
 #endif
-    char newfile[MAXSIZE];
+    char newfile[MAXFNSIZE];
 
     /* construct local path from canonical (wire-format) path */
     errcode = ConstructLocalPath(filename, "/", &lfile);
index 550f95c..2606d52 100644 (file)
@@ -15,7 +15,7 @@
 #endif
 
 #define TIMEOUT        300     /*interval after which the files are resynch'ed */
-#define MAXSIZE        1024    /*max size of filenames */
+#define MAXFNSIZE      1024    /*max size of filenames */
 #define        MAXENTRIES      20
 #define UPDATEERR      100
 
index 9b5641c..67ac54e 100644 (file)
@@ -465,7 +465,7 @@ update_SendDirInfo(char *name,              /* Name of dir to enumerate */
     struct dirent *dp;
     FILE *stream;
     struct stat tstatus;
-    char filename[MAXSIZE], dirInfoFile[MAXSIZE];
+    char filename[MAXFNSIZE], dirInfoFile[MAXFNSIZE];
     int fd, tfd, errcode, error, err;
 
     error = 0;
@@ -487,7 +487,7 @@ update_SendDirInfo(char *name,              /* Name of dir to enumerate */
                goto fail;
            }
            if ((tstatus.st_mode & S_IFMT) != S_IFDIR) {        /* not a directory */
-               char dirEntry[MAXSIZE];
+               char dirEntry[MAXFNSIZE];
 
                strcpy(dirEntry, origDir);
                strcat(dirEntry, "/");
index 65e68b1..8ae9e79 100644 (file)
@@ -35,9 +35,9 @@
 #include <afs/afsint.h>
 #include <afs/prs_fs.h>
 #include <afs/com_err.h>
+#include <afs/afs_consts.h>
 
 #define MAXNAME 100
-#define        MAXSIZE 2048
 
 #undef USS_ACL_DB
 
index 8d2cd8f..41e5655 100644 (file)
@@ -73,6 +73,7 @@
 #include <afs/afsutil.h>
 #include <rx/rx.h>
 #include <rx/xdr.h>
+#include <afs/afs_consts.h>
 #include <afs/afscbint.h>
 #include <afs/vldbint.h>
 #include <afs/vlserver.h>
 #include <des_prototypes.h>
 #include <rx_prototypes.h>
 #include "../rxkad/md5.h"
-#define        MAXHOSTS 13
 #ifdef O_LARGEFILE
 #define afs_stat        stat64
 #define afs_fstat       fstat64
index 96ce349..383590e 100644 (file)
@@ -11,6 +11,7 @@
 #include <afs/param.h>
 
 
+#include <afs/afs_consts.h>
 #include <afs/afs_args.h>
 #include <rx/xdr.h>
 #include <sys/ioctl.h>
@@ -24,6 +25,7 @@
 #include <afs/vice.h>
 #include <afs/venus.h>
 #include <afs/com_err.h>
+#include <afs/afs_consts.h>
 #ifdef AFS_AIX32_ENV
 #include <signal.h>
 #endif
 #include <afs/ptuser.h>
 #include <afs/afsutil.h>
 #include <afs/sys_prototypes.h>
-    
-#define        MAXHOSTS 13
-#define        OMAXHOSTS 8
-#define MAXCELLHOSTS 8
+
 #define MAXNAME 100
-#define        MAXSIZE 2048
 #define MAXINSIZE 1300         /* pioctl complains if data is larger than this */
 #define VMSGSIZE 128           /* size of msg buf in volume hdr */
 
index ef7e7af..9978790 100644 (file)
 #include <afs/vice.h>
 #include <afs/venus.h>
 #include <afs/com_err.h>
+#include <afs/afs_consts.h>
 #include <afs/sys_prototypes.h>
 #ifdef AFS_AIX32_ENV
 #include <signal.h>
 #endif
 
-#define        MAXSIZE 2048
 static char space[MAXSIZE];
 
 int
index 6b731ab..af736f0 100644 (file)
@@ -35,8 +35,8 @@
 #include <afs/cmd.h>
 #include <strings.h>
 #include <afs/com_err.h>
+#include <afs/afs_consts.h>
 
-#define        MAXSIZE 2048
 #define MAXINSIZE 1300         /* pioctl complains if data is larger than this */
 #define VMSGSIZE 128           /* size of msg buf in volume hdr */
 
index 1cc7e73..057d593 100644 (file)
@@ -47,6 +47,7 @@
 #include <afs/venus.h>
 #include <afs/sys_prototypes.h>
 #include <afs/afsutil.h>
+#include <afs/afs_consts.h>
 
 /* ************************************************************* */
 
@@ -64,7 +65,6 @@ short setacl = 1;
 short oldAcl = 0;
 char file1[MAXPATHLEN], file2[MAXPATHLEN];
 
-#define        MAXSIZE 2048
 static char space[MAXSIZE];
 
 struct OldAcl {
index 8f20dd9..15561da 100644 (file)
@@ -27,7 +27,6 @@
 #include "vlserver.h"
 #include "cnvldb.h"            /* CHANGEME! */
 
-#define MAXSIZE 2048           /* most I'll get back from PIOCTL */
 #define        BADSERVERID     255     /* XXX */
 
 
index 4c6bb6d..b73b6ef 100644 (file)
@@ -20,6 +20,7 @@
 #include <netinet/in.h>
 #include <afs/venus.h>
 #include <afs/cmd.h>
+#include <afs/afs_consts.h>
 
 #ifdef notdef
 #include <afs/vice.h>
@@ -32,7 +33,6 @@
 #endif
 #include <strings.h>
 
-#define MAXSIZE 2048           /* most I'll get back from PIOCTL */
 
 
 static char pn[] = "cnvldb";