changeset 94:c85c88500ea6

DEBUG_TRACE now only triggers with -v on the cmdline
author Matt Johnston <matt@ucc.asn.au>
date Wed, 18 Aug 2004 15:48:25 +0000
parents 5dda5a4d475c
children b855bf63766c
files cli-runopts.c dbutil.c dbutil.h debug.h dropbearconvert.c dropbearkey.c svr-runopts.c
diffstat 7 files changed, 45 insertions(+), 5 deletions(-) [+]
line wrap: on
line diff
--- a/cli-runopts.c	Wed Aug 18 12:42:21 2004 +0000
+++ b/cli-runopts.c	Wed Aug 18 15:48:25 2004 +0000
@@ -59,6 +59,9 @@
 					"-R <listenport:remotehost:remoteport> Remote port forwarding\n"
 #endif
 					"-l <username>\n"
+#ifdef DEBUG_TRACE
+					"-v    verbose\n"
+#endif
 					,DROPBEAR_VERSION, cli_opts.progname);
 }
 
@@ -178,6 +181,11 @@
 					printhelp();
 					exit(EXIT_SUCCESS);
 					break;
+#ifdef DEBUG_TRACE
+				case 'v':
+					debug_trace = 1;
+					break;
+#endif
 				case 'F':
 				case 'e':
 				case 'c':
--- a/dbutil.c	Wed Aug 18 12:42:21 2004 +0000
+++ b/dbutil.c	Wed Aug 18 15:48:25 2004 +0000
@@ -66,6 +66,10 @@
 void (*_dropbear_log)(int priority, const char* format, va_list param)
 						= generic_dropbear_log;
 
+#ifdef DEBUG_TRACE
+int debug_trace = 0;
+#endif
+
 int usingsyslog = 0; /* set by runopts, but required externally to sessions */
 #ifndef DISABLE_SYSLOG
 void startsyslog() {
@@ -134,6 +138,10 @@
 
 	va_list param;
 
+	if (!debug_trace) {
+		return;
+	}
+
 	va_start(param, format);
 	fprintf(stderr, "TRACE: ");
 	vfprintf(stderr, format, param);
--- a/dbutil.h	Wed Aug 18 12:42:21 2004 +0000
+++ b/dbutil.h	Wed Aug 18 15:48:25 2004 +0000
@@ -42,6 +42,7 @@
 #ifdef DEBUG_TRACE
 void dropbear_trace(const char* format, ...);
 void printhex(unsigned char* buf, int len);
+extern int debug_trace;
 #endif
 char * stripcontrol(const char * text);
 unsigned char * getaddrstring(struct sockaddr_storage* addr, int withport);
--- a/debug.h	Wed Aug 18 12:42:21 2004 +0000
+++ b/debug.h	Wed Aug 18 15:48:25 2004 +0000
@@ -33,9 +33,11 @@
  * etc. Don't use this normally, it might cause problems */
 /* #define DEBUG_VALGRIND */
 
-/* Define this to print trace statements - very verbose */
-/* Caution: Don't use this in an unfriendly environment (ie unfirewalled),
- * since the printing does not sanitise strings etc */
+/* Define this to compile in trace debugging printf()s. You'll need to 
+ * run programs with "-v" to turn this on.
+ * Caution: Don't use this in an unfriendly environment (ie unfirewalled),
+ * since the printing may not sanitise strings etc. This will add a reasonable
+ * amount to your executable size. */
 /* #define DEBUG_TRACE */
 
 /* All functions writing to the cleartext payload buffer call
--- a/dropbearconvert.c	Wed Aug 18 12:42:21 2004 +0000
+++ b/dropbearconvert.c	Wed Aug 18 15:48:25 2004 +0000
@@ -62,6 +62,11 @@
 	const char* infile;
 	const char* outfile;
 
+#ifdef DEBUG_TRACE
+	/* It's hard for it to get in the way _too_ much */
+	debug_trace = 1;
+#endif
+
 	/* get the commandline options */
 	if (argc != 5) {
 		fprintf(stderr, "All arguments must be specified\n");
--- a/dropbearkey.c	Wed Aug 18 12:42:21 2004 +0000
+++ b/dropbearkey.c	Wed Aug 18 15:48:25 2004 +0000
@@ -75,8 +75,11 @@
 #endif
 					"-f filename	Use filename for the secret key\n"
 					"-s bits	Key size in bits, should be a multiple of 8 (optional)\n"
-					"-y		Just print the publickey and fingerprint for the\n		private key in <filename>.\n",
-					progname);
+					"-y		Just print the publickey and fingerprint for the\n		private key in <filename>.\n"
+#ifdef DEBUG_TRACE
+					"-v		verbose\n"
+#endif
+					,progname);
 }
 
 #if defined(DBMULTI_dropbearkey) || !defined(DROPBEAR_MULTI)
@@ -127,6 +130,11 @@
 					printhelp(argv[0]);
 					exit(EXIT_SUCCESS);
 					break;
+#ifdef DEBUG_TRACE
+				case 'v':
+					debug_trace = 1;
+					break;
+#endif
 				default:
 					fprintf(stderr, "Unknown argument %s\n", argv[i]);
 					printhelp(argv[0]);
--- a/svr-runopts.c	Wed Aug 18 12:42:21 2004 +0000
+++ b/svr-runopts.c	Wed Aug 18 15:48:25 2004 +0000
@@ -76,6 +76,9 @@
 #ifdef INETD_MODE
 					"-i		Start for inetd\n"
 #endif
+#ifdef DEBUG_TRACE
+					"-v		verbose\n"
+#endif
 					,DROPBEAR_VERSION, progname,
 #ifdef DROPBEAR_DSS
 					DSS_PRIV_FILENAME,
@@ -193,6 +196,11 @@
 					printhelp(argv[0]);
 					exit(EXIT_FAILURE);
 					break;
+#ifdef DEBUG_TRACE
+				case 'v':
+					debug_trace = 1;
+					break;
+#endif
 					/*
 				case '4':
 					svr_opts.ipv4 = 0;