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 | 5ae04012819ae3e63102d6875088ef00c27492b4 (patch) | |
tree | 735894c128d32f3351fc5edeb75f5c12d829ffa7 /source/lib | |
parent | 83122bae2253221f085118c6e7e914982656bd64 (diff) | |
download | samba-5ae04012819ae3e63102d6875088ef00c27492b4.tar.gz samba-5ae04012819ae3e63102d6875088ef00c27492b4.tar.xz samba-5ae04012819ae3e63102d6875088ef00c27492b4.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);
Diffstat (limited to 'source/lib')
-rw-r--r-- | source/lib/debug.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/source/lib/debug.c b/source/lib/debug.c index 02bf6710f55..8f7887006f3 100644 --- a/source/lib/debug.c +++ b/source/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; } } |