diff options
| author | Purva Yeshi <[email protected]> | 2025-02-18 14:10:45 +0000 |
|---|---|---|
| committer | Jakub Kicinski <[email protected]> | 2025-02-20 23:28:46 +0000 |
| commit | 1340461e5168f8a33b2b3e0ed04557742664bee1 (patch) | |
| tree | 5285521a3cff228c6eaccfcd0f8b2c88794744a8 /net/unix/af_unix.c | |
| parent | eth: fbnic: Add ethtool support for IRQ coalescing (diff) | |
| download | kernel-1340461e5168f8a33b2b3e0ed04557742664bee1.tar.gz kernel-1340461e5168f8a33b2b3e0ed04557742664bee1.zip | |
af_unix: Fix undefined 'other' error
Fix an issue with the sparse static analysis tool where an
"undefined 'other'" error occurs due to `__releases(&unix_sk(other)->lock)`
being placed before 'other' is in scope.
Remove the `__releases()` annotation from the `unix_wait_for_peer()`
function to eliminate the sparse error. The annotation references `other`
before it is declared, leading to a false positive error during static
analysis.
Since AF_UNIX does not use sparse annotations, this annotation is
unnecessary and does not impact functionality.
Reviewed-by: Joe Damato <[email protected]>
Reviewed-by: Simon Horman <[email protected]>
Signed-off-by: Purva Yeshi <[email protected]>
Reviewed-by: Kuniyuki Iwashima <[email protected]>
Link: https://patch.msgid.link/[email protected]
Signed-off-by: Jakub Kicinski <[email protected]>
Diffstat (limited to 'net/unix/af_unix.c')
| -rw-r--r-- | net/unix/af_unix.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/net/unix/af_unix.c b/net/unix/af_unix.c index 34945de1fb1f..319153850be1 100644 --- a/net/unix/af_unix.c +++ b/net/unix/af_unix.c @@ -1508,7 +1508,6 @@ out: } static long unix_wait_for_peer(struct sock *other, long timeo) - __releases(&unix_sk(other)->lock) { struct unix_sock *u = unix_sk(other); int sched; |