summaryrefslogtreecommitdiff
path: root/source4
diff options
context:
space:
mode:
authorAndrew Bartlett <abartlet@samba.org>2010-05-14 17:31:33 +1000
committerAndrew Bartlett <abartlet@samba.org>2010-05-14 23:25:45 +1000
commit8da50c8da1527bafbb0ff4e86cc12fb54dd2b4b7 (patch)
tree7bc1b0b58c2d82f2d82a859e4c2e2891aaa368f9 /source4
parentad197f6f15054e398c0bf4f27388a86a384c09ba (diff)
downloadsamba-8da50c8da1527bafbb0ff4e86cc12fb54dd2b4b7.tar.gz
samba-8da50c8da1527bafbb0ff4e86cc12fb54dd2b4b7.tar.bz2
samba-8da50c8da1527bafbb0ff4e86cc12fb54dd2b4b7.zip
s4:winbindd Rework some winbind structures to make s3compat easier
By making the winbindd_request and winbindd_response structures pointers, we can more easily integrate with the winbindd from source3/winbindd Andrew Bartlett
Diffstat (limited to 'source4')
-rw-r--r--source4/winbind/wb_samba3_cmd.c320
-rw-r--r--source4/winbind/wb_samba3_protocol.c39
-rw-r--r--source4/winbind/wb_server.h4
3 files changed, 185 insertions, 178 deletions
diff --git a/source4/winbind/wb_samba3_cmd.c b/source4/winbind/wb_samba3_cmd.c
index c66d696cce..f0f7aa8526 100644
--- a/source4/winbind/wb_samba3_cmd.c
+++ b/source4/winbind/wb_samba3_cmd.c
@@ -61,48 +61,48 @@ static NTSTATUS wb_samba3_append_info3_as_txt(TALLOC_CTX *mem_ctx,
return ndr_map_error2ntstatus(ndr_err);
}
- s3call->response.data.auth.info3.logon_time =
+ s3call->response->data.auth.info3.logon_time =
nt_time_to_unix(info3->base.last_logon);
- s3call->response.data.auth.info3.logoff_time =
+ s3call->response->data.auth.info3.logoff_time =
nt_time_to_unix(info3->base.last_logoff);
- s3call->response.data.auth.info3.kickoff_time =
+ s3call->response->data.auth.info3.kickoff_time =
nt_time_to_unix(info3->base.acct_expiry);
- s3call->response.data.auth.info3.pass_last_set_time =
+ s3call->response->data.auth.info3.pass_last_set_time =
nt_time_to_unix(info3->base.last_password_change);
- s3call->response.data.auth.info3.pass_can_change_time =
+ s3call->response->data.auth.info3.pass_can_change_time =
nt_time_to_unix(info3->base.allow_password_change);
- s3call->response.data.auth.info3.pass_must_change_time =
+ s3call->response->data.auth.info3.pass_must_change_time =
nt_time_to_unix(info3->base.force_password_change);
- s3call->response.data.auth.info3.logon_count = info3->base.logon_count;
- s3call->response.data.auth.info3.bad_pw_count = info3->base.bad_password_count;
+ s3call->response->data.auth.info3.logon_count = info3->base.logon_count;
+ s3call->response->data.auth.info3.bad_pw_count = info3->base.bad_password_count;
- s3call->response.data.auth.info3.user_rid = info3->base.rid;
- s3call->response.data.auth.info3.group_rid = info3->base.primary_gid;
- fstrcpy(s3call->response.data.auth.info3.dom_sid, dom_sid_string(mem_ctx, info3->base.domain_sid));
+ s3call->response->data.auth.info3.user_rid = info3->base.rid;
+ s3call->response->data.auth.info3.group_rid = info3->base.primary_gid;
+ fstrcpy(s3call->response->data.auth.info3.dom_sid, dom_sid_string(mem_ctx, info3->base.domain_sid));
- s3call->response.data.auth.info3.num_groups = info3->base.groups.count;
- s3call->response.data.auth.info3.user_flgs = info3->base.user_flags;
+ s3call->response->data.auth.info3.num_groups = info3->base.groups.count;
+ s3call->response->data.auth.info3.user_flgs = info3->base.user_flags;
- s3call->response.data.auth.info3.acct_flags = info3->base.acct_flags;
- s3call->response.data.auth.info3.num_other_sids = info3->sidcount;
+ s3call->response->data.auth.info3.acct_flags = info3->base.acct_flags;
+ s3call->response->data.auth.info3.num_other_sids = info3->sidcount;
- fstrcpy(s3call->response.data.auth.info3.user_name,
+ fstrcpy(s3call->response->data.auth.info3.user_name,
info3->base.account_name.string);
- fstrcpy(s3call->response.data.auth.info3.full_name,
+ fstrcpy(s3call->response->data.auth.info3.full_name,
info3->base.full_name.string);
- fstrcpy(s3call->response.data.auth.info3.logon_script,
+ fstrcpy(s3call->response->data.auth.info3.logon_script,
info3->base.logon_script.string);
- fstrcpy(s3call->response.data.auth.info3.profile_path,
+ fstrcpy(s3call->response->data.auth.info3.profile_path,
info3->base.profile_path.string);
- fstrcpy(s3call->response.data.auth.info3.home_dir,
+ fstrcpy(s3call->response->data.auth.info3.home_dir,
info3->base.home_directory.string);
- fstrcpy(s3call->response.data.auth.info3.dir_drive,
+ fstrcpy(s3call->response->data.auth.info3.dir_drive,
info3->base.home_drive.string);
- fstrcpy(s3call->response.data.auth.info3.logon_srv,
+ fstrcpy(s3call->response->data.auth.info3.logon_srv,
info3->base.logon_server.string);
- fstrcpy(s3call->response.data.auth.info3.logon_dom,
+ fstrcpy(s3call->response->data.auth.info3.logon_dom,
info3->base.domain.string);
ex = talloc_strdup(mem_ctx, "");
@@ -129,8 +129,8 @@ static NTSTATUS wb_samba3_append_info3_as_txt(TALLOC_CTX *mem_ctx,
talloc_free(sid);
}
- s3call->response.extra_data.data = ex;
- s3call->response.length += talloc_get_size(ex);
+ s3call->response->extra_data.data = ex;
+ s3call->response->length += talloc_get_size(ex);
return NT_STATUS_OK;
}
@@ -142,7 +142,7 @@ static NTSTATUS wb_samba3_append_info3_as_txt(TALLOC_CTX *mem_ctx,
static void wbsrv_samba3_async_auth_epilogue(NTSTATUS status,
struct wbsrv_samba3_call *s3call)
{
- struct winbindd_response *resp = &s3call->response;
+ struct winbindd_response *resp = s3call->response;
if (!NT_STATUS_IS_OK(status)) {
resp->result = WINBINDD_ERROR;
} else {
@@ -167,7 +167,7 @@ static void wbsrv_samba3_async_auth_epilogue(NTSTATUS status,
static void wbsrv_samba3_async_epilogue(NTSTATUS status,
struct wbsrv_samba3_call *s3call)
{
- struct winbindd_response *resp = &s3call->response;
+ struct winbindd_response *resp = s3call->response;
if (NT_STATUS_IS_OK(status)) {
resp->result = WINBINDD_OK;
} else {
@@ -183,32 +183,32 @@ static void wbsrv_samba3_async_epilogue(NTSTATUS status,
NTSTATUS wbsrv_samba3_interface_version(struct wbsrv_samba3_call *s3call)
{
- s3call->response.result = WINBINDD_OK;
- s3call->response.data.interface_version = WINBIND_INTERFACE_VERSION;
+ s3call->response->result = WINBINDD_OK;
+ s3call->response->data.interface_version = WINBIND_INTERFACE_VERSION;
return NT_STATUS_OK;
}
NTSTATUS wbsrv_samba3_info(struct wbsrv_samba3_call *s3call)
{
- s3call->response.result = WINBINDD_OK;
- s3call->response.data.info.winbind_separator = *lp_winbind_separator(s3call->wbconn->lp_ctx);
- WBSRV_SAMBA3_SET_STRING(s3call->response.data.info.samba_version,
+ s3call->response->result = WINBINDD_OK;
+ s3call->response->data.info.winbind_separator = *lp_winbind_separator(s3call->wbconn->lp_ctx);
+ WBSRV_SAMBA3_SET_STRING(s3call->response->data.info.samba_version,
SAMBA_VERSION_STRING);
return NT_STATUS_OK;
}
NTSTATUS wbsrv_samba3_domain_name(struct wbsrv_samba3_call *s3call)
{
- s3call->response.result = WINBINDD_OK;
- WBSRV_SAMBA3_SET_STRING(s3call->response.data.domain_name,
+ s3call->response->result = WINBINDD_OK;
+ WBSRV_SAMBA3_SET_STRING(s3call->response->data.domain_name,
lp_workgroup(s3call->wbconn->lp_ctx));
return NT_STATUS_OK;
}
NTSTATUS wbsrv_samba3_netbios_name(struct wbsrv_samba3_call *s3call)
{
- s3call->response.result = WINBINDD_OK;
- WBSRV_SAMBA3_SET_STRING(s3call->response.data.netbios_name,
+ s3call->response->result = WINBINDD_OK;
+ WBSRV_SAMBA3_SET_STRING(s3call->response->data.netbios_name,
lp_netbios_name(s3call->wbconn->lp_ctx));
return NT_STATUS_OK;
}
@@ -218,31 +218,31 @@ NTSTATUS wbsrv_samba3_priv_pipe_dir(struct wbsrv_samba3_call *s3call)
struct loadparm_context *lp_ctx = s3call->wbconn->listen_socket->service->task->lp_ctx;
const char *priv_socket_dir = lp_winbindd_privileged_socket_directory(lp_ctx);
- s3call->response.result = WINBINDD_OK;
- s3call->response.extra_data.data = discard_const(priv_socket_dir);
+ s3call->response->result = WINBINDD_OK;
+ s3call->response->extra_data.data = discard_const(priv_socket_dir);
- s3call->response.length += strlen(priv_socket_dir) + 1;
+ s3call->response->length += strlen(priv_socket_dir) + 1;
return NT_STATUS_OK;
}
NTSTATUS wbsrv_samba3_ping(struct wbsrv_samba3_call *s3call)
{
- s3call->response.result = WINBINDD_OK;
+ s3call->response->result = WINBINDD_OK;
return NT_STATUS_OK;
}
NTSTATUS wbsrv_samba3_domain_info(struct wbsrv_samba3_call *s3call)
{
DEBUG(5, ("wbsrv_samba3_domain_info called, stub\n"));
- s3call->response.result = WINBINDD_OK;
- fstrcpy(s3call->response.data.domain_info.name,
- s3call->request.domain_name);
- fstrcpy(s3call->response.data.domain_info.alt_name,
- s3call->request.domain_name);
- fstrcpy(s3call->response.data.domain_info.sid, "S-1-2-3-4");
- s3call->response.data.domain_info.native_mode = false;
- s3call->response.data.domain_info.active_directory = false;
- s3call->response.data.domain_info.primary = false;
+ s3call->response->result = WINBINDD_OK;
+ fstrcpy(s3call->response->data.domain_info.name,
+ s3call->request->domain_name);
+ fstrcpy(s3call->response->data.domain_info.alt_name,
+ s3call->request->domain_name);
+ fstrcpy(s3call->response->data.domain_info.sid, "S-1-2-3-4");
+ s3call->response->data.domain_info.native_mode = false;
+ s3call->response->data.domain_info.active_directory = false;
+ s3call->response->data.domain_info.primary = false;
return NT_STATUS_OK;
}
@@ -323,7 +323,7 @@ NTSTATUS wbsrv_samba3_getdcname(struct wbsrv_samba3_call *s3call)
DEBUG(5, ("wbsrv_samba3_getdcname called\n"));
ctx = wb_cmd_getdcname_send(s3call, service,
- s3call->request.domain_name);
+ s3call->request->domain_name);
NT_STATUS_HAVE_NO_MEMORY(ctx);
ctx->async.fn = getdcname_recv_dc;
@@ -343,8 +343,8 @@ static void getdcname_recv_dc(struct composite_context *ctx)
status = wb_cmd_getdcname_recv(ctx, s3call, &dcname);
if (!NT_STATUS_IS_OK(status)) goto done;
- s3call->response.result = WINBINDD_OK;
- WBSRV_SAMBA3_SET_STRING(s3call->response.data.dc_name, dcname);
+ s3call->response->result = WINBINDD_OK;
+ WBSRV_SAMBA3_SET_STRING(s3call->response->data.dc_name, dcname);
done:
wbsrv_samba3_async_epilogue(status, s3call);
@@ -363,10 +363,10 @@ NTSTATUS wbsrv_samba3_userdomgroups(struct wbsrv_samba3_call *s3call)
DEBUG(5, ("wbsrv_samba3_userdomgroups called\n"));
- sid = dom_sid_parse_talloc(s3call, s3call->request.data.sid);
+ sid = dom_sid_parse_talloc(s3call, s3call->request->data.sid);
if (sid == NULL) {
DEBUG(5, ("Could not parse sid %s\n",
- s3call->request.data.sid));
+ s3call->request->data.sid));
return NT_STATUS_NO_MEMORY;
}
@@ -409,10 +409,10 @@ static void userdomgroups_recv_groups(struct composite_context *ctx)
goto done;
}
- s3call->response.result = WINBINDD_OK;
- s3call->response.extra_data.data = sids_string;
- s3call->response.length += strlen(sids_string)+1;
- s3call->response.data.num_entries = num_sids;
+ s3call->response->result = WINBINDD_OK;
+ s3call->response->extra_data.data = sids_string;
+ s3call->response->length += strlen(sids_string)+1;
+ s3call->response->data.num_entries = num_sids;
done:
wbsrv_samba3_async_epilogue(status, s3call);
@@ -430,10 +430,10 @@ NTSTATUS wbsrv_samba3_usersids(struct wbsrv_samba3_call *s3call)
DEBUG(5, ("wbsrv_samba3_usersids called\n"));
- sid = dom_sid_parse_talloc(s3call, s3call->request.data.sid);
+ sid = dom_sid_parse_talloc(s3call, s3call->request->data.sid);
if (sid == NULL) {
DEBUG(5, ("Could not parse sid %s\n",
- s3call->request.data.sid));
+ s3call->request->data.sid));
return NT_STATUS_NO_MEMORY;
}
@@ -475,10 +475,10 @@ static void usersids_recv_sids(struct composite_context *ctx)
}
}
- s3call->response.result = WINBINDD_OK;
- s3call->response.extra_data.data = sids_string;
- s3call->response.length += strlen(sids_string);
- s3call->response.data.num_entries = num_sids;
+ s3call->response->result = WINBINDD_OK;
+ s3call->response->extra_data.data = sids_string;
+ s3call->response->length += strlen(sids_string);
+ s3call->response->data.num_entries = num_sids;
/* Hmmmm. Nasty protocol -- who invented the zeros between the
* SIDs? Hmmm. Could have been me -- vl */
@@ -509,8 +509,8 @@ NTSTATUS wbsrv_samba3_lookupname(struct wbsrv_samba3_call *s3call)
DEBUG(5, ("wbsrv_samba3_lookupname called\n"));
ctx = wb_cmd_lookupname_send(s3call, service,
- s3call->request.data.name.dom_name,
- s3call->request.data.name.name);
+ s3call->request->data.name.dom_name,
+ s3call->request->data.name.name);
NT_STATUS_HAVE_NO_MEMORY(ctx);
/* setup the callbacks */
@@ -531,9 +531,9 @@ static void lookupname_recv_sid(struct composite_context *ctx)
status = wb_cmd_lookupname_recv(ctx, s3call, &sid);
if (!NT_STATUS_IS_OK(status)) goto done;
- s3call->response.result = WINBINDD_OK;
- s3call->response.data.sid.type = sid->type;
- WBSRV_SAMBA3_SET_STRING(s3call->response.data.sid.sid,
+ s3call->response->result = WINBINDD_OK;
+ s3call->response->data.sid.type = sid->type;
+ WBSRV_SAMBA3_SET_STRING(s3call->response->data.sid.sid,
dom_sid_string(s3call, sid->sid));
done:
@@ -555,10 +555,10 @@ NTSTATUS wbsrv_samba3_lookupsid(struct wbsrv_samba3_call *s3call)
DEBUG(5, ("wbsrv_samba3_lookupsid called\n"));
- sid = dom_sid_parse_talloc(s3call, s3call->request.data.sid);
+ sid = dom_sid_parse_talloc(s3call, s3call->request->data.sid);
if (sid == NULL) {
DEBUG(5, ("Could not parse sid %s\n",
- s3call->request.data.sid));
+ s3call->request->data.sid));
return NT_STATUS_NO_MEMORY;
}
@@ -583,11 +583,11 @@ static void lookupsid_recv_name(struct composite_context *ctx)
status = wb_cmd_lookupsid_recv(ctx, s3call, &sid);
if (!NT_STATUS_IS_OK(status)) goto done;
- s3call->response.result = WINBINDD_OK;
- s3call->response.data.name.type = sid->type;
- WBSRV_SAMBA3_SET_STRING(s3call->response.data.name.dom_name,
+ s3call->response->result = WINBINDD_OK;
+ s3call->response->data.name.type = sid->type;
+ WBSRV_SAMBA3_SET_STRING(s3call->response->data.name.dom_name,
sid->domain);
- WBSRV_SAMBA3_SET_STRING(s3call->response.data.name.name, sid->name);
+ WBSRV_SAMBA3_SET_STRING(s3call->response->data.name.name, sid->name);
done:
wbsrv_samba3_async_epilogue(status, s3call);
@@ -599,7 +599,7 @@ static void lookupsid_recv_name(struct composite_context *ctx)
NTSTATUS wbsrv_samba3_pam_logoff(struct wbsrv_samba3_call *s3call)
{
NTSTATUS status;
- struct winbindd_response *resp = &s3call->response;
+ struct winbindd_response *resp = s3call->response;
status = NT_STATUS_OK;
@@ -640,19 +640,19 @@ NTSTATUS wbsrv_samba3_pam_auth_crap(struct wbsrv_samba3_call *s3call)
DEBUG(5, ("wbsrv_samba3_pam_auth_crap called\n"));
- chal.data = s3call->request.data.auth_crap.chal;
- chal.length = sizeof(s3call->request.data.auth_crap.chal);
- nt_resp.data = (uint8_t *)s3call->request.data.auth_crap.nt_resp;
- nt_resp.length = s3call->request.data.auth_crap.nt_resp_len;
- lm_resp.data = (uint8_t *)s3call->request.data.auth_crap.lm_resp;
- lm_resp.length = s3call->request.data.auth_crap.lm_resp_len;
+ chal.data = s3call->request->data.auth_crap.chal;
+ chal.length = sizeof(s3call->request->data.auth_crap.chal);
+ nt_resp.data = (uint8_t *)s3call->request->data.auth_crap.nt_resp;
+ nt_resp.length = s3call->request->data.auth_crap.nt_resp_len;
+ lm_resp.data = (uint8_t *)s3call->request->data.auth_crap.lm_resp;
+ lm_resp.length = s3call->request->data.auth_crap.lm_resp_len;
ctx = wb_cmd_pam_auth_crap_send(
s3call, service,
- s3call->request.data.auth_crap.logon_parameters,
- s3call->request.data.auth_crap.domain,
- s3call->request.data.auth_crap.user,
- s3call->request.data.auth_crap.workstation,
+ s3call->request->data.auth_crap.logon_parameters,
+ s3call->request->data.auth_crap.domain,
+ s3call->request->data.auth_crap.user,
+ s3call->request->data.auth_crap.workstation,
chal, nt_resp, lm_resp);
NT_STATUS_HAVE_NO_MEMORY(ctx);
@@ -677,13 +677,13 @@ static void pam_auth_crap_recv(struct composite_context *ctx)
&user_session_key, &lm_key, &unix_username);
if (!NT_STATUS_IS_OK(status)) goto done;
- if (s3call->request.flags & WBFLAG_PAM_USER_SESSION_KEY) {
- memcpy(s3call->response.data.auth.user_session_key,
+ if (s3call->request->flags & WBFLAG_PAM_USER_SESSION_KEY) {
+ memcpy(s3call->response->data.auth.user_session_key,
&user_session_key.key,
- sizeof(s3call->response.data.auth.user_session_key));
+ sizeof(s3call->response->data.auth.user_session_key));
}
- if (s3call->request.flags & WBFLAG_PAM_INFO3_TEXT) {
+ if (s3call->request->flags & WBFLAG_PAM_INFO3_TEXT) {
status = wb_samba3_append_info3_as_txt(ctx, s3call, info3);
if (!NT_STATUS_IS_OK(status)) {
DEBUG(10,("Failed to append INFO3 (TXT): %s\n",
@@ -692,19 +692,19 @@ static void pam_auth_crap_recv(struct composite_context *ctx)
}
}
- if (s3call->request.flags & WBFLAG_PAM_INFO3_NDR) {
- s3call->response.extra_data.data = info3.data;
- s3call->response.length += info3.length;
+ if (s3call->request->flags & WBFLAG_PAM_INFO3_NDR) {
+ s3call->response->extra_data.data = info3.data;
+ s3call->response->length += info3.length;
}
- if (s3call->request.flags & WBFLAG_PAM_LMKEY) {
- memcpy(s3call->response.data.auth.first_8_lm_hash,
+ if (s3call->request->flags & WBFLAG_PAM_LMKEY) {
+ memcpy(s3call->response->data.auth.first_8_lm_hash,
lm_key.key,
- sizeof(s3call->response.data.auth.first_8_lm_hash));
+ sizeof(s3call->response->data.auth.first_8_lm_hash));
}
- if (s3call->request.flags & WBFLAG_PAM_UNIX_NAME) {
- WBSRV_SAMBA3_SET_STRING(s3call->response.data.auth.unix_username,unix_username);
+ if (s3call->request->flags & WBFLAG_PAM_UNIX_NAME) {
+ WBSRV_SAMBA3_SET_STRING(s3call->response->data.auth.unix_username,unix_username);
}
done:
@@ -729,7 +729,7 @@ NTSTATUS wbsrv_samba3_pam_auth(struct wbsrv_samba3_call *s3call)
char *user, *domain;
if (!wb_samba3_split_username(s3call, s3call->wbconn->lp_ctx,
- s3call->request.data.auth.user,
+ s3call->request->data.auth.user,
&domain, &user)) {
return NT_STATUS_NO_SUCH_USER;
}
@@ -742,7 +742,7 @@ NTSTATUS wbsrv_samba3_pam_auth(struct wbsrv_samba3_call *s3call)
cli_credentials_set_domain(credentials, domain, CRED_SPECIFIED);
cli_credentials_set_username(credentials, user, CRED_SPECIFIED);
- cli_credentials_set_password(credentials, s3call->request.data.auth.pass, CRED_SPECIFIED);
+ cli_credentials_set_password(credentials, s3call->request->data.auth.pass, CRED_SPECIFIED);
ctx = wb_cmd_pam_auth_send(s3call, service, credentials);
NT_STATUS_HAVE_NO_MEMORY(ctx);
@@ -769,13 +769,13 @@ static void pam_auth_recv(struct composite_context *ctx)
if (!NT_STATUS_IS_OK(status)) goto done;
- if (s3call->request.flags & WBFLAG_PAM_USER_SESSION_KEY) {
- memcpy(s3call->response.data.auth.user_session_key,
+ if (s3call->request->flags & WBFLAG_PAM_USER_SESSION_KEY) {
+ memcpy(s3call->response->data.auth.user_session_key,
&user_session_key.key,
- sizeof(s3call->response.data.auth.user_session_key));
+ sizeof(s3call->response->data.auth.user_session_key));
}
- if (s3call->request.flags & WBFLAG_PAM_INFO3_TEXT) {
+ if (s3call->request->flags & WBFLAG_PAM_INFO3_TEXT) {
status = wb_samba3_append_info3_as_txt(ctx, s3call, info3);
if (!NT_STATUS_IS_OK(status)) {
DEBUG(10,("Failed to append INFO3 (TXT): %s\n",
@@ -784,19 +784,19 @@ static void pam_auth_recv(struct composite_context *ctx)
}
}
- if (s3call->request.flags & WBFLAG_PAM_INFO3_NDR) {
- s3call->response.extra_data.data = info3.data;
- s3call->response.length += info3.length;
+ if (s3call->request->flags & WBFLAG_PAM_INFO3_NDR) {
+ s3call->response->extra_data.data = info3.data;
+ s3call->response->length += info3.length;
}
- if (s3call->request.flags & WBFLAG_PAM_LMKEY) {
- memcpy(s3call->response.data.auth.first_8_lm_hash,
+ if (s3call->request->flags & WBFLAG_PAM_LMKEY) {
+ memcpy(s3call->response->data.auth.first_8_lm_hash,
lm_key.key,
- sizeof(s3call->response.data.auth.first_8_lm_hash));
+ sizeof(s3call->response->data.auth.first_8_lm_hash));
}
- if (s3call->request.flags & WBFLAG_PAM_UNIX_NAME) {
- WBSRV_SAMBA3_SET_STRING(s3call->response.data.auth.unix_username,unix_username);
+ if (s3call->request->flags & WBFLAG_PAM_UNIX_NAME) {
+ WBSRV_SAMBA3_SET_STRING(s3call->response->data.auth.unix_username,unix_username);
}
@@ -859,11 +859,11 @@ static void list_trustdom_recv_doms(struct composite_context *ctx)
goto done;
}
- s3call->response.result = WINBINDD_OK;
+ s3call->response->result = WINBINDD_OK;
if (num_domains > 0) {
- s3call->response.extra_data.data = result;
- s3call->response.length += strlen(result)+1;
- s3call->response.data.num_entries = num_domains;
+ s3call->response->extra_data.data = result;
+ s3call->response->length += strlen(result)+1;
+ s3call->response->data.num_entries = num_domains;
}
done:
@@ -881,7 +881,7 @@ NTSTATUS wbsrv_samba3_list_groups(struct wbsrv_samba3_call *s3call)
DEBUG(5, ("wbsrv_samba4_list_groups called\n"));
ctx = wb_cmd_list_groups_send(s3call, service,
- s3call->request.domain_name);
+ s3call->request->domain_name);
NT_STATUS_HAVE_NO_MEMORY(ctx);
ctx->async.fn = list_groups_recv;
@@ -906,11 +906,11 @@ static void list_groups_recv(struct composite_context *ctx)
&extra_data, &num_groups);
if (NT_STATUS_IS_OK(status)) {
- s3call->response.extra_data.data = extra_data;
- s3call->response.length += extra_data_len;
+ s3call->response->extra_data.data = extra_data;
+ s3call->response->length += extra_data_len;
if (extra_data) {
- s3call->response.length += 1;
- s3call->response.data.num_entries = num_groups;
+ s3call->response->length += 1;
+ s3call->response->data.num_entries = num_groups;
}
}
@@ -930,7 +930,7 @@ NTSTATUS wbsrv_samba3_list_users(struct wbsrv_samba3_call *s3call)
DEBUG(5, ("wbsrv_samba3_list_users called\n"));
ctx = wb_cmd_list_users_send(s3call, service,
- s3call->request.domain_name);
+ s3call->request->domain_name);
NT_STATUS_HAVE_NO_MEMORY(ctx);
ctx->async.fn = list_users_recv;
@@ -955,11 +955,11 @@ static void list_users_recv(struct composite_context *ctx)
&extra_data, &num_users);
if (NT_STATUS_IS_OK(status)) {
- s3call->response.extra_data.data = extra_data;
- s3call->response.length += extra_data_len;
+ s3call->response->extra_data.data = extra_data;
+ s3call->response->length += extra_data_len;
if (extra_data) {
- s3call->response.length += 1;
- s3call->response.data.num_entries = num_users;
+ s3call->response->length += 1;
+ s3call->response->data.num_entries = num_users;
}
}
@@ -979,7 +979,7 @@ NTSTATUS wbsrv_samba3_getpwnam(struct wbsrv_samba3_call *s3call)
DEBUG(5, ("wbsrv_samba3_getpwnam called\n"));
ctx = wb_cmd_getpwnam_send(s3call, service,
- s3call->request.data.username);
+ s3call->request->data.username);
NT_STATUS_HAVE_NO_MEMORY(ctx);
ctx->async.fn = getpwnam_recv;
@@ -1000,7 +1000,7 @@ static void getpwnam_recv(struct composite_context *ctx)
status = wb_cmd_getpwnam_recv(ctx, s3call, &pw);
if(NT_STATUS_IS_OK(status))
- s3call->response.data.pw = *pw;
+ s3call->response->data.pw = *pw;
wbsrv_samba3_async_epilogue(status, s3call);
}
@@ -1015,7 +1015,7 @@ NTSTATUS wbsrv_samba3_getpwuid(struct wbsrv_samba3_call *s3call)
DEBUG(5, ("wbsrv_samba3_getpwuid called\n"));
ctx = wb_cmd_getpwuid_send(s3call, service,
- s3call->request.data.uid);
+ s3call->request->data.uid);
NT_STATUS_HAVE_NO_MEMORY(ctx);
ctx->async.fn = getpwuid_recv;
@@ -1036,7 +1036,7 @@ static void getpwuid_recv(struct composite_context *ctx)
status = wb_cmd_getpwuid_recv(ctx, s3call, &pw);
if (NT_STATUS_IS_OK(status))
- s3call->response.data.pw = *pw;
+ s3call->response->data.pw = *pw;
wbsrv_samba3_async_epilogue(status, s3call);
}
@@ -1094,7 +1094,7 @@ NTSTATUS wbsrv_samba3_getpwent(struct wbsrv_samba3_call *s3call)
NT_STATUS_HAVE_NO_MEMORY(pwent);
ctx = wb_cmd_getpwent_send(s3call, service, pwent,
- s3call->request.data.num_entries);
+ s3call->request->data.num_entries);
NT_STATUS_HAVE_NO_MEMORY(ctx);
ctx->async.fn = getpwent_recv;
@@ -1118,9 +1118,9 @@ static void getpwent_recv(struct composite_context *ctx)
if (NT_STATUS_IS_OK(status)) {
uint32_t extra_len = sizeof(struct winbindd_pw) * num_users;
- s3call->response.data.num_entries = num_users;
- s3call->response.extra_data.data = pw;
- s3call->response.length += extra_len;
+ s3call->response->data.num_entries = num_users;
+ s3call->response->extra_data.data = pw;
+ s3call->response->length += extra_len;
}
wbsrv_samba3_async_epilogue(status, s3call);
@@ -1136,7 +1136,7 @@ NTSTATUS wbsrv_samba3_endpwent(struct wbsrv_samba3_call *s3call)
talloc_free(pwent);
s3call->wbconn->protocol_private_data = NULL;
- s3call->response.result = WINBINDD_OK;
+ s3call->response->result = WINBINDD_OK;
return NT_STATUS_OK;
}
@@ -1152,7 +1152,7 @@ NTSTATUS wbsrv_samba3_getgrnam(struct wbsrv_samba3_call *s3call)
DEBUG(5, ("wbsrv_samba3_getgrnam called\n"));
ctx = wb_cmd_getgrnam_send(s3call, service,
- s3call->request.data.groupname);
+ s3call->request->data.groupname);
NT_STATUS_HAVE_NO_MEMORY(ctx);
ctx->async.fn = getgrnam_recv;
@@ -1173,7 +1173,7 @@ static void getgrnam_recv(struct composite_context *ctx)
status = wb_cmd_getgrnam_recv(ctx, s3call, &gr);
if(NT_STATUS_IS_OK(status))
- s3call->response.data.gr = *gr;
+ s3call->response->data.gr = *gr;
wbsrv_samba3_async_epilogue(status, s3call);
}
@@ -1188,7 +1188,7 @@ NTSTATUS wbsrv_samba3_getgrgid(struct wbsrv_samba3_call *s3call)
DEBUG(5, ("wbsrv_samba3_getgrgid called\n"));
ctx = wb_cmd_getgrgid_send(s3call, service,
- s3call->request.data.gid);
+ s3call->request->data.gid);
NT_STATUS_HAVE_NO_MEMORY(ctx);
ctx->async.fn = getgrgid_recv;
@@ -1209,7 +1209,7 @@ static void getgrgid_recv(struct composite_context *ctx)
status = wb_cmd_getgrgid_recv(ctx, s3call, &gr);
if (NT_STATUS_IS_OK(status))
- s3call->response.data.gr = *gr;
+ s3call->response->data.gr = *gr;
wbsrv_samba3_async_epilogue(status, s3call);
}
@@ -1223,8 +1223,8 @@ NTSTATUS wbsrv_samba3_getgroups(struct wbsrv_samba3_call *s3call)
DEBUG(5, ("wbsrv_samba3_getgroups called\n"));
/* S3 code do the same so why not ... */
- s3call->request.data.username[sizeof(s3call->request.data.username)-1]='\0';
- ctx = wb_cmd_getgroups_send(s3call, service, s3call->request.data.username);
+ s3call->request->data.username[sizeof(s3call->request->data.username)-1]='\0';
+ ctx = wb_cmd_getgroups_send(s3call, service, s3call->request->data.username);
NT_STATUS_HAVE_NO_MEMORY(ctx);
ctx->async.fn = getgroups_recv;
@@ -1247,11 +1247,11 @@ static void getgroups_recv(struct composite_context *ctx)
if (NT_STATUS_IS_OK(status)) {
uint32_t extra_len = sizeof(gid_t) * num_groups;
- s3call->response.data.num_entries = num_groups;
- s3call->response.extra_data.data = gids;
- s3call->response.length += extra_len;
+ s3call->response->data.num_entries = num_groups;
+ s3call->response->extra_data.data = gids;
+ s3call->response->length += extra_len;
} else {
- s3call->response.result = WINBINDD_ERROR;
+ s3call->response->result = WINBINDD_ERROR;
}
wbsrv_samba3_async_epilogue(status, s3call);
@@ -1310,7 +1310,7 @@ NTSTATUS wbsrv_samba3_getgrent(struct wbsrv_samba3_call *s3call)
NT_STATUS_HAVE_NO_MEMORY(grent);
ctx = wb_cmd_getgrent_send(s3call, service, grent,
- s3call->request.data.num_entries);
+ s3call->request->data.num_entries);
NT_STATUS_HAVE_NO_MEMORY(ctx);
ctx->async.fn = getgrent_recv;
@@ -1334,9 +1334,9 @@ static void getgrent_recv(struct composite_context *ctx)
if (NT_STATUS_IS_OK(status)) {
uint32_t extra_len = sizeof(struct winbindd_gr) * num_groups;
- s3call->response.data.num_entries = num_groups;
- s3call->response.extra_data.data = gr;
- s3call->response.length += extra_len;
+ s3call->response->data.num_entries = num_groups;
+ s3call->response->extra_data.data = gr;
+ s3call->response->length += extra_len;
}
wbsrv_samba3_async_epilogue(status, s3call);
@@ -1345,7 +1345,7 @@ static void getgrent_recv(struct composite_context *ctx)
NTSTATUS wbsrv_samba3_endgrent(struct wbsrv_samba3_call *s3call)
{
DEBUG(5, ("wbsrv_samba3_endgrent called\n"));
- s3call->response.result = WINBINDD_OK;
+ s3call->response->result = WINBINDD_OK;
return NT_STATUS_OK;
}
@@ -1360,7 +1360,7 @@ NTSTATUS wbsrv_samba3_sid2uid(struct wbsrv_samba3_call *s3call)
DEBUG(5, ("wbsrv_samba3_sid2uid called\n"));
- sid = dom_sid_parse_talloc(s3call, s3call->request.data.sid);
+ sid = dom_sid_parse_talloc(s3call, s3call->request->data.sid);
NT_STATUS_HAVE_NO_MEMORY(sid);
ctx = wb_sid2uid_send(s3call, service, sid);
@@ -1382,7 +1382,7 @@ static void sid2uid_recv(struct composite_context *ctx)
DEBUG(5, ("sid2uid_recv called\n"));
- status = wb_sid2uid_recv(ctx, &s3call->response.data.uid);
+ status = wb_sid2uid_recv(ctx, &s3call->response->data.uid);
wbsrv_samba3_async_epilogue(status, s3call);
}
@@ -1398,7 +1398,7 @@ NTSTATUS wbsrv_samba3_sid2gid(struct wbsrv_samba3_call *s3call)
DEBUG(5, ("wbsrv_samba3_sid2gid called\n"));
- sid = dom_sid_parse_talloc(s3call, s3call->request.data.sid);
+ sid = dom_sid_parse_talloc(s3call, s3call->request->data.sid);
NT_STATUS_HAVE_NO_MEMORY(sid);
ctx = wb_sid2gid_send(s3call, service, sid);
@@ -1420,7 +1420,7 @@ static void sid2gid_recv(struct composite_context *ctx)
DEBUG(5, ("sid2gid_recv called\n"));
- status = wb_sid2gid_recv(ctx, &s3call->response.data.gid);
+ status = wb_sid2gid_recv(ctx, &s3call->response->data.gid);
wbsrv_samba3_async_epilogue(status, s3call);
}
@@ -1435,7 +1435,7 @@ NTSTATUS wbsrv_samba3_uid2sid(struct wbsrv_samba3_call *s3call)
DEBUG(5, ("wbsrv_samba3_uid2sid called\n"));
- ctx = wb_uid2sid_send(s3call, service, s3call->request.data.uid);
+ ctx = wb_uid2sid_send(s3call, service, s3call->request->data.uid);
NT_STATUS_HAVE_NO_MEMORY(ctx);
ctx->async.fn = uid2sid_recv;
@@ -1466,8 +1466,8 @@ static void uid2sid_recv(struct composite_context *ctx)
/* But we assume this worked, so we'll set the string. Work
* done. */
- WBSRV_SAMBA3_SET_STRING(s3call->response.data.sid.sid, sid_str);
- s3call->response.data.sid.type = SID_NAME_USER;
+ WBSRV_SAMBA3_SET_STRING(s3call->response->data.sid.sid, sid_str);
+ s3call->response->data.sid.type = SID_NAME_USER;
}
wbsrv_samba3_async_epilogue(status, s3call);
@@ -1483,7 +1483,7 @@ NTSTATUS wbsrv_samba3_gid2sid(struct wbsrv_samba3_call *s3call)
DEBUG(5, ("wbsrv_samba3_gid2sid called\n"));
- ctx = wb_gid2sid_send(s3call, service, s3call->request.data.gid);
+ ctx = wb_gid2sid_send(s3call, service, s3call->request->data.gid);
NT_STATUS_HAVE_NO_MEMORY(ctx);
ctx->async.fn = gid2sid_recv;
@@ -1511,8 +1511,8 @@ static void gid2sid_recv(struct composite_context *ctx)
if (sid_str == NULL)
wbsrv_samba3_async_epilogue(NT_STATUS_NO_MEMORY,s3call);
- WBSRV_SAMBA3_SET_STRING(s3call->response.data.sid.sid, sid_str);
- s3call->response.data.sid.type = SID_NAME_DOMAIN;
+ WBSRV_SAMBA3_SET_STRING(s3call->response->data.sid.sid, sid_str);
+ s3call->response->data.sid.type = SID_NAME_DOMAIN;
}
wbsrv_samba3_async_epilogue(status, s3call);
diff --git a/source4/winbind/wb_samba3_protocol.c b/source4/winbind/wb_samba3_protocol.c
index 88eecdb4ff..8d0127ba04 100644
--- a/source4/winbind/wb_samba3_protocol.c
+++ b/source4/winbind/wb_samba3_protocol.c
@@ -46,26 +46,32 @@ NTSTATUS wbsrv_samba3_packet_full_request(void *private_data, DATA_BLOB blob, si
NTSTATUS wbsrv_samba3_pull_request(struct wbsrv_samba3_call *call)
{
- if (call->in.length != sizeof(call->request)) {
+ if (call->in.length != sizeof(*call->request)) {
DEBUG(0,("wbsrv_samba3_pull_request: invalid blob length %lu should be %lu\n"
" make sure you use the correct winbind client tools!\n",
- (long)call->in.length, (long)sizeof(call->request)));
+ (long)call->in.length, (long)sizeof(*call->request)));
return NT_STATUS_INVALID_PARAMETER;
}
+ call->request = talloc_zero(call, struct winbindd_request);
+ NT_STATUS_HAVE_NO_MEMORY(call->request);
+
/* the packet layout is the same as the in memory layout of the request, so just copy it */
- memcpy(&call->request, call->in.data, sizeof(call->request));
+ memcpy(call->request, call->in.data, sizeof(*call->request));
return NT_STATUS_OK;
}
NTSTATUS wbsrv_samba3_handle_call(struct wbsrv_samba3_call *s3call)
{
- DEBUG(10, ("Got winbind samba3 request %d\n", s3call->request.cmd));
+ DEBUG(10, ("Got winbind samba3 request %d\n", s3call->request->cmd));
+
+ s3call->response = talloc_zero(s3call, struct winbindd_response);
+ NT_STATUS_HAVE_NO_MEMORY(s3call->request);
- s3call->response.length = sizeof(s3call->response);
+ s3call->response->length = sizeof(*s3call->response);
- switch(s3call->request.cmd) {
+ switch(s3call->request->cmd) {
case WINBINDD_INTERFACE_VERSION:
return wbsrv_samba3_interface_version(s3call);
@@ -200,11 +206,11 @@ NTSTATUS wbsrv_samba3_handle_call(struct wbsrv_samba3_call *s3call)
case WINBINDD_CCACHE_NTLMAUTH:
case WINBINDD_NUM_CMDS:
DEBUG(10, ("Unimplemented winbind samba3 request %d\n",
- s3call->request.cmd));
+ s3call->request->cmd));
break;
}
- s3call->response.result = WINBINDD_ERROR;
+ s3call->response->result = WINBINDD_ERROR;
return NT_STATUS_OK;
}
@@ -213,25 +219,26 @@ static NTSTATUS wbsrv_samba3_push_reply(struct wbsrv_samba3_call *call)
uint8_t *extra_data;
size_t extra_data_len = 0;
- extra_data = (uint8_t *)call->response.extra_data.data;
+ extra_data = (uint8_t *)call->response->extra_data.data;
if (extra_data != NULL) {
- extra_data_len = call->response.length -
- sizeof(call->response);
+ extra_data_len = call->response->length -
+ sizeof(*call->response);
}
- call->out = data_blob_talloc(call, NULL, call->response.length);
+ call->out = data_blob_talloc(call, NULL, call->response->length);
NT_STATUS_HAVE_NO_MEMORY(call->out.data);
/* don't push real pointer values into sockets */
if (extra_data) {
- call->response.extra_data.data = (void *)0xFFFFFFFF;
+ call->response->extra_data.data = (void *)0xFFFFFFFF;
}
- memcpy(call->out.data, &call->response, sizeof(call->response));
+
+ memcpy(call->out.data, call->response, sizeof(*call->response));
/* set back the pointer */
- call->response.extra_data.data = extra_data;
+ call->response->extra_data.data = extra_data;
if (extra_data) {
- memcpy(call->out.data + sizeof(call->response),
+ memcpy(call->out.data + sizeof(*call->response),
extra_data,
extra_data_len);
}
diff --git a/source4/winbind/wb_server.h b/source4/winbind/wb_server.h
index 8b79bc7406..d408fbb8e1 100644
--- a/source4/winbind/wb_server.h
+++ b/source4/winbind/wb_server.h
@@ -161,10 +161,10 @@ struct wbsrv_samba3_call {
void *private_data;
/* the request structure of the samba3 protocol */
- struct winbindd_request request;
+ struct winbindd_request *request;
/* the response structure of the samba3 protocol*/
- struct winbindd_response response;
+ struct winbindd_response *response;
DATA_BLOB in;
DATA_BLOB out;