diff options
| author | John Johansen <john.johansen@canonical.com> | 2022-10-10 20:18:50 +0200 |
|---|---|---|
| committer | John Johansen <john.johansen@canonical.com> | 2022-10-10 20:18:50 +0200 |
| commit | ee21a175ecfa821b74822881d354c7f848930738 (patch) | |
| tree | 8a2c0dab37cb783ea5a07a254e5c62370ecf0345 /security/apparmor/policy_unpack.c | |
| parent | apparmor: store return value of unpack_perms_table() to signed variable (diff) | |
| download | linux-ee21a175ecfa821b74822881d354c7f848930738.tar.xz linux-ee21a175ecfa821b74822881d354c7f848930738.zip | |
apparmor: fix uninitialize table variable in error in unpack_trans_table
The error path has one case where *table is uninitialized, initialize
it.
Fixes: a0792e2ceddc ("apparmor: make transition table unpack generic so it can be reused")
Reported-by: kernel test robot <lkp@intel.com>
Signed-off-by: John Johansen <john.johansen@canonical.com>
Diffstat (limited to 'security/apparmor/policy_unpack.c')
| -rw-r--r-- | security/apparmor/policy_unpack.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/security/apparmor/policy_unpack.c b/security/apparmor/policy_unpack.c index 09f316943951..3b956b1235f3 100644 --- a/security/apparmor/policy_unpack.c +++ b/security/apparmor/policy_unpack.c @@ -477,7 +477,7 @@ static struct aa_dfa *unpack_dfa(struct aa_ext *e, int flags) static bool unpack_trans_table(struct aa_ext *e, struct aa_str_table *strs) { void *saved_pos = e->pos; - char **table; + char **table = NULL; /* exec table is optional */ if (unpack_nameX(e, AA_STRUCT, "xtable")) { |