[SCM] Samba Shared Repository - branch v3-5-test updated
Jeremy Allison
jra at samba.org
Thu Oct 22 16:27:38 MDT 2009
The branch, v3-5-test has been updated
via 2b02c97... Simplify the logic. Jeremy.
from 322f076... Fix bug 6829 - smbclient does not show special characters properly. All successful calls to cli_session_setup() *must* be followed by calls to cli_init_creds() to stash the credentials we successfully connected with. There were 2 codepaths where this was missing. This caused smbclient to be unable to open the \srvsvc pipe to do an RPC netserverenum, and cause it to fall back to a RAP netserverenum, which uses DOS codepage conversion rather than the full UCS2 of RPC, so the returned characters were not correct (unless the DOS codepage was set correctly). Phew. That was fun to track down :-). Jeremy.
http://gitweb.samba.org/?p=samba.git;a=shortlog;h=v3-5-test
- Log -----------------------------------------------------------------
commit 2b02c9786fc96277bc18bb088c202d3a71df8868
Author: Jeremy Allison <jra at samba.org>
Date: Thu Oct 22 15:26:22 2009 -0700
Simplify the logic.
Jeremy.
-----------------------------------------------------------------------
Summary of changes:
source3/libsmb/libsmb_server.c | 7 ++-----
1 files changed, 2 insertions(+), 5 deletions(-)
Changeset truncated at 500 lines:
diff --git a/source3/libsmb/libsmb_server.c b/source3/libsmb/libsmb_server.c
index 18b12a1..50ecee2 100644
--- a/source3/libsmb/libsmb_server.c
+++ b/source3/libsmb/libsmb_server.c
@@ -498,13 +498,10 @@ again:
errno = EPERM;
return NULL;
}
- status = cli_init_creds(c, username_used,
- *pp_workgroup, *pp_password);
- } else {
- status = cli_init_creds(c, username_used,
- *pp_workgroup, *pp_password);
}
+ status = cli_init_creds(c, username_used,
+ *pp_workgroup, *pp_password);
if (!NT_STATUS_IS_OK(status)) {
errno = map_errno_from_nt_status(status);
cli_shutdown(c);
--
Samba Shared Repository
More information about the samba-cvs
mailing list