summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAndrew Tridgell <tridge@samba.org>1998-08-11 14:09:05 +0000
committerAndrew Tridgell <tridge@samba.org>1998-08-11 14:09:05 +0000
commitab1bfe5b0d9795d3e7d993ac5247f1be63950353 (patch)
treea8fd14b85a9bd1ff59dc621011c755916041bf4d
parente107f730824ce9ab9915dab4745e5eba1845bbe4 (diff)
downloadsamba-ab1bfe5b0d9795d3e7d993ac5247f1be63950353.tar.gz
samba-ab1bfe5b0d9795d3e7d993ac5247f1be63950353.tar.bz2
samba-ab1bfe5b0d9795d3e7d993ac5247f1be63950353.zip
fixed a nasty bug in debug.c
Debug1() was being called like this: Debug1( format_bufr ); but if format_bufr contains any %s or other % arguments (such as when processing a smb.conf file containing % macros) then smbd dies a horrible death. The quick fix is to use: Debug1( "%s", format_bufr); (This used to be commit 5ae04012819ae3e63102d6875088ef00c27492b4)
-rw-r--r--source3/lib/debug.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/source3/lib/debug.c b/source3/lib/debug.c
index 02bf6710f5..8f7887006f 100644
--- a/source3/lib/debug.c
+++ b/source3/lib/debug.c
@@ -422,7 +422,7 @@ static void format_debug_text( char *msg )
if( '\n' == msg[i] )
{
format_bufr[format_pos] = '\0';
- Debug1( format_bufr );
+ Debug1( "%s", format_bufr );
format_pos = 0;
}
}