libopie: Raise WARNS to 2 and fix warnings.
authorSascha Wildner <saw@online.de>
Thu, 5 Dec 2013 03:44:11 +0000 (04:44 +0100)
committerSascha Wildner <saw@online.de>
Thu, 5 Dec 2013 03:44:11 +0000 (04:44 +0100)
contrib/opie/libopie/accessfile.c
contrib/opie/libopie/generator.c
contrib/opie/libopie/insecure.c
contrib/opie/libopie/keycrunch.c
contrib/opie/libopie/lookup.c
contrib/opie/libopie/newseed.c
contrib/opie/libopie/readrec.c
contrib/opie/libopie/verify.c
contrib/opie/libopie/writerec.c
lib/libopie/Makefile
lib/libopie/opieextra.c

index 154ec1e..a061579 100644 (file)
@@ -58,8 +58,7 @@ int opieaccessfile FUNCTION((host), char *host)
  */
   long n;
   struct hostent *hp;
-  FILE *fp;
-  char buf[128], **lp;
+  char **lp;
 
 #ifdef DEBUG
   syslog(LOG_DEBUG, "accessfile: host=%s", host);
index 1ccedce..d6a58a2 100644 (file)
@@ -30,7 +30,6 @@ you didn't get a copy, you may request one from <license@inner.net>.
        Created at NRL for OPIE 2.2.
 
 $FreeBSD: src/contrib/opie/libopie/generator.c,v 1.3.6.2 2002/07/15 14:48:47 des Exp $
-$DragonFly: src/contrib/opie/libopie/generator.c,v 1.2 2003/06/17 04:24:05 dillon Exp $
 */
 
 #include "opie_cfg.h"
@@ -165,7 +164,7 @@ int opiegenerator FUNCTION((challenge, secret, response), char *challenge AND ch
     if (opiepasscheck(secret))
       return -2;
 
-    if (i = opiekeycrunch(algorithm, &key, seed, secret))
+    if ((i = opiekeycrunch(algorithm, &key, seed, secret)) != 0)
       return i;
 
     if (sequence <= OPIE_SEQUENCE_RESTRICT) {
@@ -175,7 +174,6 @@ int opiegenerator FUNCTION((challenge, secret, response), char *challenge AND ch
       {
        char newseed[OPIE_SEED_MAX + 1];
        struct opie_otpkey newkey;
-       char *c;
        char buf[OPIE_SEED_MAX + 48 + 1];
 
        while (sequence-- != 0)
index db42205..279e375 100644 (file)
@@ -29,7 +29,6 @@ License Agreement applies to this software.
              assignment that should have been a comparison.
 
 $FreeBSD: src/contrib/opie/libopie/insecure.c,v 1.1.1.2.6.2 2002/07/15 14:48:47 des Exp $
-$DragonFly: src/contrib/opie/libopie/insecure.c,v 1.2 2003/06/17 04:24:05 dillon Exp $
 
 */
 #include "opie_cfg.h"
@@ -77,7 +76,7 @@ int opieinsecure FUNCTION_NOARGS
 
   if (display_name) {
     insecure = 1;
-    if (s = strchr(display_name, ':')) {
+    if ((s = strchr(display_name, ':')) != NULL) {
       int n = s - display_name;
       if (!n)
        insecure = 0;
@@ -97,7 +96,7 @@ int opieinsecure FUNCTION_NOARGS
            if (!strncmp(utsname.nodename, display_name, n))
              insecure = 0;
            else {
-             if (s = strchr(display_name, '.')) {
+             if ((s = strchr(display_name, '.')) != NULL) {
                int n2 = s - display_name;
                 if (n < n2)
                   n2 = n;
@@ -138,7 +137,7 @@ int opieinsecure FUNCTION_NOARGS
        strncpy(host, utmp.ut_host, sizeof(utmp.ut_host));
        host[sizeof(utmp.ut_host)] = 0;
 
-       if (s = strchr(host, ':')) {
+       if ((s = strchr(host, ':')) != NULL) {
          int n = s - host;
          if (!n)
            insecure = 0;
@@ -148,9 +147,9 @@ int opieinsecure FUNCTION_NOARGS
                insecure = 0;
 #if 1 /* def SOLARIS */
              else
-               if (s = strchr(host, ' ')) {
+               if ((s = strchr(host, ' ')) != NULL) {
                  *s = ':';
-                 if (s = strchr(s + 1, ' '))
+                 if ((s = strchr(s + 1, ' ')) != NULL)
                    *s = '.';
                  if (!strncmp(host, display_name, n))
                    insecure = 0; 
index 2ca57e0..d9ff5ca 100644 (file)
@@ -41,7 +41,7 @@ struct opie_otpkey *result AND char *seed AND char *secret)
     char *c2 = c;
 
     if (algorithm & 0x10)
-      while(*c2 = *(secret++)) c2++;
+      while((*c2 = *(secret++)) != '\0') c2++;
 
     while(*seed)
       if (isspace(*(c2++) = tolower(*(seed++))))
index de6d614..254172b 100644 (file)
@@ -23,7 +23,7 @@ int opielookup FUNCTION((opie, principal), struct opie *opie AND char *principal
   memset(opie, 0, sizeof(struct opie));
   opie->opie_principal = principal;
 
-  if (i = __opiereadrec(opie))
+  if ((i = __opiereadrec(opie)) != 0)
     return i;
 
   return (opie->opie_flags & __OPIE_FLAGS_RW) ? 0 : 2;
index dcd8826..0903f85 100644 (file)
@@ -74,7 +74,7 @@ int opienewseed FUNCTION((seed), char *seed)
                         * seed which is at least as long as the previous one
                         * was.  snprintf() will add a NUL character as well.
                         */
-                       snprintf(c, slen + 1, "%0*d", slen, i);
+                       snprintf(c, slen + 1, "%0*d", (int)slen, i);
                        seed[OPIE_SEED_MAX] = 0;
                        return 0;
                }
index 0f80315..f6eccae 100644 (file)
@@ -135,9 +135,8 @@ int __opiereadrec FUNCTION((opie), struct opie *opie)
 
   {
     char *c, principal[OPIE_PRINCIPAL_MAX];
-    int i;
     
-    if (c = strchr(opie->opie_principal, ':'))
+    if ((c = strchr(opie->opie_principal, ':')) != NULL)
       *c = 0;
     
     strlcpy(principal, opie->opie_principal, sizeof(principal));
index 766d5cb..71e746f 100644 (file)
@@ -48,8 +48,6 @@ static struct _rtrans rtrans[] = {
   { RESPONSE_UNKNOWN, NULL }
 };
 
-static char *algids[] = { NULL, NULL, NULL, "sha1", "md4", "md5" };
-
 static int changed FUNCTION((opie), struct opie *opie)
 {
   struct opie opie2;
@@ -71,7 +69,6 @@ int opieverify FUNCTION((opie, response), struct opie *opie AND char *response)
   int i, rval = -1;
   char *c;
   struct opie_otpkey key, fkey, lastkey;
-  struct opie nopie;
 
   if (!opie || !response)
     goto verret;
@@ -93,7 +90,7 @@ int opieverify FUNCTION((opie, response), struct opie *opie AND char *response)
   if (opie->opie_n <= 0)
     goto verret;
 
-  if (c = strchr(response, ':')) {
+  if ((c = strchr(response, ':')) != NULL) {
     *(c++) = 0;
     {
       struct _rtrans *r;
index 5ba53fe..fcee2df 100644 (file)
@@ -39,14 +39,14 @@ char *__opienone = "****************";
 
 int __opiewriterec FUNCTION((opie), struct opie *opie)
 {
-  char buf[17], buf2[64];
+  char buf[64];
   time_t now;
-  FILE *f, *f2 = NULL;
+  FILE *f;
   int i = 0;
   char *c;
 
   time(&now);
-  if (strftime(buf2, sizeof(buf2), " %b %d,%Y %T", localtime(&now)) < 1)
+  if (strftime(buf, sizeof(buf), " %b %d,%Y %T", localtime(&now)) < 1)
     return -1;
 
   if (!(opie->opie_flags & __OPIE_FLAGS_READ)) {
@@ -78,7 +78,7 @@ int __opiewriterec FUNCTION((opie), struct opie *opie)
     return -1;
   }
 
-  if (fprintf(f, "%s %04d %-16s %s %-21s\n", opie->opie_principal, opie->opie_n, opie->opie_seed, opie->opie_val ? opie->opie_val : __opienone, buf2) < 1)
+  if (fprintf(f, "%s %04d %-16s %s %-21s\n", opie->opie_principal, opie->opie_n, opie->opie_seed, opie->opie_val ? opie->opie_val : __opienone, buf) < 1)
     return -1;
 
   fclose(f);
index 3749547..927acd4 100644 (file)
@@ -5,7 +5,7 @@
 OPIE_DIST?=    ${.CURDIR}/../../contrib/opie
 DIST_DIR=      ${OPIE_DIST}/${.CURDIR:T}
 SHLIB_MAJOR=    3
-WARNS?=                1
+WARNS?=                2
 
 KEYFILE?=      \"/etc/opiekeys\"
 
index b6dccda..0e98841 100644 (file)
@@ -4,7 +4,6 @@
  * and possibly others.
  *
  * $FreeBSD: src/lib/libopie/opieextra.c,v 1.1.2.2 2002/07/15 14:17:08 des Exp $
- * $DragonFly: src/lib/libopie/opieextra.c,v 1.4 2008/09/30 16:57:05 swildner Exp $
  */
 
 #include <sys/types.h>
@@ -81,7 +80,7 @@ struct opie_algorithm_table {
 };
 
 static struct opie_algorithm_table opie_algorithm_table[] = {
-       "md4", "md5"
+       { "md4" }, { "md5" }
 };
 
 /* Get current hash type */