diff options
author | rwatson <rwatson@FreeBSD.org> | 2006-12-28 21:15:37 +0000 |
---|---|---|
committer | rwatson <rwatson@FreeBSD.org> | 2006-12-28 21:15:37 +0000 |
commit | 8a59ab518d7f3acbb4450fac6d41d3ac7f911e81 (patch) | |
tree | fcc80f994f4d7bbd991ae7c2cd4db2ba9fdbdd3d /sys | |
parent | ebba43f55adf280a09ab043ca5b6ad70c2474264 (diff) | |
download | FreeBSD-src-8a59ab518d7f3acbb4450fac6d41d3ac7f911e81.zip FreeBSD-src-8a59ab518d7f3acbb4450fac6d41d3ac7f911e81.tar.gz |
Move mac_init_label() and mac_destroy_label() from mac_framework.c to
mac_label.c, and use these instead of replicated code in the label zone
constructor and destructor.
Obtained from: TrustedBSD Project
Diffstat (limited to 'sys')
-rw-r--r-- | sys/security/mac/mac_framework.c | 19 | ||||
-rw-r--r-- | sys/security/mac/mac_label.c | 41 |
2 files changed, 31 insertions, 29 deletions
diff --git a/sys/security/mac/mac_framework.c b/sys/security/mac/mac_framework.c index 3f3e9b0..f248e55 100644 --- a/sys/security/mac/mac_framework.c +++ b/sys/security/mac/mac_framework.c @@ -559,25 +559,6 @@ mac_error_select(int error1, int error2) return (error2); } -void -mac_init_label(struct label *label) -{ - - bzero(label, sizeof(*label)); - label->l_flags = MAC_FLAG_INITIALIZED; -} - -void -mac_destroy_label(struct label *label) -{ - - KASSERT(label->l_flags & MAC_FLAG_INITIALIZED, - ("destroying uninitialized label")); - - bzero(label, sizeof(*label)); - /* implicit: label->l_flags &= ~MAC_FLAG_INITIALIZED; */ -} - int mac_check_structmac_consistent(struct mac *mac) { diff --git a/sys/security/mac/mac_label.c b/sys/security/mac/mac_label.c index 9e162a4..619df35 100644 --- a/sys/security/mac/mac_label.c +++ b/sys/security/mac/mac_label.c @@ -45,7 +45,7 @@ __FBSDID("$FreeBSD$"); #include <security/mac/mac_internal.h> /* - * zone_label is the UMA zone from which all labels are allocated. Label + * zone_label is the UMA zone from which most labels are allocated. Label * structures are initialized to zero bytes so that policies see a NULL/0 * slot on first use, even if the policy is loaded after the label is * allocated for an object. @@ -64,6 +64,34 @@ mac_labelzone_init(void) UMA_ALIGN_PTR, 0); } +/* + * mac_init_label() and mac_destroy_label() are exported so that they can be + * used in mbuf tag initialization, where labels are not slab allocated from + * the zone_label zone. + */ +void +mac_init_label(struct label *label) +{ + + bzero(label, sizeof(*label)); + label->l_flags = MAC_FLAG_INITIALIZED; +} + +void +mac_destroy_label(struct label *label) +{ + + KASSERT(label->l_flags & MAC_FLAG_INITIALIZED, + ("destroying uninitialized label")); + +#ifdef DIAGNOSTIC + bzero(label, sizeof(*label)); +#else + label->l_flags &= ~MAC_FLAG_INITIALIZED; +#endif +} + + static int mac_labelzone_ctor(void *mem, int size, void *arg, int flags) { @@ -71,8 +99,7 @@ mac_labelzone_ctor(void *mem, int size, void *arg, int flags) KASSERT(size == sizeof(*label), ("mac_labelzone_ctor: wrong size\n")); label = mem; - bzero(label, sizeof(*label)); - label->l_flags = MAC_FLAG_INITIALIZED; + mac_init_label(label); return (0); } @@ -83,13 +110,7 @@ mac_labelzone_dtor(void *mem, int size, void *arg) KASSERT(size == sizeof(*label), ("mac_labelzone_dtor: wrong size\n")); label = mem; - KASSERT(label->l_flags & MAC_FLAG_INITIALIZED, - ("mac_labelzone_dtor: label not initialized")); -#ifdef DIAGNOSTIC - bzero(label, sizeof(*label)); -#else - label->l_flags &= ~MAC_FLAG_INITIALIZED; -#endif + mac_destroy_label(label); } struct label * |