diff options
| author | James Morris <james.l.morris@oracle.com> | 2014-01-08 17:22:32 +1100 |
|---|---|---|
| committer | James Morris <james.l.morris@oracle.com> | 2014-01-08 17:22:32 +1100 |
| commit | 923b49ff69fcbffe6f8b2739de218c45544392a7 (patch) | |
| tree | 44d03d61cad1edd916e5e0d88641353aa7cd0d4c /security/selinux | |
| parent | d4a82a4a033d563f1dc2c944eec2358cb38432d0 (diff) | |
| parent | 8ed814602876bec9bad2649ca17f34b499357a1c (diff) | |
Merge branch 'master' of git://git.infradead.org/users/pcmoore/selinux into next
Diffstat (limited to 'security/selinux')
| -rw-r--r-- | security/selinux/ss/policydb.c | 14 |
1 files changed, 13 insertions, 1 deletions
diff --git a/security/selinux/ss/policydb.c b/security/selinux/ss/policydb.c index dc4011643b5..c0f49884212 100644 --- a/security/selinux/ss/policydb.c +++ b/security/selinux/ss/policydb.c @@ -1995,7 +1995,19 @@ static int filename_trans_read(struct policydb *p, void *fp) if (rc) goto out; - hashtab_insert(p->filename_trans, ft, otype); + rc = hashtab_insert(p->filename_trans, ft, otype); + if (rc) { + /* + * Do not return -EEXIST to the caller, or the system + * will not boot. + */ + if (rc != -EEXIST) + goto out; + /* But free memory to avoid memory leak. */ + kfree(ft); + kfree(name); + kfree(otype); + } } hash_eval(p->filename_trans, "filenametr"); return 0; |