diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2018-03-19 12:12:31 +0300 |
|---|---|---|
| committer | John Johansen <john.johansen@canonical.com> | 2018-03-23 17:28:38 -0700 |
| commit | d53c9f4d212c25b09670a71e2a993071d1e637a2 (patch) | |
| tree | 187fbebda854a6541eb3f72bd316838779dc8a96 /security/apparmor | |
| parent | a61ecd329cfa951b7d36c13e9e2a07e7761c0e89 (diff) | |
| download | linux-d53c9f4d212c25b09670a71e2a993071d1e637a2.tar.bz2 | |
apparmor: Fix an error code in verify_table_headers()
We accidentally return a positive EPROTO instead of a negative -EPROTO.
Since 71 is not an error pointer, that means it eventually results in an
Oops in the caller.
Fixes: d901d6a298dc ("apparmor: dfa split verification of table headers")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: John Johansen <john.johansen@canonical.com>
Diffstat (limited to 'security/apparmor')
| -rw-r--r-- | security/apparmor/match.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/security/apparmor/match.c b/security/apparmor/match.c index dd4c995c5e25..280eba082c7b 100644 --- a/security/apparmor/match.c +++ b/security/apparmor/match.c @@ -198,7 +198,7 @@ out: static int verify_dfa(struct aa_dfa *dfa) { size_t i, state_count, trans_count; - int error = EPROTO; + int error = -EPROTO; state_count = dfa->tables[YYTD_ID_BASE]->td_lolen; trans_count = dfa->tables[YYTD_ID_NXT]->td_lolen; |