mirror of
https://github.com/SELinuxProject/selinux
synced 2024-12-25 15:32:07 +00:00
Fix expand logic for policy versions older than 24.
This was broken for older policy versions when we updated to version 24. Broken by commit 787f2f00f5d8ed6f5f. Change-Id: I4063334c5c0462ef5c3706611c7dff5c60c612aa Reported-by: William Roberts <bill.c.roberts@gmail.com> Signed-off-by: Stephen Smalley <sds@tycho.nsa.gov>
This commit is contained in:
parent
9da070f700
commit
678ec5df79
@ -3159,12 +3159,12 @@ static int expand_avtab_node(avtab_key_t * k, avtab_datum_t * d, void *args)
|
||||
newkey.target_class = k->target_class;
|
||||
newkey.specified = k->specified;
|
||||
|
||||
if (stype->flavor != TYPE_ATTRIB && ttype->flavor != TYPE_ATTRIB) {
|
||||
if (stype && ttype && stype->flavor != TYPE_ATTRIB && ttype->flavor != TYPE_ATTRIB) {
|
||||
/* Both are individual types, no expansion required. */
|
||||
return expand_avtab_insert(expa, k, d);
|
||||
}
|
||||
|
||||
if (stype->flavor != TYPE_ATTRIB) {
|
||||
if (stype && stype->flavor != TYPE_ATTRIB) {
|
||||
/* Source is an individual type, target is an attribute. */
|
||||
newkey.source_type = k->source_type;
|
||||
ebitmap_for_each_bit(tattr, tnode, j) {
|
||||
@ -3178,7 +3178,7 @@ static int expand_avtab_node(avtab_key_t * k, avtab_datum_t * d, void *args)
|
||||
return 0;
|
||||
}
|
||||
|
||||
if (ttype->flavor != TYPE_ATTRIB) {
|
||||
if (ttype && ttype->flavor != TYPE_ATTRIB) {
|
||||
/* Target is an individual type, source is an attribute. */
|
||||
newkey.target_type = k->target_type;
|
||||
ebitmap_for_each_bit(sattr, snode, i) {
|
||||
@ -3289,12 +3289,12 @@ int expand_cond_av_node(policydb_t * p,
|
||||
newkey.target_class = k->target_class;
|
||||
newkey.specified = k->specified;
|
||||
|
||||
if (stype->flavor != TYPE_ATTRIB && ttype->flavor != TYPE_ATTRIB) {
|
||||
if (stype && ttype && stype->flavor != TYPE_ATTRIB && ttype->flavor != TYPE_ATTRIB) {
|
||||
/* Both are individual types, no expansion required. */
|
||||
return expand_cond_insert(newl, expa, k, d);
|
||||
}
|
||||
|
||||
if (stype->flavor != TYPE_ATTRIB) {
|
||||
if (stype && stype->flavor != TYPE_ATTRIB) {
|
||||
/* Source is an individual type, target is an attribute. */
|
||||
newkey.source_type = k->source_type;
|
||||
ebitmap_for_each_bit(tattr, tnode, j) {
|
||||
@ -3308,7 +3308,7 @@ int expand_cond_av_node(policydb_t * p,
|
||||
return 0;
|
||||
}
|
||||
|
||||
if (ttype->flavor != TYPE_ATTRIB) {
|
||||
if (ttype && ttype->flavor != TYPE_ATTRIB) {
|
||||
/* Target is an individual type, source is an attribute. */
|
||||
newkey.target_type = k->target_type;
|
||||
ebitmap_for_each_bit(sattr, snode, i) {
|
||||
|
Loading…
Reference in New Issue
Block a user