diff options
author | Andrew Tridgell <tridge@samba.org> | 1998-08-11 14:09:05 +0000 |
---|---|---|
committer | Andrew Tridgell <tridge@samba.org> | 1998-08-11 14:09:05 +0000 |
commit | ab1bfe5b0d9795d3e7d993ac5247f1be63950353 (patch) | |
tree | a8fd14b85a9bd1ff59dc621011c755916041bf4d | |
parent | e107f730824ce9ab9915dab4745e5eba1845bbe4 (diff) | |
download | samba-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.c | 2 |
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; } } |