* gpgkeys_finger.c, gpgkeys_hkp.c, gpgkeys_http.c, gpgkeys_ldap.c: Part 1
authorDavid Shaw <dshaw@jabberwocky.com>
Thu, 13 Jan 2005 23:22:10 +0000 (23:22 +0000)
committerDavid Shaw <dshaw@jabberwocky.com>
Thu, 13 Jan 2005 23:22:10 +0000 (23:22 +0000)
of a minor cleanup to use #defines instead of hard-coded sizes.

keyserver/ChangeLog
keyserver/gpgkeys_finger.c
keyserver/gpgkeys_hkp.c
keyserver/gpgkeys_http.c
keyserver/gpgkeys_ldap.c

index a5bcb7d..37cb73a 100644 (file)
@@ -1,5 +1,9 @@
 2005-01-13  David Shaw  <dshaw@jabberwocky.com>
 
+       * gpgkeys_finger.c, gpgkeys_hkp.c, gpgkeys_http.c, gpgkeys_ldap.c:
+       Part 1 of a minor cleanup to use #defines instead of hard-coded
+       sizes.
+
        * gpgkeys_finger.c (connect_server): Use INADDR_NONE instead of
        SOCKET_ERROR.  Noted by Timo.
 
index cf06fec..535c919 100644 (file)
@@ -57,9 +57,15 @@ extern int optind;
 
 #define GET    0
 #define MAX_LINE 80
+#define MAX_COMMAND 7
+#define MAX_OPAQUE 1024
+#define MAX_OPTION 256
+
+#define STRINGIFY(x) #x
+#define MKSTRING(x) STRINGIFY(x)
 
 static int verbose=0;
-static char path[1024];
+static char path[MAX_OPAQUE+1];
 static FILE *input, *output, *console;
 
 #define BEGIN "-----BEGIN PGP PUBLIC KEY BLOCK-----"
@@ -433,8 +439,8 @@ main(int argc,char *argv[])
   while(fgets(line,MAX_LINE,input)!=NULL)
     {
       int version;
-      char commandstr[7];
-      char optionstr[256];
+      char command[MAX_COMMAND+1];
+      char option[MAX_OPTION+1];
       char hash;
 
       if(line[0]=='\n')
@@ -443,11 +449,11 @@ main(int argc,char *argv[])
       if(sscanf(line,"%c",&hash)==1 && hash=='#')
        continue;
 
-      if(sscanf(line,"COMMAND %6s\n",commandstr)==1)
+      if(sscanf(line,"COMMAND %" MKSTRING(MAX_COMMAND) "s\n",command)==1)
        {
-         commandstr[6]='\0';
+         command[MAX_COMMAND]='\0';
 
-         if(strcasecmp(commandstr,"get")==0)
+         if(strcasecmp(command,"get")==0)
            action=GET;
 
          continue;
@@ -461,9 +467,9 @@ main(int argc,char *argv[])
          goto fail;
        }
 
-      if(sscanf(line,"OPAQUE %1023s\n",path)==1)
+      if(sscanf(line,"OPAQUE %" MKSTRING(MAX_OPAQUE) "s\n",path)==1)
        {
-         path[1023]='\0';
+         path[MAX_OPAQUE]='\0';
          continue;
        }
 
@@ -478,17 +484,17 @@ main(int argc,char *argv[])
          continue;
        }
 
-      if(sscanf(line,"OPTION %255s\n",optionstr)==1)
+      if(sscanf(line,"OPTION %" MKSTRING(MAX_OPTION) "s\n",option)==1)
        {
          int no=0;
-         char *start=&optionstr[0];
+         char *start=&option[0];
 
-         optionstr[255]='\0';
+         option[MAX_OPTION]='\0';
 
-         if(strncasecmp(optionstr,"no-",3)==0)
+         if(strncasecmp(option,"no-",3)==0)
            {
              no=1;
-             start=&optionstr[3];
+             start=&option[3];
            }
 
          if(strcasecmp(start,"verbose")==0)
index db1e7ea..4639a50 100644 (file)
@@ -41,10 +41,18 @@ extern int optind;
 #define SEND   1
 #define SEARCH 2
 #define MAX_LINE 80
+#define MAX_HOST 80
+#define MAX_PORT 10
+#define MAX_PROXY 80
+#define MAX_COMMAND 7
+#define MAX_OPTION 110
+
+#define STRINGIFY(x) #x
+#define MKSTRING(x) STRINGIFY(x)
 
 static int verbose=0,include_revoked=0,include_disabled=0;
 static unsigned int http_flags=0;
-static char host[80]={'\0'},proxy[80]={'\0'},port[10]={'\0'};
+static char host[MAX_HOST+1]={'\0'},proxy[MAX_PROXY+1]={'\0'},port[MAX_PORT+1]={'\0'};
 static FILE *input=NULL,*output=NULL,*console=NULL;
 
 #define BEGIN "-----BEGIN PGP PUBLIC KEY BLOCK-----"
@@ -831,8 +839,8 @@ main(int argc,char *argv[])
   while(fgets(line,MAX_LINE,input)!=NULL)
     {
       int version;
-      char commandstr[7];
-      char optionstr[110];
+      char command[MAX_COMMAND+1];
+      char option[MAX_OPTION+1];
       char hash;
 
       if(line[0]=='\n')
@@ -841,29 +849,29 @@ main(int argc,char *argv[])
       if(sscanf(line,"%c",&hash)==1 && hash=='#')
        continue;
 
-      if(sscanf(line,"COMMAND %6s\n",commandstr)==1)
+      if(sscanf(line,"COMMAND %" MKSTRING(MAX_COMMAND) "s\n",command)==1)
        {
-         commandstr[6]='\0';
+         command[MAX_COMMAND]='\0';
 
-         if(strcasecmp(commandstr,"get")==0)
+         if(strcasecmp(command,"get")==0)
            action=GET;
-         else if(strcasecmp(commandstr,"send")==0)
+         else if(strcasecmp(command,"send")==0)
            action=SEND;
-         else if(strcasecmp(commandstr,"search")==0)
+         else if(strcasecmp(command,"search")==0)
            action=SEARCH;
 
          continue;
        }
 
-      if(sscanf(line,"HOST %79s\n",host)==1)
+      if(sscanf(line,"HOST %" MKSTRING(MAX_HOST) "s\n",host)==1)
        {
-         host[79]='\0';
+         host[MAX_HOST]='\0';
          continue;
        }
 
-      if(sscanf(line,"PORT %9s\n",port)==1)
+      if(sscanf(line,"PORT %" MKSTRING(MAX_PORT) "s\n",port)==1)
        {
-         port[9]='\0';
+         port[MAX_PORT]='\0';
          continue;
        }
 
@@ -878,17 +886,17 @@ main(int argc,char *argv[])
          continue;
        }
 
-      if(sscanf(line,"OPTION %109s\n",optionstr)==1)
+      if(sscanf(line,"OPTION %" MKSTRING(MAX_OPTION) "s\n",option)==1)
        {
          int no=0;
-         char *start=&optionstr[0];
+         char *start=&option[0];
 
-         optionstr[109]='\0';
+         option[MAX_OPTION]='\0';
 
-         if(strncasecmp(optionstr,"no-",3)==0)
+         if(strncasecmp(option,"no-",3)==0)
            {
              no=1;
-             start=&optionstr[3];
+             start=&option[3];
            }
 
          if(strcasecmp(start,"verbose")==0)
@@ -918,16 +926,16 @@ main(int argc,char *argv[])
                proxy[0]='\0';
              else if(start[10]=='=')
                {
-                 strncpy(proxy,&start[11],79);
-                 proxy[79]='\0';
+                 strncpy(proxy,&start[11],MAX_PROXY);
+                 proxy[MAX_PROXY]='\0';
                }
              else if(start[10]=='\0')
                {
                  char *http_proxy=getenv(HTTP_PROXY_ENV);
                  if(http_proxy)
                    {
-                     strncpy(proxy,http_proxy,79);
-                     proxy[79]='\0';
+                     strncpy(proxy,http_proxy,MAX_PROXY);
+                     proxy[MAX_PROXY]='\0';
                    }
                }
            }
index 76da9e0..606e26f 100644 (file)
@@ -38,10 +38,20 @@ extern int optind;
 
 #define GET    0
 #define MAX_LINE 80
+#define MAX_AUTH 128
+#define MAX_HOST 80
+#define MAX_PROXY 80
+#define MAX_PORT 10
+#define MAX_PATH 1024
+#define MAX_COMMAND 7
+#define MAX_OPTION 256
+
+#define STRINGIFY(x) #x
+#define MKSTRING(x) STRINGIFY(x)
 
 static int verbose=0;
 static unsigned int http_flags=0;
-static char auth[128]={'\0'},host[80]={'\0'},proxy[80]={'\0'},port[10]={'\0'},path[1024]={'\0'};
+static char auth[MAX_AUTH+1]={'\0'},host[MAX_HOST+1]={'\0'},proxy[MAX_PROXY+1]={'\0'},port[MAX_PORT+1]={'\0'},path[MAX_PATH+1]={'\0'};
 static FILE *input=NULL,*output=NULL,*console=NULL;
 
 #define BEGIN "-----BEGIN PGP PUBLIC KEY BLOCK-----"
@@ -199,8 +209,8 @@ main(int argc,char *argv[])
   while(fgets(line,MAX_LINE,input)!=NULL)
     {
       int version;
-      char commandstr[7];
-      char optionstr[256];
+      char command[MAX_COMMAND+1];
+      char option[MAX_OPTION+1];
       char hash;
 
       if(line[0]=='\n')
@@ -209,37 +219,37 @@ main(int argc,char *argv[])
       if(sscanf(line,"%c",&hash)==1 && hash=='#')
        continue;
 
-      if(sscanf(line,"COMMAND %6s\n",commandstr)==1)
+      if(sscanf(line,"COMMAND %" MKSTRING(MAX_COMMAND) "s\n",command)==1)
        {
-         commandstr[6]='\0';
+         command[MAX_COMMAND]='\0';
 
-         if(strcasecmp(commandstr,"get")==0)
+         if(strcasecmp(command,"get")==0)
            action=GET;
 
          continue;
        }
 
-      if(sscanf(line,"AUTH %127s\n",auth)==1)
+      if(sscanf(line,"AUTH %" MKSTRING(MAX_AUTH) "s\n",auth)==1)
        {
-         auth[127]='\0';
+         auth[MAX_AUTH]='\0';
          continue;
        }
 
-      if(sscanf(line,"HOST %79s\n",host)==1)
+      if(sscanf(line,"HOST %" MKSTRING(MAX_HOST) "s\n",host)==1)
        {
-         host[79]='\0';
+         host[MAX_HOST]='\0';
          continue;
        }
 
-      if(sscanf(line,"PORT %9s\n",port)==1)
+      if(sscanf(line,"PORT %" MKSTRING(MAX_PORT) "s\n",port)==1)
        {
-         port[9]='\0';
+         port[MAX_PORT]='\0';
          continue;
        }
 
-      if(sscanf(line,"PATH %1023s\n",path)==1)
+      if(sscanf(line,"PATH %" MKSTRING(MAX_PATH) "s\n",path)==1)
        {
-         path[1023]='\0';
+         path[MAX_PATH]='\0';
          continue;
        }
 
@@ -254,17 +264,17 @@ main(int argc,char *argv[])
          continue;
        }
 
-      if(sscanf(line,"OPTION %255s\n",optionstr)==1)
+      if(sscanf(line,"OPTION %" MKSTRING(MAX_OPTION) "s\n",option)==1)
        {
          int no=0;
-         char *start=&optionstr[0];
+         char *start=&option[0];
 
-         optionstr[255]='\0';
+         option[MAX_OPTION]='\0';
 
-         if(strncasecmp(optionstr,"no-",3)==0)
+         if(strncasecmp(option,"no-",3)==0)
            {
              no=1;
-             start=&optionstr[3];
+             start=&option[3];
            }
 
          if(strcasecmp(start,"verbose")==0)
index bc603e7..8c1335d 100644 (file)
@@ -50,13 +50,21 @@ extern int optind;
 #define GET    0
 #define SEND   1
 #define SEARCH 2
-#define MAX_LINE 256
+#define MAX_LINE 255
+#define MAX_HOST 80
+#define MAX_SCHEME   20
+#define MAX_PORT 10
+#define MAX_OPTION 255
+#define MAX_COMMAND 7
+
+#define STRINGIFY(x) #x
+#define MKSTRING(x) STRINGIFY(x)
 
 static int verbose=0,include_disabled=0,include_revoked=0,include_subkeys=0;
 static int real_ldap=0;
 static char *basekeyspacedn=NULL;
-static char host[80]={'\0'};
-static char portstr[10]={'\0'};
+static char host[MAX_HOST+1]={'\0'};
+static char portstr[MAX_PORT+1]={'\0'};
 static char *pgpkeystr="pgpKey";
 static FILE *input=NULL,*output=NULL,*console=NULL;
 static LDAP *ldap=NULL;
@@ -1609,9 +1617,9 @@ main(int argc,char *argv[])
 
   while(fgets(line,MAX_LINE,input)!=NULL)
     {
-      char commandstr[7];
-      char optionstr[256];
-      char schemestr[80];
+      char command[MAX_COMMAND+1];
+      char optionstr[MAX_OPTION+1];
+      char scheme[MAX_SCHEME+1];
       char hash;
 
       if(line[0]=='\n')
@@ -1620,37 +1628,37 @@ main(int argc,char *argv[])
       if(sscanf(line,"%c",&hash)==1 && hash=='#')
        continue;
 
-      if(sscanf(line,"COMMAND %6s\n",commandstr)==1)
+      if(sscanf(line,"COMMAND %" MKSTRING(MAX_COMMAND) "s\n",command)==1)
        {
-         commandstr[6]='\0';
+         command[MAX_COMMAND]='\0';
 
-         if(strcasecmp(commandstr,"get")==0)
+         if(strcasecmp(command,"get")==0)
            action=GET;
-         else if(strcasecmp(commandstr,"send")==0)
+         else if(strcasecmp(command,"send")==0)
            action=SEND;
-         else if(strcasecmp(commandstr,"search")==0)
+         else if(strcasecmp(command,"search")==0)
            action=SEARCH;
 
          continue;
        }
 
-      if(sscanf(line,"HOST %79s\n",host)==1)
+      if(sscanf(line,"HOST %" MKSTRING(MAX_HOST) "s\n",host)==1)
        {
-         host[79]='\0';
+         host[MAX_HOST]='\0';
          continue;
        }
 
-      if(sscanf(line,"PORT %9s\n",portstr)==1)
+      if(sscanf(line,"PORT %" MKSTRING(MAX_PORT) "s\n",portstr)==1)
        {
-         portstr[9]='\0';
+         portstr[MAX_PORT]='\0';
          port=atoi(portstr);
          continue;
        }
 
-      if(sscanf(line,"SCHEME %79s\n",schemestr)==1)
+      if(sscanf(line,"SCHEME %" MKSTRING(MAX_SCHEME) "s\n",scheme)==1)
        {
-         schemestr[79]='\0';
-         if(strcasecmp(schemestr,"ldaps")==0)
+         scheme[MAX_SCHEME]='\0';
+         if(strcasecmp(scheme,"ldaps")==0)
            {
              port=636;
              use_ssl=1;
@@ -1669,12 +1677,12 @@ main(int argc,char *argv[])
          continue;
        }
 
-      if(sscanf(line,"OPTION %255[^\n]\n",optionstr)==1)
+      if(sscanf(line,"OPTION %" MKSTRING(MAX_OPTION) "[^\n]\n",optionstr)==1)
        {
          int no=0;
          char *start=&optionstr[0];
 
-         optionstr[255]='\0';
+         optionstr[MAX_OPTION]='\0';
 
          if(strncasecmp(optionstr,"no-",3)==0)
            {