The branch, master has been updated via b5d5b9a libsmbclient: Fix cli_session_setup_guest_send from ed76e65 ntvfs: support setfileinfo with FULL_EA_INFORMATION
http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master - Log ----------------------------------------------------------------- commit b5d5b9a40eb13f89a552e29096deed52229c147e Author: Volker Lendecke <v...@samba.org> Date: Wed Apr 3 13:24:53 2013 +0200 libsmbclient: Fix cli_session_setup_guest_send This worked fine so far, but for the wrong reason: We only ever called this through the sync wrapper. The tevent_req_nterror in the NT_STATUS_OK case does not call tevent_req_finish. So the tevent_loop_poll did call into the core send/receive smb code. This is a fix for the case when smb1cli_req_chain_submit fails for some reason. Signed-off-by: Volker Lendecke <v...@samba.org> Reviewed-by: Stefan Metzmacher <me...@samba.org> Autobuild-User(master): Stefan Metzmacher <me...@samba.org> Autobuild-Date(master): Wed Apr 3 18:39:30 CEST 2013 on sn-devel-104 ----------------------------------------------------------------------- Summary of changes: source3/libsmb/cliconnect.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) Changeset truncated at 500 lines: diff --git a/source3/libsmb/cliconnect.c b/source3/libsmb/cliconnect.c index 872900c..ec5c273 100644 --- a/source3/libsmb/cliconnect.c +++ b/source3/libsmb/cliconnect.c @@ -446,7 +446,7 @@ struct tevent_req *cli_session_setup_guest_send(TALLOC_CTX *mem_ctx, } status = smb1cli_req_chain_submit(&subreq, 1); - if (NT_STATUS_IS_OK(status)) { + if (!NT_STATUS_IS_OK(status)) { tevent_req_nterror(req, status); return tevent_req_post(req, ev); } -- Samba Shared Repository