PERFORCE change 34051 for review
Robert Watson
rwatson at FreeBSD.org
Fri Jul 4 03:17:24 GMT 2003
http://perforce.freebsd.org/chv.cgi?CH=34051
Change 34051 by rwatson at rwatson_paprika on 2003/07/03 20:16:31
As new objects begin to support new labels, start to generalize
the default label support in /etc/mac.conf. Rather than maintain
each default label type in an explicit global variable in mac.c,
keep a list of defaults loaded from the configuration file.
Generalize the parsing so that we support both the older:
default_file_labels foo
default_ifnet_labels foo
default_process_labels foo
And also a new:
default_labels file foo
default_labels ifnet foo
default_labels process foo
We now accept arbitrary object classes in the first argument. If
the same object is specified more than once, we discard the
earlier definition in favor of the later one.
Add a new API, mac_prepare_type(), which accepts a mac_t to
prepare, as well as an object name in the second argument, which
will pull a default label set for the object out of the
configuration loaded by mac_init_internal(). This permits the libc
to adapt to new objects known about by applications but not by libc
at compile-time.
Also liberalize the error handling a bit: if we're using implicit
initialization (i.e., the application didn't explicitly initialize
the MAC code), ignore syntax errors and only use valid lines. In
the future, we may want to add explicit warnings and do this a
bit more consistently.
While here, add support for a MAC_CONFFILE environmental variable,
which may be used to specify an alternative mac.conf configuration
file if the application isn't running with modified privilege
(issetugid()).
Affected files ...
.. //depot/projects/trustedbsd/mac/lib/libc/posix1e/mac.c#10 edit
Differences ...
==== //depot/projects/trustedbsd/mac/lib/libc/posix1e/mac.c#10 (text+ko) ====
@@ -1,6 +1,6 @@
/*
* Copyright (c) 1999, 2000, 2001, 2002 Robert N. M. Watson
- * Copyright (c) 2002 Networks Associates Technology, Inc.
+ * Copyright (c) 2002, 2003 Networks Associates Technology, Inc.
* All rights reserved.
*
* This software was developed by Robert Watson for the TrustedBSD Project.
@@ -49,123 +49,211 @@
static int internal_initialized;
-/* Default sets of labels for various query operations. */
-static char *default_file_labels;
-static char *default_ifnet_labels;
-static char *default_process_labels;
+/*
+ * Maintain a list of default label preparations for various object
+ * types. Each name will appear only once in the list.
+ *
+ * XXXMAC: Not thread-safe.
+ */
+LIST_HEAD(, label_default) label_default_head;
+struct label_default {
+ char *ld_name;
+ char *ld_labels;
+ LIST_ENTRY(label_default) ld_entries;
+};
static void
mac_destroy_labels(void)
{
+ struct label_default *ld;
+
+ while ((ld = LIST_FIRST(&label_default_head))) {
+ free(ld->ld_name);
+ free(ld->ld_labels);
+ LIST_REMOVE(ld, ld_entries);
+ free(ld);
+ }
+}
- if (default_file_labels != NULL) {
- free(default_file_labels);
- default_file_labels = NULL;
+static void
+mac_destroy_internal(void)
+{
+
+ mac_destroy_labels();
+
+ internal_initialized = 0;
+}
+
+static int
+mac_add_type(const char *name, const char *labels)
+{
+ struct label_default *ld, *ld_new;
+ char *name_dup, *labels_dup;
+
+ /*
+ * Speculatively allocate all the memory now to avoid allocating
+ * later when we will someday hold a mutex.
+ */
+ name_dup = strdup(name);
+ if (name_dup == NULL) {
+ errno = ENOMEM;
+ return (-1);
+ }
+ labels_dup = strdup(labels);
+ if (labels_dup == NULL) {
+ free(name_dup);
+ errno = ENOMEM;
+ return (-1);
+ }
+ ld_new = malloc(sizeof(*ld));
+ if (ld_new == NULL) {
+ free(name_dup);
+ free(labels_dup);
+ errno = ENOMEM;
+ return (-1);
}
- if (default_ifnet_labels != NULL) {
- free(default_ifnet_labels);
- default_ifnet_labels = NULL;
+ /*
+ * If the type is already present, replace the current entry
+ * rather than add a new instance.
+ */
+ for (ld = LIST_FIRST(&label_default_head); ld != NULL;
+ ld = LIST_NEXT(ld, ld_entries)) {
+ if (strcmp(name, ld->ld_name) == 0)
+ break;
}
- if (default_process_labels != NULL) {
- free(default_process_labels);
- default_process_labels = NULL;
+ if (ld != NULL) {
+ free(ld->ld_labels);
+ ld->ld_labels = labels_dup;
+ labels_dup = NULL;
+ } else {
+ ld = ld_new;
+ ld->ld_name = name_dup;
+ ld->ld_labels = labels_dup;
+
+ ld_new = NULL;
+ name_dup = NULL;
+ labels_dup = NULL;
+
+ LIST_INSERT_HEAD(&label_default_head, ld, ld_entries);
}
+
+ if (name_dup != NULL)
+ free(name_dup);
+ if (labels_dup != NULL)
+ free(labels_dup);
+ if (ld_new != NULL)
+ free(ld_new);
+
+ return (0);
}
-static void
-mac_destroy_internal(void)
+static char *
+next_token(char **string)
{
+ char *token;
- mac_destroy_labels();
+ token = strsep(string, " \t");
+ while (token != NULL && *token == '\0')
+ token = strsep(string, " \t");
- internal_initialized = 0;
+ return (token);
}
static int
-mac_init_internal(void)
+mac_init_internal(int ignore_errors)
{
+ const char *filename;
+ char line[LINE_MAX];
FILE *file;
- char line[LINE_MAX];
int error;
error = 0;
- file = fopen(MAC_CONFFILE, "r");
+ LIST_INIT(&label_default_head);
+
+ if (!issetugid() && getenv("MAC_CONFFILE") != NULL)
+ filename = getenv("MAC_CONFFILE");
+ else
+ filename = MAC_CONFFILE;
+ file = fopen(filename, "r");
if (file == NULL)
return (0);
while (fgets(line, LINE_MAX, file)) {
- char *arg, *parse, *statement;
+ char *arg, *comment, *parse, *statement;
if (line[strlen(line)-1] == '\n')
line[strlen(line)-1] = '\0';
else {
+ if (ignore_errors)
+ continue;
fclose(file);
error = EINVAL;
goto just_return;
}
- parse = line;
- statement = "";
- while (parse && statement[0] == '\0')
- statement = strsep(&parse, " \t");
+ /* Remove any comment. */
+ comment = line;
+ parse = strsep(&comment, "#");
- /* Blank lines ok. */
- if (strlen(statement) == 0)
+ /* Blank lines OK. */
+ statement = next_token(&parse);
+ if (statement == NULL)
continue;
- /* Lines that consist only of comments ok. */
- if (statement[0] == '#')
- continue;
+ if (strcmp(statement, "default_labels") == 0) {
+ char *name, *labels;
- if (strcmp(statement, "default_file_labels") == 0) {
- if (default_file_labels != NULL) {
- free(default_file_labels);
- default_file_labels = NULL;
+ name = next_token(&parse);
+ labels = next_token(&parse);
+ if (name == NULL || labels == NULL ||
+ next_token(&parse) != NULL) {
+ if (ignore_errors)
+ continue;
+ error = EINVAL;
+ fclose(file);
+ goto just_return;
}
- arg = strsep(&parse, "# \t");
- if (arg != NULL && arg[0] != '\0') {
- default_file_labels = strdup(arg);
- if (default_file_labels == NULL) {
- error = ENOMEM;
- fclose(file);
- goto just_return;
- }
+ if (mac_add_type(name, labels) == -1) {
+ if (ignore_errors)
+ continue;
+ fclose(file);
+ goto just_return;
}
- } else if (strcmp(statement, "default_ifnet_labels") == 0) {
- if (default_ifnet_labels != NULL) {
- free(default_ifnet_labels);
- default_ifnet_labels = NULL;
- }
+ } else if (strcmp(statement, "default_ifnet_labels") == 0 ||
+ strcmp(statement, "default_file_labels") == 0 ||
+ strcmp(statement, "default_process_labels") == 0) {
+ char *labels, *type;
+
+ if (strcmp(statement, "default_ifnet_labels") == 0)
+ type = "ifnet";
+ else if (strcmp(statement, "default_file_labels") == 0)
+ type = "file";
+ else if (strcmp(statement, "default_process_labels") ==
+ 0)
+ type = "process";
- arg = strsep(&parse, "# \t");
- if (arg != NULL && arg[0] != '\0') {
- default_ifnet_labels = strdup(arg);
- if (default_ifnet_labels == NULL) {
- error = ENOMEM;
- fclose(file);
- goto just_return;
- }
+ labels = next_token(&parse);
+ if (labels == NULL || next_token(&parse) != NULL) {
+ if (ignore_errors)
+ continue;
+ error = EINVAL;
+ fclose(file);
+ goto just_return;
}
- } else if (strcmp(statement, "default_process_labels") == 0) {
- if (default_process_labels != NULL) {
- free(default_process_labels);
- default_process_labels = NULL;
- }
- arg = strsep(&parse, "# \t");
- if (arg != NULL && arg[0] != '\0') {
- default_process_labels = strdup(arg);
- if (default_process_labels == NULL) {
- error = ENOMEM;
- fclose(file);
- goto just_return;
- }
+ if (mac_add_type(type, labels) == -1) {
+ if (ignore_errors)
+ continue;
+ fclose(file);
+ goto just_return;
}
} else {
+ if (ignore_errors)
+ continue;
fclose(file);
error = EINVAL;
goto just_return;
@@ -187,7 +275,7 @@
{
if (!internal_initialized)
- return (mac_init_internal());
+ return (mac_init_internal(1));
else
return (0);
}
@@ -198,7 +286,7 @@
if (internal_initialized)
mac_destroy_internal();
- return (mac_init_internal());
+ return (mac_init_internal(0));
}
int
@@ -267,6 +355,32 @@
}
int
+mac_prepare_type(struct mac **mac, const char *name)
+{
+ struct label_default *ld;
+
+ for (ld = LIST_FIRST(&label_default_head); ld != NULL;
+ ld = LIST_NEXT(ld, ld_entries)) {
+ if (strcmp(name, ld->ld_name) == 0)
+ return (mac_prepare(mac, ld->ld_labels));
+ }
+
+ return (ENOENT); /* XXXMAC: ENOLABEL */
+}
+
+int
+mac_prepare_ifnet_label(struct mac **mac)
+{
+ int error;
+
+ error = mac_maybe_init_internal();
+ if (error != 0)
+ return (error);
+
+ return (mac_prepare_type(mac, "ifnet"));
+}
+
+int
mac_prepare_file_label(struct mac **mac)
{
int error;
@@ -275,14 +389,11 @@
if (error != 0)
return (error);
- if (default_file_labels == NULL)
- return (mac_prepare(mac, ""));
-
- return (mac_prepare(mac, default_file_labels));
+ return (mac_prepare_type(mac, "file"));
}
int
-mac_prepare_ifnet_label(struct mac **mac)
+mac_prepare_packet_label(struct mac **mac)
{
int error;
@@ -290,11 +401,9 @@
if (error != 0)
return (error);
- if (default_ifnet_labels == NULL)
- return (mac_prepare(mac, ""));
+ return (mac_prepare_type(mac, "packet"));
+}
- return (mac_prepare(mac, default_ifnet_labels));
-}
int
mac_prepare_process_label(struct mac **mac)
{
@@ -304,10 +413,7 @@
if (error != 0)
return (error);
- if (default_process_labels == NULL)
- return (mac_prepare(mac, ""));
-
- return (mac_prepare(mac, default_process_labels));
+ return (mac_prepare_type(mac, "process"));
}
/*
To Unsubscribe: send mail to majordomo at trustedbsd.org
with "unsubscribe trustedbsd-cvs" in the body of the message
More information about the trustedbsd-cvs
mailing list