diff options
author | Kent Overstreet <kent.overstreet@gmail.com> | 2019-03-21 12:51:48 -0400 |
---|---|---|
committer | Kent Overstreet <kent.overstreet@gmail.com> | 2019-03-21 12:52:10 -0400 |
commit | 9a03b23d3d213d1b7f173c1c413e3507f5c95fe9 (patch) | |
tree | ba547573434da7e27deaef19079c26b87d81f9c7 /security/selinux/ss/policydb.c | |
parent | 54c490164523de90c42b1d89e7de3befe3284d1b (diff) |
selinux: fix calling ebitmap_destroy() on null ptrsyzbot-fix
Reported-and-tested-by: syzbot <syzbot+a57b2aff60832666fc28@syzkaller.appspotmail.com>
Signed-off-by: Kent Overstreet <kent.overstreet@gmail.com>
Diffstat (limited to 'security/selinux/ss/policydb.c')
-rw-r--r-- | security/selinux/ss/policydb.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/security/selinux/ss/policydb.c b/security/selinux/ss/policydb.c index 6b576e588725..97c60b342a07 100644 --- a/security/selinux/ss/policydb.c +++ b/security/selinux/ss/policydb.c @@ -828,9 +828,11 @@ void policydb_destroy(struct policydb *p) hashtab_map(p->range_tr, range_tr_destroy, NULL); hashtab_destroy(p->range_tr); - for (i = 0; i < p->p_types.nprim; i++) - ebitmap_destroy(&p->type_attr_map_array[i]); - kvfree(p->type_attr_map_array); + if (p->type_attr_map_array) { + for (i = 0; i < p->p_types.nprim; i++) + ebitmap_destroy(&p->type_attr_map_array[i]); + kvfree(p->type_attr_map_array); + } ebitmap_destroy(&p->filename_trans_ttypes); ebitmap_destroy(&p->policycaps); |