diff options
| author | Thorsten Blum <thorsten.blum@linux.dev> | 2026-02-04 23:07:35 +0100 |
|---|---|---|
| committer | John Johansen <john.johansen@canonical.com> | 2026-04-22 10:57:52 -0700 |
| commit | e6a522c5b4803b8f5632d5ce8f27431a1ae73222 (patch) | |
| tree | 9ee516d34472b60b057615ade80f1deef0acef4b /security | |
| parent | 46401cc99c6237ba825cfd65ef023955ce2a6316 (diff) | |
| download | linux-next-history-e6a522c5b4803b8f5632d5ce8f27431a1ae73222.tar.gz | |
apparmor: Remove redundant if check in sk_peer_get_label
Remove the redundant if check in sk_peer_get_label() and return
ERR_PTR(-ENOPROTOOPT) directly.
Signed-off-by: Thorsten Blum <thorsten.blum@linux.dev>
Signed-off-by: John Johansen <john.johansen@canonical.com>
Diffstat (limited to 'security')
| -rw-r--r-- | security/apparmor/lsm.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/security/apparmor/lsm.c b/security/apparmor/lsm.c index 553f4127d59f5..6f15b968a32a7 100644 --- a/security/apparmor/lsm.c +++ b/security/apparmor/lsm.c @@ -1523,15 +1523,11 @@ static int apparmor_socket_sock_rcv_skb(struct sock *sk, struct sk_buff *skb) static struct aa_label *sk_peer_get_label(struct sock *sk) { struct aa_sk_ctx *ctx = aa_sock(sk); - struct aa_label *label = ERR_PTR(-ENOPROTOOPT); if (rcu_access_pointer(ctx->peer)) return aa_get_label_rcu(&ctx->peer); - if (sk->sk_family != PF_UNIX) - return ERR_PTR(-ENOPROTOOPT); - - return label; + return ERR_PTR(-ENOPROTOOPT); } /** |