The branch, v4-17-test has been updated via 2803e76fba0 smbd: Fix Bug 15221 via b1cf93f7a48 heimdal: Fix the 32-bit build on FreeBSD via 159054c3bb7 third_party/heimdal: Introduce macro for common plugin structure elements from 5c32c822edd docs-xml: ea support option restricted to user ns
https://git.samba.org/?p=samba.git;a=shortlog;h=v4-17-test - Log ----------------------------------------------------------------- commit 2803e76fba0ee0eb6bb7e0b7acaca5c397249941 Author: Volker Lendecke <v...@samba.org> Date: Fri Nov 4 13:53:21 2022 +0100 smbd: Fix Bug 15221 In 4.17 process_symlink_open() will replace smb_fname_rel->base_name with the link target relative to the share root. So if the link target ends up in a subdirectory of a share, we put a target including a slash into the memcache. Later access will trust the stat cache, passing the target directly to openat_pathref_fsp() which will panic if it gets a real dirfsp and a relname with a slash. Name mangling is not required: Accessing a symlink pointing at a subdirectory at least 2 levels deep in the share with a wrong upper/lower case combination reproduces it. This patch is really a workaround. The "real" fix would be to backport the patches removing process_symlink_open() from master, but this is a bigger change. Bug: https://bugzilla.samba.org/show_bug.cgi?id=15221 Signed-off-by: Volker Lendecke <v...@samba.org> Reviewed-by: Ralph Boehme <s...@samba.org> Autobuild-User(v4-17-test): Jule Anger <jan...@samba.org> Autobuild-Date(v4-17-test): Tue Nov 8 09:23:52 UTC 2022 on sn-devel-184 commit b1cf93f7a48e94a77c0429c90d3a2163f994ccaf Author: Volker Lendecke <v...@samba.org> Date: Fri Jul 22 18:38:21 2022 +0200 heimdal: Fix the 32-bit build on FreeBSD REF: https://github.com/heimdal/heimdal/pull/1004 BUG: https://bugzilla.samba.org/show_bug.cgi?id=15220 Signed-off-by: Volker Lendecke <v...@samba.org> Reviewed-by: Andrew Bartlett <abart...@samba.org> (cherry picked from commit ab4c7bda8daccdb99adaf6ec7fddf8b5f84be09a) commit 159054c3bb760eb8f7a199591d95e79e99fa6eb0 Author: Joseph Sutton <josephsut...@catalyst.net.nz> Date: Sat Oct 22 10:11:53 2022 +1300 third_party/heimdal: Introduce macro for common plugin structure elements Heimdal's HDB plugin interface, and hence Samba's KDC that depends upon it, doesn't work on 32-bit builds due to structure fields being arranged in the wrong order. This problem presents itself in the form of segmentation faults on 32-bit systems, but goes unnoticed on 64-bit builds thanks to extra structure padding absorbing the errant fields. This commit reorders the HDB plugin structure fields to prevent crashes and introduces a common macro to ensure every plugin presents a consistent interface. BUG: https://bugzilla.samba.org/show_bug.cgi?id=15110 Signed-off-by: Joseph Sutton <josephsut...@catalyst.net.nz> Reviewed-by: Andrew Bartlett <abart...@samba.org> (cherry picked from commit 074e92849715ed3485703cfbba3771d405e4e78a) ----------------------------------------------------------------------- Summary of changes: source3/smbd/filename.c | 16 +++++---- third_party/heimdal/kdc/csr_authorizer_plugin.h | 4 +-- third_party/heimdal/kdc/gss_preauth.c | 2 +- .../heimdal/kdc/gss_preauth_authorizer_plugin.h | 4 +-- third_party/heimdal/kdc/kdc-plugin.h | 4 +-- third_party/heimdal/kdc/token_validator_plugin.h | 4 +-- third_party/heimdal/lib/asn1/gen_encode.c | 4 +-- third_party/heimdal/lib/asn1/gen_template.c | 4 +-- third_party/heimdal/lib/base/common_plugin.h | 6 ++-- third_party/heimdal/lib/base/heimbase-svc.h | 5 +++ third_party/heimdal/lib/base/log.c | 2 +- third_party/heimdal/lib/base/plugin.c | 2 +- third_party/heimdal/lib/hdb/hdb-ldap.c | 3 +- third_party/heimdal/lib/hdb/hdb.c | 40 +++++++++++----------- third_party/heimdal/lib/hdb/hdb.h | 4 +-- third_party/heimdal/lib/hdb/test_namespace.c | 8 ++--- third_party/heimdal/lib/kadm5/kadm5-hook.h | 6 ++-- third_party/heimdal/lib/krb5/an2ln_plugin.h | 6 ++-- third_party/heimdal/lib/krb5/db_plugin.h | 6 ++-- third_party/heimdal/lib/krb5/kuserok_plugin.h | 6 ++-- third_party/heimdal/lib/krb5/locate_plugin.h | 6 ++-- third_party/heimdal/lib/krb5/send_to_kdc_plugin.h | 5 ++- third_party/heimdal/lib/krb5/ticket.c | 2 +- 23 files changed, 74 insertions(+), 75 deletions(-) Changeset truncated at 500 lines: diff --git a/source3/smbd/filename.c b/source3/smbd/filename.c index e7873eb124f..2e03c6a5ab7 100644 --- a/source3/smbd/filename.c +++ b/source3/smbd/filename.c @@ -943,12 +943,16 @@ lookup: } if (NT_STATUS_IS_OK(status) && (cache_key.data != NULL)) { - DATA_BLOB value = { - .data = (uint8_t *)smb_fname_rel->base_name, - .length = strlen(smb_fname_rel->base_name) + 1, - }; - - memcache_add(NULL, GETREALFILENAME_CACHE, cache_key, value); + const char *slash = strchr_m(smb_fname_rel->base_name, '/'); + + if (slash == NULL) { + DATA_BLOB value = { + .data = (uint8_t *)smb_fname_rel->base_name, + .length = strlen(smb_fname_rel->base_name) + 1, + }; + memcache_add( + NULL, GETREALFILENAME_CACHE, cache_key, value); + } } TALLOC_FREE(cache_key.data); diff --git a/third_party/heimdal/kdc/csr_authorizer_plugin.h b/third_party/heimdal/kdc/csr_authorizer_plugin.h index 45f42014b60..022fedac0e1 100644 --- a/third_party/heimdal/kdc/csr_authorizer_plugin.h +++ b/third_party/heimdal/kdc/csr_authorizer_plugin.h @@ -62,9 +62,7 @@ * @ingroup krb5_support */ typedef struct krb5plugin_csr_authorizer_ftable_desc { - int minor_version; - krb5_error_code (KRB5_LIB_CALL *init)(krb5_context, void **); - void (KRB5_LIB_CALL *fini)(void *); + HEIM_PLUGIN_FTABLE_COMMON_ELEMENTS(krb5_context); krb5_error_code (KRB5_LIB_CALL *authorize)(void *, /*plug_ctx*/ krb5_context, /*context*/ const char *, /*app*/ diff --git a/third_party/heimdal/kdc/gss_preauth.c b/third_party/heimdal/kdc/gss_preauth.c index d8a2a24fd94..24663deb03a 100644 --- a/third_party/heimdal/kdc/gss_preauth.c +++ b/third_party/heimdal/kdc/gss_preauth.c @@ -493,7 +493,7 @@ _kdc_gss_endtime(astgs_request_t r, endtime = kdc_time + gcp->lifetime; kdc_log(r->context, r->config, 10, - "GSS pre-authentication endtime is %ld", endtime); + "GSS pre-authentication endtime is %ld", (long)endtime); return endtime; } diff --git a/third_party/heimdal/kdc/gss_preauth_authorizer_plugin.h b/third_party/heimdal/kdc/gss_preauth_authorizer_plugin.h index 293e59da47b..1bc1c914ca5 100644 --- a/third_party/heimdal/kdc/gss_preauth_authorizer_plugin.h +++ b/third_party/heimdal/kdc/gss_preauth_authorizer_plugin.h @@ -60,9 +60,7 @@ */ typedef struct krb5plugin_gss_preauth_authorizer_ftable_desc { - int minor_version; - krb5_error_code (KRB5_LIB_CALL *init)(krb5_context, void **); - void (KRB5_LIB_CALL *fini)(void *); + HEIM_PLUGIN_FTABLE_COMMON_ELEMENTS(krb5_context); krb5_error_code (KRB5_LIB_CALL *authorize)(void *, /*plug_ctx*/ astgs_request_t, /*r*/ gss_const_name_t, /*initiator_name*/ diff --git a/third_party/heimdal/kdc/kdc-plugin.h b/third_party/heimdal/kdc/kdc-plugin.h index 9fc5946df17..05286257bf7 100644 --- a/third_party/heimdal/kdc/kdc-plugin.h +++ b/third_party/heimdal/kdc/kdc-plugin.h @@ -120,9 +120,7 @@ typedef krb5_error_code #define KRB5_PLUGIN_KDC_VERSION_10 10 typedef struct krb5plugin_kdc_ftable { - int minor_version; - krb5_error_code (KRB5_CALLCONV *init)(krb5_context, void **); - void (KRB5_CALLCONV *fini)(void *); + HEIM_PLUGIN_FTABLE_COMMON_ELEMENTS(krb5_context); krb5plugin_kdc_pac_generate pac_generate; krb5plugin_kdc_pac_verify pac_verify; krb5plugin_kdc_client_access client_access; diff --git a/third_party/heimdal/kdc/token_validator_plugin.h b/third_party/heimdal/kdc/token_validator_plugin.h index 73b05e5d007..8ff00149aef 100644 --- a/third_party/heimdal/kdc/token_validator_plugin.h +++ b/third_party/heimdal/kdc/token_validator_plugin.h @@ -67,9 +67,7 @@ * @ingroup krb5_support */ typedef struct krb5plugin_token_validator_ftable_desc { - int minor_version; - krb5_error_code (KRB5_LIB_CALL *init)(krb5_context, void **); - void (KRB5_LIB_CALL *fini)(void *); + HEIM_PLUGIN_FTABLE_COMMON_ELEMENTS(krb5_context); krb5_error_code (KRB5_LIB_CALL *validate)(void *, /*plug_ctx*/ krb5_context, const char *, /*realm*/ diff --git a/third_party/heimdal/lib/asn1/gen_encode.c b/third_party/heimdal/lib/asn1/gen_encode.c index d61dc2e6d50..403ddac2f77 100644 --- a/third_party/heimdal/lib/asn1/gen_encode.c +++ b/third_party/heimdal/lib/asn1/gen_encode.c @@ -552,7 +552,7 @@ encode_type (const char *name, const Type *t, const char *tmpstr) * \ * +-- psave_<fieldName> */ - "e = der_put_tag(psave_%s, %lu, %s, %s, %d, &l2_%s);\n" + "e = der_put_tag(psave_%s, %zu, %s, %s, %d, &l2_%s);\n" "if (e) { free(pfree_%s); return e; }\n" /* Restore `len' and adjust it (see `p' below) */ "len = lensave_%s - (l + %zu - asn1_tag_length_%s);\n" @@ -573,7 +573,7 @@ encode_type (const char *name, const Type *t, const char *tmpstr) * \ * +-- p */ - "p = psave_%s - (1 + %lu - asn1_tag_length_%s); }\n", + "p = psave_%s - (1 + %zu - asn1_tag_length_%s); }\n", tmpstr, tmpstr, tmpstr, t->subtype->symbol->name, tmpstr, t->subtype->symbol->name, t->subtype->symbol->name, tmpstr, length_tag(t->tag.tagvalue), diff --git a/third_party/heimdal/lib/asn1/gen_template.c b/third_party/heimdal/lib/asn1/gen_template.c index 883eab4b671..e053a8bdd8b 100644 --- a/third_party/heimdal/lib/asn1/gen_template.c +++ b/third_party/heimdal/lib/asn1/gen_template.c @@ -830,7 +830,7 @@ template_object_set(IOSObjectSet *os, Field *typeidfield, Field *opentypefield) } free(objects); - tlist_header(tl, "{ 0, 0, ((void *)(uintptr_t)%lu) }", nobjs); + tlist_header(tl, "{ 0, 0, ((void *)(uintptr_t)%zu) }", nobjs); tlist_print(tl); tlist_add(tl); os->symbol->emitted_template = 1; @@ -970,7 +970,7 @@ template_members(struct templatehead *temp, "{ A1_OP_NAME, %d, \"%s\" }", (int)m->val, m->name); nmemb++; } - tlist_header(tl, "{ 0, 0, ((void *)(uintptr_t)%lu) }", nmemb); + tlist_header(tl, "{ 0, 0, ((void *)(uintptr_t)%zu) }", nmemb); /* XXX Accidentally O(N^2)? */ if (!tlist_find_dup(tl)) { tlist_print(tl); diff --git a/third_party/heimdal/lib/base/common_plugin.h b/third_party/heimdal/lib/base/common_plugin.h index 82c58956383..ece1d28174a 100644 --- a/third_party/heimdal/lib/base/common_plugin.h +++ b/third_party/heimdal/lib/base/common_plugin.h @@ -36,6 +36,8 @@ #ifndef HEIMDAL_BASE_COMMON_PLUGIN_H #define HEIMDAL_BASE_COMMON_PLUGIN_H +#include <heimbase-svc.h> + #ifdef _WIN32 # ifndef HEIM_CALLCONV # define HEIM_CALLCONV __stdcall @@ -69,9 +71,7 @@ typedef heim_get_instance_func_t krb5_get_instance_t; * All plugin function tables extend the following structure. */ struct heim_plugin_common_ftable_desc { - int version; - int (HEIM_LIB_CALL *init)(heim_pcontext, void **); - void (HEIM_LIB_CALL *fini)(void *); + HEIM_PLUGIN_FTABLE_COMMON_ELEMENTS(heim_pcontext); }; typedef struct heim_plugin_common_ftable_desc heim_plugin_common_ftable; typedef struct heim_plugin_common_ftable_desc *heim_plugin_common_ftable_p; diff --git a/third_party/heimdal/lib/base/heimbase-svc.h b/third_party/heimdal/lib/base/heimbase-svc.h index 083917fb806..0e7454c8382 100644 --- a/third_party/heimdal/lib/base/heimbase-svc.h +++ b/third_party/heimdal/lib/base/heimbase-svc.h @@ -75,4 +75,9 @@ heim_dict_t attributes; \ int32_t error_code +#define HEIM_PLUGIN_FTABLE_COMMON_ELEMENTS(CONTEXT_TYPE) \ + int minor_version; \ + int (HEIM_LIB_CALL *init)(CONTEXT_TYPE, void **); \ + void (HEIM_LIB_CALL *fini)(void *) + #endif /* HEIMBASE_SVC_H */ diff --git a/third_party/heimdal/lib/base/log.c b/third_party/heimdal/lib/base/log.c index 818ac8398d5..24295b5adbc 100644 --- a/third_party/heimdal/lib/base/log.c +++ b/third_party/heimdal/lib/base/log.c @@ -849,7 +849,7 @@ heim_audit_addkv_timediff(heim_svc_req_desc r, const char *k, sec -= 1; } - heim_audit_addkv(r, 0, k, "%s%ld.%06d", sign, sec, usec); + heim_audit_addkv(r, 0, k, "%s%ld.%06d", sign, (long)sec, usec); } void diff --git a/third_party/heimdal/lib/base/plugin.c b/third_party/heimdal/lib/base/plugin.c index 631a3386c83..355306dd9d6 100644 --- a/third_party/heimdal/lib/base/plugin.c +++ b/third_party/heimdal/lib/base/plugin.c @@ -653,7 +653,7 @@ reduce_by_version(heim_object_t value, void *ctx, int *stop) struct iter_ctx *s = ctx; struct heim_plugin *pl = value; - if (pl->ftable && pl->ftable->version >= s->caller->min_version) + if (pl->ftable && pl->ftable->minor_version >= s->caller->min_version) heim_array_append_value(s->result, pl); } diff --git a/third_party/heimdal/lib/hdb/hdb-ldap.c b/third_party/heimdal/lib/hdb/hdb-ldap.c index 6a2876c51d7..5cd097f5b6b 100644 --- a/third_party/heimdal/lib/hdb/hdb-ldap.c +++ b/third_party/heimdal/lib/hdb/hdb-ldap.c @@ -2097,9 +2097,9 @@ fini(void *ctx) struct hdb_method hdb_ldap_interface = { HDB_INTERFACE_VERSION, - 0 /*is_file_based*/, 0 /*can_taste*/, init, fini, + 0 /*is_file_based*/, 0 /*can_taste*/, "ldap", hdb_ldap_create }; @@ -2108,6 +2108,7 @@ struct hdb_method hdb_ldapi_interface = { HDB_INTERFACE_VERSION, init, fini, + 0 /*is_file_based*/, 0 /*can_taste*/, "ldapi", hdb_ldapi_create }; diff --git a/third_party/heimdal/lib/hdb/hdb.c b/third_party/heimdal/lib/hdb/hdb.c index 56c403842e6..171ba9e3fd3 100644 --- a/third_party/heimdal/lib/hdb/hdb.c +++ b/third_party/heimdal/lib/hdb/hdb.c @@ -66,41 +66,41 @@ const int hdb_interface_version = HDB_INTERFACE_VERSION; static struct hdb_method methods[] = { /* "db:" should be db3 if we have db3, or db1 if we have db1 */ #if HAVE_DB3 - { HDB_INTERFACE_VERSION, 1 /*is_file_based*/, 1 /*can_taste*/, NULL, NULL, + { HDB_INTERFACE_VERSION, NULL, NULL, 1 /*is_file_based*/, 1 /*can_taste*/, "db:", hdb_db3_create}, #elif HAVE_DB1 - { HDB_INTERFACE_VERSION, 1, 1, NULL, NULL, "db:", hdb_db1_create}, + { HDB_INTERFACE_VERSION, NULL, NULL, 1, 1, "db:", hdb_db1_create}, #endif #if HAVE_DB1 - { HDB_INTERFACE_VERSION, 1, 1, NULL, NULL, "db1:", hdb_db1_create}, + { HDB_INTERFACE_VERSION, NULL, NULL, 1, 1, "db1:", hdb_db1_create}, #endif #if HAVE_DB3 - { HDB_INTERFACE_VERSION, 1, 1, NULL, NULL, "db3:", hdb_db3_create}, + { HDB_INTERFACE_VERSION, NULL, NULL, 1, 1, "db3:", hdb_db3_create}, #endif #if HAVE_DB1 - { HDB_INTERFACE_VERSION, 1, 1, NULL, NULL, "mit-db:", hdb_mitdb_create}, + { HDB_INTERFACE_VERSION, NULL, NULL, 1, 1, "mit-db:", hdb_mitdb_create}, #endif #if HAVE_LMDB - { HDB_INTERFACE_VERSION, 1, 1, NULL, NULL, "mdb:", hdb_mdb_create}, - { HDB_INTERFACE_VERSION, 1, 1, NULL, NULL, "lmdb:", hdb_mdb_create}, + { HDB_INTERFACE_VERSION, NULL, NULL, 1, 1, "mdb:", hdb_mdb_create}, + { HDB_INTERFACE_VERSION, NULL, NULL, 1, 1, "lmdb:", hdb_mdb_create}, #endif #if HAVE_NDBM - { HDB_INTERFACE_VERSION, 1, 0, NULL, NULL, "ndbm:", hdb_ndbm_create}, + { HDB_INTERFACE_VERSION, NULL, NULL, 1, 0, "ndbm:", hdb_ndbm_create}, #endif #ifdef HAVE_SQLITE3 - { HDB_INTERFACE_VERSION, 1, 1, NULL, NULL, "sqlite:", hdb_sqlite_create}, + { HDB_INTERFACE_VERSION, NULL, NULL, 1, 1, "sqlite:", hdb_sqlite_create}, #endif /* The keytab interface can't use its hdb_open() method to "taste" a DB */ - { HDB_INTERFACE_VERSION, 1, 0, NULL, NULL, "keytab:", hdb_keytab_create}, + { HDB_INTERFACE_VERSION, NULL, NULL, 1, 0, "keytab:", hdb_keytab_create}, /* The rest are not file-based */ #if defined(OPENLDAP) && !defined(OPENLDAP_MODULE) - { HDB_INTERFACE_VERSION, 0, 0, NULL, NULL, "ldap:", hdb_ldap_create}, - { HDB_INTERFACE_VERSION, 0, 0, NULL, NULL, "ldapi:", hdb_ldapi_create}, + { HDB_INTERFACE_VERSION, NULL, NULL, 0, 0, "ldap:", hdb_ldap_create}, + { HDB_INTERFACE_VERSION, NULL, NULL, 0, 0, "ldapi:", hdb_ldapi_create}, #elif defined(OPENLDAP) - { HDB_INTERFACE_VERSION, 0, 0, NULL, NULL, "ldap:", NULL}, - { HDB_INTERFACE_VERSION, 0, 0, NULL, NULL, "ldapi:", NULL}, + { HDB_INTERFACE_VERSION, NULL, NULL, 0, 0, "ldap:", NULL}, + { HDB_INTERFACE_VERSION, NULL, NULL, 0, 0, "ldapi:", NULL}, #endif - { 0, 0, 0, NULL, NULL, NULL, NULL} + { 0, NULL, NULL, 0, 0, NULL, NULL} }; /** @@ -494,19 +494,19 @@ hdb_init_db(krb5_context context, HDB *db) */ #if defined(HAVE_LMDB) static struct hdb_method default_dbmethod = - { HDB_INTERFACE_VERSION, 1, 1, NULL, NULL, "", hdb_mdb_create }; + { HDB_INTERFACE_VERSION, NULL, NULL, 1, 1, "", hdb_mdb_create }; #elif defined(HAVE_DB3) static struct hdb_method default_dbmethod = - { HDB_INTERFACE_VERSION, 1, 1, NULL, NULL, "", hdb_db3_create }; + { HDB_INTERFACE_VERSION, NULL, NULL, 1, 1, "", hdb_db3_create }; #elif defined(HAVE_DB1) static struct hdb_method default_dbmethod = - { HDB_INTERFACE_VERSION, 1, 1, NULL, NULL, "", hdb_db1_create }; + { HDB_INTERFACE_VERSION, NULL, NULL, 1, 1, "", hdb_db1_create }; #elif defined(HAVE_NDBM) static struct hdb_method default_dbmethod = - { HDB_INTERFACE_VERSION, 0, 1, NULL, NULL, "", hdb_ndbm_create }; + { HDB_INTERFACE_VERSION, NULL, NULL, 0, 1, "", hdb_ndbm_create }; #else static struct hdb_method default_dbmethod = - { 0, 0, 0, NULL, NULL, NULL, NULL}; + { 0, NULL, NULL, 0, 0, NULL, NULL}; #endif static int diff --git a/third_party/heimdal/lib/hdb/hdb.h b/third_party/heimdal/lib/hdb/hdb.h index 0f2c92151e5..2a3d1c4bb8c 100644 --- a/third_party/heimdal/lib/hdb/hdb.h +++ b/third_party/heimdal/lib/hdb/hdb.h @@ -307,11 +307,9 @@ typedef struct HDB { #define HDB_INTERFACE_VERSION 11 struct hdb_method { - int version; + HEIM_PLUGIN_FTABLE_COMMON_ELEMENTS(krb5_context); unsigned int is_file_based:1; unsigned int can_taste:1; - krb5_error_code (*init)(krb5_context, void **); - void (*fini)(void *); const char *prefix; krb5_error_code (*create)(krb5_context, HDB **, const char *filename); }; diff --git a/third_party/heimdal/lib/hdb/test_namespace.c b/third_party/heimdal/lib/hdb/test_namespace.c index a4c44ba190e..f9b4cdbdde8 100644 --- a/third_party/heimdal/lib/hdb/test_namespace.c +++ b/third_party/heimdal/lib/hdb/test_namespace.c @@ -243,17 +243,17 @@ struct hdb_method hdb_test = #ifdef WIN32 /* Not c99 */ HDB_INTERFACE_VERSION, - 1 /*is_file_based*/, 1 /*can_taste*/, hdb_test_init, hdb_test_fini, + 1 /*is_file_based*/, 1 /*can_taste*/, "test", hdb_test_create #else - .version = HDB_INTERFACE_VERSION, - .is_file_based = 1, - .can_taste = 1, + .minor_version = HDB_INTERFACE_VERSION, .init = hdb_test_init, .fini = hdb_test_fini, + .is_file_based = 1, + .can_taste = 1, .prefix = "test", .create = hdb_test_create #endif diff --git a/third_party/heimdal/lib/kadm5/kadm5-hook.h b/third_party/heimdal/lib/kadm5/kadm5-hook.h index 78236a5ed85..7d47f556add 100644 --- a/third_party/heimdal/lib/kadm5/kadm5-hook.h +++ b/third_party/heimdal/lib/kadm5/kadm5-hook.h @@ -35,6 +35,8 @@ #define KADM5_HOOK_VERSION_V1 1 +#include <heimbase-svc.h> + /* * Each hook is called before the operation using KADM5_STAGE_PRECOMMIT and * then after the operation using KADM5_STAGE_POSTCOMMIT. If the hook returns @@ -53,9 +55,7 @@ enum kadm5_hook_stage { #define KADM5_HOOK_FLAG_CONDITIONAL 0x2 /* only change password if different */ typedef struct kadm5_hook_ftable { - int version; - krb5_error_code (KRB5_CALLCONV *init)(krb5_context, void **data); - void (KRB5_CALLCONV *fini)(void *data); + HEIM_PLUGIN_FTABLE_COMMON_ELEMENTS(krb5_context); const char *name; const char *vendor; diff --git a/third_party/heimdal/lib/krb5/an2ln_plugin.h b/third_party/heimdal/lib/krb5/an2ln_plugin.h index 89913b5780a..b592f23b845 100644 --- a/third_party/heimdal/lib/krb5/an2ln_plugin.h +++ b/third_party/heimdal/lib/krb5/an2ln_plugin.h @@ -36,6 +36,8 @@ #ifndef HEIMDAL_KRB5_AN2LN_PLUGIN_H #define HEIMDAL_KRB5_AN2LN_PLUGIN_H 1 +#include <heimbase-svc.h> + #define KRB5_PLUGIN_AN2LN "an2ln" #define KRB5_PLUGIN_AN2LN_VERSION_0 0 @@ -80,9 +82,7 @@ typedef krb5_error_code (KRB5_LIB_CALL *set_result_f)(void *, const char *); * @ingroup krb5_support */ typedef struct krb5plugin_an2ln_ftable_desc { - int minor_version; - krb5_error_code (KRB5_LIB_CALL *init)(krb5_context, void **); - void (KRB5_LIB_CALL *fini)(void *); + HEIM_PLUGIN_FTABLE_COMMON_ELEMENTS(krb5_context); krb5_error_code (KRB5_LIB_CALL *an2ln)(void *, krb5_context, const char *, krb5_const_principal, set_result_f, void *); } krb5plugin_an2ln_ftable; diff --git a/third_party/heimdal/lib/krb5/db_plugin.h b/third_party/heimdal/lib/krb5/db_plugin.h index 730c0609555..ab676d51a6f 100644 --- a/third_party/heimdal/lib/krb5/db_plugin.h +++ b/third_party/heimdal/lib/krb5/db_plugin.h @@ -33,6 +33,8 @@ #ifndef HEIMDAL_KRB5_DB_PLUGIN_H #define HEIMDAL_KRB5_DB_PLUGIN_H 1 +#include <heimbase-svc.h> + #define KRB5_PLUGIN_DB "krb5_db_plug" #define KRB5_PLUGIN_DB_VERSION_0 0 @@ -59,9 +61,7 @@ * @ingroup krb5_support */ typedef struct krb5plugin_db_ftable_desc { - int minor_version; - krb5_error_code (KRB5_LIB_CALL *init)(krb5_context, void **); - void (KRB5_LIB_CALL *fini)(void *); + HEIM_PLUGIN_FTABLE_COMMON_ELEMENTS(krb5_context); } krb5plugin_db_ftable; #endif /* HEIMDAL_KRB5_DB_PLUGIN_H */ diff --git a/third_party/heimdal/lib/krb5/kuserok_plugin.h b/third_party/heimdal/lib/krb5/kuserok_plugin.h index b45071d18e7..7c3f3b4c8dc 100644 --- a/third_party/heimdal/lib/krb5/kuserok_plugin.h +++ b/third_party/heimdal/lib/krb5/kuserok_plugin.h @@ -32,6 +32,8 @@ #ifndef HEIMDAL_KRB5_KUSEROK_PLUGIN_H #define HEIMDAL_KRB5_KUSEROK_PLUGIN_H 1 +#include <heimbase-svc.h> + #define KRB5_PLUGIN_KUSEROK "krb5_plugin_kuserok" #define KRB5_PLUGIN_KUSEROK_VERSION_0 0 @@ -76,9 +78,7 @@ * @ingroup krb5_support */ typedef struct krb5plugin_kuserok_ftable_desc { - int minor_version; - krb5_error_code (KRB5_LIB_CALL *init)(krb5_context, void **); - void (KRB5_LIB_CALL *fini)(void *); + HEIM_PLUGIN_FTABLE_COMMON_ELEMENTS(krb5_context); krb5_error_code (KRB5_LIB_CALL *kuserok)(void *, krb5_context, const char *, unsigned int, const char *, const char *, krb5_const_principal, diff --git a/third_party/heimdal/lib/krb5/locate_plugin.h b/third_party/heimdal/lib/krb5/locate_plugin.h index 52ef0f380ee..7fcb5ec6f5a 100644 --- a/third_party/heimdal/lib/krb5/locate_plugin.h +++ b/third_party/heimdal/lib/krb5/locate_plugin.h @@ -38,6 +38,8 @@ #ifndef HEIMDAL_KRB5_LOCATE_PLUGIN_H #define HEIMDAL_KRB5_LOCATE_PLUGIN_H 1 +#include <heimbase-svc.h> + #define KRB5_PLUGIN_LOCATE "service_locator" #define KRB5_PLUGIN_LOCATE_VERSION 1 #define KRB5_PLUGIN_LOCATE_VERSION_0 0 @@ -70,9 +72,7 @@ typedef krb5_error_code typedef struct krb5plugin_service_locate_ftable { - int minor_version; - krb5_error_code (KRB5_CALLCONV *init)(krb5_context, void **); - void (KRB5_CALLCONV *fini)(void *); + HEIM_PLUGIN_FTABLE_COMMON_ELEMENTS(krb5_context); krb5plugin_service_locate_lookup_old old_lookup; krb5plugin_service_locate_lookup lookup; /* version 2 */ } krb5plugin_service_locate_ftable; diff --git a/third_party/heimdal/lib/krb5/send_to_kdc_plugin.h b/third_party/heimdal/lib/krb5/send_to_kdc_plugin.h index 0fa43d3aba9..30d6892e536 100644 --- a/third_party/heimdal/lib/krb5/send_to_kdc_plugin.h +++ b/third_party/heimdal/lib/krb5/send_to_kdc_plugin.h @@ -37,6 +37,7 @@ #define HEIMDAL_KRB5_SEND_TO_KDC_PLUGIN_H 1 #include <krb5.h> +#include <heimbase-svc.h> #define KRB5_PLUGIN_SEND_TO_KDC "send_to_kdc" @@ -61,9 +62,7 @@ typedef krb5_error_code typedef struct krb5plugin_send_to_kdc_ftable { - int minor_version; - krb5_error_code (KRB5_CALLCONV *init)(krb5_context, void **); - void (KRB5_CALLCONV *fini)(void *); + HEIM_PLUGIN_FTABLE_COMMON_ELEMENTS(krb5_context); -- Samba Shared Repository