From 85e3561dc9883aa957de85ad2ff82e0cc13c81b4 Mon Sep 17 00:00:00 2001 From: Simo Sorce Date: Tue, 26 Jan 2010 11:06:00 -0500 Subject: [PATCH] s4:PAC Streamline pac-glue First step, preparing to share the code between multiple plugins. --- source4/kdc/pac-glue.c | 59 ++++++++++++++++++++++++++++++++++---------------- 1 file changed, 40 insertions(+), 19 deletions(-) diff --git a/source4/kdc/pac-glue.c b/source4/kdc/pac-glue.c index d921899c610..9f82f6f3b27 100644 --- a/source4/kdc/pac-glue.c +++ b/source4/kdc/pac-glue.c @@ -4,6 +4,7 @@ PAC Glue between Samba and the KDC Copyright (C) Andrew Bartlett 2005-2009 + Copyright (C) Simo Sorce 2010 This program is free software; you can redistribute it and/or modify it under the terms of the GNU General Public License as published by @@ -45,41 +46,61 @@ static void samba_kdc_plugin_fini(void *ptr) return; } -static krb5_error_code make_pac(krb5_context context, - TALLOC_CTX *mem_ctx, - struct smb_iconv_convenience *iconv_convenience, - struct auth_serversupplied_info *server_info, - krb5_pac *pac) +static NTSTATUS +get_logon_info_pac_blob(TALLOC_CTX *mem_ctx, + struct smb_iconv_convenience *ic, + struct auth_serversupplied_info *info, + DATA_BLOB pac_data) { - union PAC_INFO info; struct netr_SamInfo3 *info3; - krb5_data pac_data; - NTSTATUS nt_status; + union PAC_INFO pac_info; enum ndr_err_code ndr_err; - DATA_BLOB pac_out; - krb5_error_code ret; + NTSTATUS nt_status; - ZERO_STRUCT(info); + ZERO_STRUCT(pac_info); - nt_status = auth_convert_server_info_saminfo3(mem_ctx, server_info, &info3); + nt_status = auth_convert_server_info_saminfo3(mem_ctx, info, &info3); if (!NT_STATUS_IS_OK(nt_status)) { - DEBUG(1, ("Getting Samba info failed: %s\n", nt_errstr(nt_status))); - return EINVAL; + DEBUG(1, ("Getting Samba info failed: %s\n", + nt_errstr(nt_status))); + return nt_status; } - info.logon_info.info = talloc_zero(mem_ctx, struct PAC_LOGON_INFO); + pac_info.logon_info.info = talloc_zero(mem_ctx, struct PAC_LOGON_INFO); if (!mem_ctx) { - return ENOMEM; + return NT_STATUS_NO_MEMORY; } - info.logon_info.info->info3 = *info3; + pac_info.logon_info.info->info3 = *info3; - ndr_err = ndr_push_union_blob(&pac_out, mem_ctx, iconv_convenience, &info, + ndr_err = ndr_push_union_blob(&pac_data, mem_ctx, ic, &pac_info, PAC_TYPE_LOGON_INFO, (ndr_push_flags_fn_t)ndr_push_PAC_INFO); if (!NDR_ERR_CODE_IS_SUCCESS(ndr_err)) { nt_status = ndr_map_error2ntstatus(ndr_err); - DEBUG(1, ("PAC (presig) push failed: %s\n", nt_errstr(nt_status))); + DEBUG(1, ("PAC (presig) push failed: %s\n", + nt_errstr(nt_status))); + return nt_status; + } + + return NT_STATUS_OK; +} + +static krb5_error_code make_pac(krb5_context context, + TALLOC_CTX *mem_ctx, + struct smb_iconv_convenience *iconv_convenience, + struct auth_serversupplied_info *server_info, + krb5_pac *pac) +{ + krb5_data pac_data; + DATA_BLOB pac_out; + NTSTATUS nt_status; + krb5_error_code ret; + + nt_status = get_logon_info_pac_blob(mem_ctx, + iconv_convenience, + server_info, pac_out); + if (!NT_STATUS_IS_OK(nt_status)) { return EINVAL; } -- 2.11.4.GIT