diff options
Diffstat (limited to 'source3/libsmb')
| -rw-r--r-- | source3/libsmb/libsmb_context.c | 8 | ||||
| -rw-r--r-- | source3/libsmb/libsmb_setget.c | 12 |
2 files changed, 15 insertions, 5 deletions
diff --git a/source3/libsmb/libsmb_context.c b/source3/libsmb/libsmb_context.c index 8e76387346a..22e0a047172 100644 --- a/source3/libsmb/libsmb_context.c +++ b/source3/libsmb/libsmb_context.c @@ -557,9 +557,11 @@ smbc_init_context(SMBCCTX *context) if (context->internal->debug_stderr) { /* - * Hmmm... Do we want a unique dbf per-thread? For now, we'll just - * leave it up to the user. If any one context spefies debug to - * stderr then all will be. + * We do not have a unique per-thread debug state? For + * now, we'll just leave it up to the user. If any one + * context spefies debug to stderr then all will be (and + * will stay that way, as it is unsafe to flip back if + * stdout is in use for other things) */ setup_logging("libsmbclient", DEBUG_STDERR); } diff --git a/source3/libsmb/libsmb_setget.c b/source3/libsmb/libsmb_setget.c index f2f5aec6c46..0d260d7cf29 100644 --- a/source3/libsmb/libsmb_setget.c +++ b/source3/libsmb/libsmb_setget.c @@ -120,10 +120,18 @@ smbc_setTimeout(SMBCCTX *c, int timeout) smbc_bool smbc_getOptionDebugToStderr(SMBCCTX *c) { - return c->internal->debug_stderr; + /* Because this is a global concept, it is better to check + * what is really set, rather than what we wanted set + * (particularly as you cannot go back to stdout). */ + return debug_get_output_is_stderr(); } -/** Set whether to log to standard error instead of standard output */ +/** Set whether to log to standard error instead of standard output. + * This option is 'sticky' - once set to true, it cannot be set to + * false again, as it is global to the process, as once we have been + * told that it is not safe to safe to write to stdout, we shouldn't + * go back as we don't know it was this context that set it that way. + */ void smbc_setOptionDebugToStderr(SMBCCTX *c, smbc_bool b) { |