|
 |
26765fa |
diff -up setools-3.3.7/secmds/seinfo.c.exitstatus setools-3.3.7/secmds/seinfo.c
|
|
|
26765fa |
--- setools-3.3.7/secmds/seinfo.c.exitstatus 2010-05-03 12:39:02.000000000 -0400
|
|
|
26765fa |
+++ setools-3.3.7/secmds/seinfo.c 2010-11-05 09:54:39.000000000 -0400
|
|
|
26765fa |
@@ -827,7 +827,7 @@ static int print_sens(FILE * fp, const c
|
|
|
26765fa |
*/
|
|
|
26765fa |
static int print_cats(FILE * fp, const char *name, int expand, const apol_policy_t * policydb)
|
|
|
26765fa |
{
|
|
|
26765fa |
- int retval = 0;
|
|
|
26765fa |
+ int retval = -1;
|
|
|
26765fa |
apol_cat_query_t *query = NULL;
|
|
|
26765fa |
apol_vector_t *v = NULL;
|
|
|
26765fa |
const qpol_cat_t *cat_datum = NULL;
|
|
|
26765fa |
@@ -911,9 +911,10 @@ static int print_fsuse(FILE * fp, const
|
|
|
26765fa |
fprintf(fp, " %s\n", tmp);
|
|
|
26765fa |
free(tmp);
|
|
|
26765fa |
}
|
|
|
26765fa |
- if (type && !apol_vector_get_size(v))
|
|
|
26765fa |
+ if (type && !apol_vector_get_size(v)) {
|
|
|
26765fa |
ERR(policydb, "No fs_use statement for filesystem of type %s.", type);
|
|
|
26765fa |
-
|
|
|
26765fa |
+ goto cleanup;
|
|
|
26765fa |
+ }
|
|
|
26765fa |
retval = 0;
|
|
|
26765fa |
cleanup:
|
|
|
26765fa |
apol_fs_use_query_destroy(&query);
|
|
|
26765fa |
@@ -949,7 +950,6 @@ static int print_genfscon(FILE * fp, con
|
|
|
26765fa |
ERR(policydb, "%s", strerror(ENOMEM));
|
|
|
26765fa |
goto cleanup;
|
|
|
26765fa |
}
|
|
|
26765fa |
-
|
|
|
26765fa |
if (apol_genfscon_query_set_filesystem(policydb, query, type))
|
|
|
26765fa |
goto cleanup;
|
|
|
26765fa |
if (apol_genfscon_get_by_query(policydb, query, &v))
|
|
|
26765fa |
@@ -967,8 +967,10 @@ static int print_genfscon(FILE * fp, con
|
|
|
26765fa |
free(tmp);
|
|
|
26765fa |
}
|
|
|
26765fa |
|
|
|
26765fa |
- if (type && !apol_vector_get_size(v))
|
|
|
26765fa |
+ if (type && !apol_vector_get_size(v)) {
|
|
|
26765fa |
ERR(policydb, "No genfscon statement for filesystem of type %s.", type);
|
|
|
26765fa |
+ goto cleanup;
|
|
|
26765fa |
+ }
|
|
|
26765fa |
|
|
|
26765fa |
retval = 0;
|
|
|
26765fa |
cleanup:
|
|
|
26765fa |
@@ -1646,6 +1648,7 @@ cleanup: // close and destroy iterators
|
|
|
26765fa |
|
|
|
26765fa |
int main(int argc, char **argv)
|
|
|
26765fa |
{
|
|
|
26765fa |
+ int rc = 0;
|
|
|
26765fa |
int classes, types, attribs, roles, users, all, expand, stats, rt, optc, isids, bools, sens, cats, fsuse, genfs, netif,
|
|
|
26765fa |
node, port, permissives, polcaps, constrain, linebreaks;
|
|
|
26765fa |
apol_policy_t *policydb = NULL;
|
|
|
26765fa |
@@ -1851,46 +1854,46 @@ int main(int argc, char **argv)
|
|
|
26765fa |
|
|
|
26765fa |
/* display requested info */
|
|
|
26765fa |
if (stats || all)
|
|
|
26765fa |
- print_stats(stdout, policydb);
|
|
|
26765fa |
+ rc = print_stats(stdout, policydb);
|
|
|
26765fa |
if (classes || all)
|
|
|
26765fa |
- print_classes(stdout, class_name, expand, policydb);
|
|
|
26765fa |
+ rc = print_classes(stdout, class_name, expand, policydb);
|
|
|
26765fa |
if (types || all)
|
|
|
26765fa |
- print_types(stdout, type_name, expand, policydb);
|
|
|
26765fa |
+ rc = print_types(stdout, type_name, expand, policydb);
|
|
|
26765fa |
if (attribs || all)
|
|
|
26765fa |
- print_attribs(stdout, attrib_name, expand, policydb);
|
|
|
26765fa |
+ rc = print_attribs(stdout, attrib_name, expand, policydb);
|
|
|
26765fa |
if (roles || all)
|
|
|
26765fa |
- print_roles(stdout, role_name, expand, policydb);
|
|
|
26765fa |
+ rc = print_roles(stdout, role_name, expand, policydb);
|
|
|
26765fa |
if (users || all)
|
|
|
26765fa |
- print_users(stdout, user_name, expand, policydb);
|
|
|
26765fa |
+ rc = print_users(stdout, user_name, expand, policydb);
|
|
|
26765fa |
if (bools || all)
|
|
|
26765fa |
- print_booleans(stdout, bool_name, expand, policydb);
|
|
|
26765fa |
+ rc = print_booleans(stdout, bool_name, expand, policydb);
|
|
|
26765fa |
if (sens || all)
|
|
|
26765fa |
- print_sens(stdout, sens_name, expand, policydb);
|
|
|
26765fa |
+ rc = print_sens(stdout, sens_name, expand, policydb);
|
|
|
26765fa |
if (cats || all)
|
|
|
26765fa |
- print_cats(stdout, cat_name, expand, policydb);
|
|
|
26765fa |
+ rc = print_cats(stdout, cat_name, expand, policydb);
|
|
|
26765fa |
if (fsuse || all)
|
|
|
26765fa |
- print_fsuse(stdout, fsuse_type, policydb);
|
|
|
26765fa |
+ rc = print_fsuse(stdout, fsuse_type, policydb);
|
|
|
26765fa |
if (genfs || all)
|
|
|
26765fa |
- print_genfscon(stdout, genfs_type, policydb);
|
|
|
26765fa |
+ rc = print_genfscon(stdout, genfs_type, policydb);
|
|
|
26765fa |
if (netif || all)
|
|
|
26765fa |
- print_netifcon(stdout, netif_name, policydb);
|
|
|
26765fa |
+ rc = print_netifcon(stdout, netif_name, policydb);
|
|
|
26765fa |
if (node || all)
|
|
|
26765fa |
- print_nodecon(stdout, node_addr, policydb);
|
|
|
26765fa |
+ rc = print_nodecon(stdout, node_addr, policydb);
|
|
|
26765fa |
if (port || all)
|
|
|
26765fa |
- print_portcon(stdout, port_num, protocol, policydb);
|
|
|
26765fa |
+ rc = print_portcon(stdout, port_num, protocol, policydb);
|
|
|
26765fa |
if (isids || all)
|
|
|
26765fa |
- print_isids(stdout, isid_name, expand, policydb);
|
|
|
26765fa |
+ rc = print_isids(stdout, isid_name, expand, policydb);
|
|
|
26765fa |
if (permissives || all)
|
|
|
26765fa |
- print_permissives(stdout, permissive_name, expand, policydb);
|
|
|
26765fa |
+ rc = print_permissives(stdout, permissive_name, expand, policydb);
|
|
|
26765fa |
if (polcaps || all)
|
|
|
26765fa |
- print_polcaps(stdout, polcap_name, expand, policydb);
|
|
|
26765fa |
+ rc = print_polcaps(stdout, polcap_name, expand, policydb);
|
|
|
26765fa |
if (constrain || all)
|
|
|
26765fa |
- print_constraints(stdout, expand, policydb, linebreaks);
|
|
|
26765fa |
+ rc = print_constraints(stdout, expand, policydb, linebreaks);
|
|
|
26765fa |
|
|
|
26765fa |
apol_policy_destroy(&policydb);
|
|
|
26765fa |
apol_policy_path_destroy(&pol_path);
|
|
|
26765fa |
free(policy_file);
|
|
|
26765fa |
- exit(0);
|
|
|
26765fa |
+ exit(rc);
|
|
|
26765fa |
}
|
|
|
26765fa |
|
|
|
26765fa |
/**
|