[26980] in CVS-changelog-for-Kerberos-V5
krb5 commit [krb5-1.10]: Do not emit debug printfs under NODEBUG
daemon@ATHENA.MIT.EDU (Tom Yu)
Wed Aug 29 19:18:09 2012
Date: Wed, 29 Aug 2012 19:17:46 -0400
From: Tom Yu <tlyu@mit.edu>
Message-Id: <201208292317.q7TNHkjJ017522@drugstore.mit.edu>
To: cvs-krb5@mit.edu
Reply-To: krbdev@mit.edu
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Errors-To: cvs-krb5-bounces@mit.edu
https://github.com/krb5/krb5/commit/d6a17136ac0ed42ddaba73d18b39cd8f3cce0c63
commit d6a17136ac0ed42ddaba73d18b39cd8f3cce0c63
Author: Ben Kaduk <kaduk@mit.edu>
Date: Thu Aug 23 12:38:57 2012 -0400
Do not emit debug printfs under NODEBUG
These printfs spew to the console when command-line utilities
such as 'klist' and 'aklog' are run, reducing usability.
These printfs can also cause application hangs.
On a multiprocessor machine, when PuTTY and the ccapiserver are
running on different CPUs, PuTTY appears to deadlock with three
concurrent threads inside cci_debug_printf().
(cherry picked from commit 932866ad41edf830123bdfef927f1314f3100be9)
ticket: 7342
version_fixed: 1.10.4
status: resolved
src/ccapi/common/win/cci_os_debugging.c | 2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/src/ccapi/common/win/cci_os_debugging.c b/src/ccapi/common/win/cci_os_debugging.c
index de31c7c..597ac85 100644
--- a/src/ccapi/common/win/cci_os_debugging.c
+++ b/src/ccapi/common/win/cci_os_debugging.c
@@ -32,7 +32,9 @@
/* ------------------------------------------------------------------------ */
void cci_os_debug_vprintf (const char *in_format, va_list in_args) {
+#ifdef DEBUG
printf ( "%s %ld ", timestamp(), GetCurrentThreadId() );
vprintf ( in_format, in_args );
printf ( "\n" );
+#endif
}
_______________________________________________
cvs-krb5 mailing list
cvs-krb5@mit.edu
https://mailman.mit.edu/mailman/listinfo/cvs-krb5
