Raise WARNS in secure/ as high as possible and fix some resulting warnings.
authorSascha Wildner <saw@online.de>
Mon, 18 Jan 2010 09:01:43 +0000 (10:01 +0100)
committerSascha Wildner <saw@online.de>
Mon, 18 Jan 2010 09:01:43 +0000 (10:01 +0100)
23 files changed:
crypto/openssh/clientloop.c
crypto/openssh/readconf.c
crypto/openssh/servconf.c
crypto/openssh/session.c
crypto/openssh/sshd.c
crypto/openssh/sshlogin.c
crypto/openssh/sshlogin.h
secure/lib/libcipher/Makefile
secure/lib/libcipher/crypt.c
secure/lib/libssl/Makefile
secure/libexec/Makefile.inc
secure/usr.bin/bdes/Makefile
secure/usr.bin/openssl/Makefile
secure/usr.bin/scp/Makefile
secure/usr.bin/sftp/Makefile
secure/usr.bin/ssh-add/Makefile
secure/usr.bin/ssh-agent/Makefile
secure/usr.bin/ssh-keygen/Makefile
secure/usr.bin/ssh-keyscan/Makefile
secure/usr.bin/ssh-vulnkey/Makefile
secure/usr.bin/ssh/Makefile
secure/usr.sbin/Makefile.inc
secure/usr.sbin/sshd/auth-passwd-freebsd.c

index 8d28bda..98949a5 100644 (file)
@@ -1553,8 +1553,8 @@ client_loop(int have_pty, int escape_char_arg, int ssh2_chan_id)
        total_time = get_current_time() - start_time;
        packet_get_state(MODE_IN, NULL, NULL, NULL, &ibytes);
        packet_get_state(MODE_OUT, NULL, NULL, NULL, &obytes);
        total_time = get_current_time() - start_time;
        packet_get_state(MODE_IN, NULL, NULL, NULL, &ibytes);
        packet_get_state(MODE_OUT, NULL, NULL, NULL, &obytes);
-       verbose("Transferred: sent %llu, received %llu bytes, in %.1f seconds",
-           obytes, ibytes, total_time);
+       verbose("Transferred: sent %ju, received %ju bytes, in %.1f seconds",
+           (uintmax_t)obytes, (uintmax_t)ibytes, total_time);
        if (total_time > 0)
                verbose("Bytes per second: sent %.1f, received %.1f",
                    obytes / total_time, ibytes / total_time);
        if (total_time > 0)
                verbose("Bytes per second: sent %.1f, received %.1f",
                    obytes / total_time, ibytes / total_time);
index e26d952..7377ab9 100644 (file)
@@ -43,6 +43,7 @@
 #include "kex.h"
 #include "mac.h"
 #include "uidswap.h"
 #include "kex.h"
 #include "mac.h"
 #include "uidswap.h"
+#include "version.h"
 
 /* Format of the configuration file:
 
 
 /* Format of the configuration file:
 
index c134053..19253a8 100644 (file)
@@ -41,6 +41,7 @@
 #include "match.h"
 #include "channels.h"
 #include "groupaccess.h"
 #include "match.h"
 #include "channels.h"
 #include "groupaccess.h"
+#include "version.h"
 
 static void add_listen_addr(ServerOptions *, char *, int);
 static void add_one_listen_addr(ServerOptions *, char *, int);
 
 static void add_listen_addr(ServerOptions *, char *, int);
 static void add_one_listen_addr(ServerOptions *, char *, int);
index b18a369..85b3eef 100644 (file)
@@ -1137,7 +1137,10 @@ do_setup_env(Session *s, const char *shell)
        char *path = NULL;
 #else
        extern char **environ;
        char *path = NULL;
 #else
        extern char **environ;
-       char **senv, **var;
+       char **senv;
+#if 0
+       char **var;
+#endif
 #endif
 
        /* Initialize the environment. */
 #endif
 
        /* Initialize the environment. */
index 52d07e2..2eedc96 100644 (file)
@@ -1963,7 +1963,8 @@ main(int ac, char **av)
        /* The connection has been terminated. */
        packet_get_state(MODE_IN, NULL, NULL, NULL, &ibytes);
        packet_get_state(MODE_OUT, NULL, NULL, NULL, &obytes);
        /* The connection has been terminated. */
        packet_get_state(MODE_IN, NULL, NULL, NULL, &ibytes);
        packet_get_state(MODE_OUT, NULL, NULL, NULL, &obytes);
-       verbose("Transferred: sent %llu, received %llu bytes", obytes, ibytes);
+       verbose("Transferred: sent %ju, received %ju bytes",
+           (uintmax_t)obytes, (uintmax_t)ibytes);
 
        verbose("Closing connection to %.500s port %d", remote_ip, remote_port);
 
 
        verbose("Closing connection to %.500s port %d", remote_ip, remote_port);
 
index 33bd652..0e95154 100644 (file)
@@ -59,6 +59,7 @@
 #include "log.h"
 #include "buffer.h"
 #include "servconf.h"
 #include "log.h"
 #include "buffer.h"
 #include "servconf.h"
+#include "sshlogin.h"
 
 extern Buffer loginmsg;
 extern ServerOptions options;
 
 extern Buffer loginmsg;
 extern ServerOptions options;
index 500d3fe..52119a9 100644 (file)
@@ -15,7 +15,7 @@
 void   record_login(pid_t, const char *, const char *, uid_t,
     const char *, struct sockaddr *, socklen_t);
 void   record_logout(pid_t, const char *, const char *);
 void   record_login(pid_t, const char *, const char *, uid_t,
     const char *, struct sockaddr *, socklen_t);
 void   record_logout(pid_t, const char *, const char *);
-time_t get_last_login_time(uid_t, const char *, char *, u_int);
+time_t get_last_login_time(uid_t, const char *, char *, size_t);
 
 #ifdef LOGIN_NEEDS_UTMPX
 void   record_utmp_only(pid_t, const char *, const char *, const char *,
 
 #ifdef LOGIN_NEEDS_UTMPX
 void   record_utmp_only(pid_t, const char *, const char *, const char *,
index 44e5b5c..44c30cc 100644 (file)
@@ -7,6 +7,7 @@ LIB=            cipher
 SRCS=          crypt.c
 PRECIOUSLIB=   yes
 #NOPROFILE=    yes
 SRCS=          crypt.c
 PRECIOUSLIB=   yes
 #NOPROFILE=    yes
+WARNS?=                2
 
 MAN=   cipher.3
 MLINKS= cipher.3 encrypt.3 cipher.3 setkey.3
 
 MAN=   cipher.3
 MLINKS= cipher.3 encrypt.3 cipher.3 setkey.3
index 13488cb..abecf0b 100644 (file)
@@ -180,11 +180,6 @@ static u_long      key_perm_maskl[8][128], key_perm_maskr[8][128];
 static u_long  comp_maskl[8][128], comp_maskr[8][128];
 static u_long  old_rawkey0, old_rawkey1;
 
 static u_long  comp_maskl[8][128], comp_maskr[8][128];
 static u_long  old_rawkey0, old_rawkey1;
 
-static u_char  ascii64[] =
-        "./0123456789ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz";
-/*       0000000000111111111122222222223333333333444444444455555555556666 */
-/*       0123456789012345678901234567890123456789012345678901234567890123 */
-
 static inline int
 ascii_to_bin(char ch)
 {
 static inline int
 ascii_to_bin(char ch)
 {
@@ -361,7 +356,7 @@ int
 des_setkey(const char *key)
 {
        u_long  k0, k1, rawkey0, rawkey1;
 des_setkey(const char *key)
 {
        u_long  k0, k1, rawkey0, rawkey1;
-       int     shifts, i, b, round;
+       int     shifts, round;
 
        if (!des_initialised)
                des_init();
 
        if (!des_initialised)
                des_init();
@@ -408,7 +403,6 @@ des_setkey(const char *key)
        shifts = 0;
        for (round = 0; round < 16; round++) {
                u_long  t0, t1;
        shifts = 0;
        for (round = 0; round < 16; round++) {
                u_long  t0, t1;
-               int     bit;
 
                shifts += key_shifts[round];
 
 
                shifts += key_shifts[round];
 
@@ -445,9 +439,9 @@ do_des(     u_long l_in, u_long r_in, u_long *l_out, u_long *r_out, int count)
        /*
         *      l_in, r_in, l_out, and r_out are in pseudo-"big-endian" format.
         */
        /*
         *      l_in, r_in, l_out, and r_out are in pseudo-"big-endian" format.
         */
-       u_long  mask, rawl, rawr, l, r, *kl, *kr, *kl1, *kr1;
+       u_long  l, r, *kl, *kr, *kl1, *kr1;
        u_long  f, r48l, r48r;
        u_long  f, r48l, r48r;
-       int     i, j, b, round;
+       int     round;
 
        if (count == 0) {
                return(1);
 
        if (count == 0) {
                return(1);
index 209237d..8873c46 100644 (file)
@@ -3,6 +3,7 @@
 
 LIB=           ssl
 SHLIB_MAJOR=   4
 
 LIB=           ssl
 SHLIB_MAJOR=   4
+WARNS?=                2
 
 NOLINT=                true
 
 
 NOLINT=                true
 
index f544144..22a1017 100644 (file)
@@ -2,5 +2,6 @@
 #      $DragonFly: src/secure/libexec/Makefile.inc,v 1.2 2003/06/17 04:27:48 dillon Exp $
 
 BINDIR?=       /usr/libexec
 #      $DragonFly: src/secure/libexec/Makefile.inc,v 1.2 2003/06/17 04:27:48 dillon Exp $
 
 BINDIR?=       /usr/libexec
+WARNS?=                3
 
 .include "${.CURDIR}/../../Makefile.inc"
 
 .include "${.CURDIR}/../../Makefile.inc"
index c43f6bb..742f400 100644 (file)
@@ -4,6 +4,7 @@
 
 PROG=  bdes
 SRCS=  bdes.c
 
 PROG=  bdes
 SRCS=  bdes.c
+WARNS?=        1
 
 .if exists(${.OBJDIR}/../../lib/libcipher)
 LDFLAGS+= -L${.OBJDIR}/../../lib/libcipher
 
 .if exists(${.OBJDIR}/../../lib/libcipher)
 LDFLAGS+= -L${.OBJDIR}/../../lib/libcipher
index c91ddb6..416e712 100644 (file)
@@ -3,6 +3,7 @@
 
 PROG=  xopenssl
 PROGNAME=      openssl
 
 PROG=  xopenssl
 PROGNAME=      openssl
+WARNS?=        3
 
 DPADD= ${LIBSSL} ${LIBCRYPTO}
 LDADD= -lssl -lcrypto
 
 DPADD= ${LIBSSL} ${LIBCRYPTO}
 LDADD= -lssl -lcrypto
index b24c25d..a4f3da3 100644 (file)
@@ -3,6 +3,7 @@
 
 PROG=  scp
 SRCS=  scp.c roaming_dummy.c
 
 PROG=  scp
 SRCS=  scp.c roaming_dummy.c
+WARNS?=        3
 
 .include "../../Makefile.ssh.common"
 .include <bsd.prog.mk>
 
 .include "../../Makefile.ssh.common"
 .include <bsd.prog.mk>
index a5e84c9..65c8d9d 100644 (file)
@@ -3,6 +3,7 @@
 
 PROG=  sftp
 SRCS=  sftp.c sftp-client.c sftp-common.c sftp-glob.c roaming_dummy.c
 
 PROG=  sftp
 SRCS=  sftp.c sftp-client.c sftp-common.c sftp-glob.c roaming_dummy.c
+WARNS?=        2
 
 # Portability layer
 SRCS+= fmt_scaled.c
 
 # Portability layer
 SRCS+= fmt_scaled.c
index 3f1e585..6e03ee5 100644 (file)
@@ -3,6 +3,7 @@
 
 PROG=  ssh-add
 SRCS+= ssh-add.c roaming_dummy.c
 
 PROG=  ssh-add
 SRCS+= ssh-add.c roaming_dummy.c
+WARNS?=        3
 
 .include "../../Makefile.ssh.common"
 .include <bsd.prog.mk>
 
 .include "../../Makefile.ssh.common"
 .include <bsd.prog.mk>
index 1e15e66..edf3485 100644 (file)
@@ -3,6 +3,7 @@
 
 PROG=  ssh-agent
 SRCS=  ssh-agent.c roaming_dummy.c
 
 PROG=  ssh-agent
 SRCS=  ssh-agent.c roaming_dummy.c
+WARNS?=        3
 
 .include "../../Makefile.ssh.common"
 .include <bsd.prog.mk>
 
 .include "../../Makefile.ssh.common"
 .include <bsd.prog.mk>
index f98d7b4..a30722b 100644 (file)
@@ -3,6 +3,7 @@
 
 PROG=  ssh-keygen
 SRCS=  ssh-keygen.c roaming_dummy.c
 
 PROG=  ssh-keygen
 SRCS=  ssh-keygen.c roaming_dummy.c
+WARNS?=        3
 
 .include "../../Makefile.ssh.common"
 .include <bsd.prog.mk>
 
 .include "../../Makefile.ssh.common"
 .include <bsd.prog.mk>
index 0450847..9db3de4 100644 (file)
@@ -2,8 +2,8 @@
 # $DragonFly: src/secure/usr.bin/ssh-keyscan/Makefile,v 1.4 2008/04/07 01:20:18 pavalos Exp $
 
 PROG=  ssh-keyscan
 # $DragonFly: src/secure/usr.bin/ssh-keyscan/Makefile,v 1.4 2008/04/07 01:20:18 pavalos Exp $
 
 PROG=  ssh-keyscan
-
 SRCS=  ssh-keyscan.c misc.c roaming_dummy.c
 SRCS=  ssh-keyscan.c misc.c roaming_dummy.c
+WARNS?=        3
 
 DPADD+=        ${LIBZ}
 LDADD+=        -lz
 
 DPADD+=        ${LIBZ}
 LDADD+=        -lz
index 2761b42..9620fee 100644 (file)
@@ -2,8 +2,8 @@
 # $DragonFly: src/secure/usr.bin/ssh-vulnkey/Makefile,v 1.1 2008/05/16 11:44:56 corecode Exp $
 
 PROG=  ssh-vulnkey
 # $DragonFly: src/secure/usr.bin/ssh-vulnkey/Makefile,v 1.1 2008/05/16 11:44:56 corecode Exp $
 
 PROG=  ssh-vulnkey
-
 SRCS=  ssh-vulnkey.c roaming_dummy.c
 SRCS=  ssh-vulnkey.c roaming_dummy.c
+WARNS?=        2
 
 .include "../../Makefile.ssh.common"
 .include <bsd.prog.mk>
 
 .include "../../Makefile.ssh.common"
 .include <bsd.prog.mk>
index f98deec..4f2f03b 100644 (file)
@@ -6,6 +6,7 @@ PROG=   ssh
 LINKS= ${BINDIR}/ssh ${BINDIR}/slogin
 MAN=   ssh.1 ssh_config.5
 MLINKS=        ssh.1 slogin.1
 LINKS= ${BINDIR}/ssh ${BINDIR}/slogin
 MAN=   ssh.1 ssh_config.5
 MLINKS=        ssh.1 slogin.1
+WARNS?=        3
 
 SRCS=  ssh.c readconf.c clientloop.c sshtty.c \
        sshconnect.c sshconnect1.c sshconnect2.c mux.c \
 
 SRCS=  ssh.c readconf.c clientloop.c sshtty.c \
        sshconnect.c sshconnect1.c sshconnect2.c mux.c \
index fdfa06f..d531b05 100644 (file)
@@ -2,5 +2,6 @@
 # $DragonFly: src/secure/usr.sbin/Makefile.inc,v 1.2 2003/06/17 04:27:48 dillon Exp $
 
 BINDIR?=       /usr/sbin
 # $DragonFly: src/secure/usr.sbin/Makefile.inc,v 1.2 2003/06/17 04:27:48 dillon Exp $
 
 BINDIR?=       /usr/sbin
+WARNS?=                2
 
 .include "${.CURDIR}/../../Makefile.inc"
 
 .include "${.CURDIR}/../../Makefile.inc"
index 0fcddc3..eb966d6 100644 (file)
@@ -2,6 +2,7 @@
  * $DragonFly: src/secure/usr.sbin/sshd/auth-passwd-freebsd.c,v 1.2 2006/09/28 18:42:50 corecode Exp $
  */
 
  * $DragonFly: src/secure/usr.sbin/sshd/auth-passwd-freebsd.c,v 1.2 2006/09/28 18:42:50 corecode Exp $
  */
 
+#include <string.h>
 #include <unistd.h>
 
 #include "buffer.h"
 #include <unistd.h>
 
 #include "buffer.h"