diff options
| author | John Johansen <john.johansen@canonical.com> | 2026-05-06 01:17:38 -0700 |
|---|---|---|
| committer | John Johansen <john.johansen@canonical.com> | 2026-06-13 20:18:17 -0700 |
| commit | e27bfb2ae9ad8522aea82d435fd6d73cccee7e17 (patch) | |
| tree | d3ecbee62bcd28c4e6fe49b8ec51b3b07bd31012 | |
| parent | 7306c41672487a6c28430714be063bc6942c28f2 (diff) | |
apparmor: remove unnecessary goto and associated label
There is no need for a goto a label immediately following the
conditional block when the jump is the last statement in the block.
Fixes: 7306c41672487 ("apparmor: release exe file resources on path failure")
Signed-off-by: John Johansen <john.johansen@canonical.com>
| -rw-r--r-- | security/apparmor/task.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/security/apparmor/task.c b/security/apparmor/task.c index 6445cb5f8526..b9fb3738124e 100644 --- a/security/apparmor/task.c +++ b/security/apparmor/task.c @@ -313,12 +313,9 @@ static const char *get_current_exe_path(char *buffer, int buffer_size) p = exe_file->f_path; path_get(&p); - if (aa_path_name(&p, FLAG_VIEW_SUBNS, buffer, &path_str, NULL, NULL)) { + if (aa_path_name(&p, FLAG_VIEW_SUBNS, buffer, &path_str, NULL, NULL)) path_str = ERR_PTR(-ENOMEM); - goto out; - } -out: fput(exe_file); path_put(&p); |