diff options
| author | Purva Yeshi <purvayeshi550@gmail.com> | 2025-02-18 17:10:45 +0300 |
|---|---|---|
| committer | Jakub Kicinski <kuba@kernel.org> | 2025-02-21 02:28:46 +0300 |
| commit | 1340461e5168f8a33b2b3e0ed04557742664bee1 (patch) | |
| tree | 5285521a3cff228c6eaccfcd0f8b2c88794744a8 /net/unix/af_unix.c | |
| parent | 7b5b7a597fbc19dacf6eefba3dd632a149a539b8 (diff) | |
| download | linux-1340461e5168f8a33b2b3e0ed04557742664bee1.tar.xz | |
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 <jdamato@fastly.com>
Reviewed-by: Simon Horman <horms@kernel.org>
Signed-off-by: Purva Yeshi <purvayeshi550@gmail.com>
Reviewed-by: Kuniyuki Iwashima <kuniyu@amazon.com>
Link: https://patch.msgid.link/20250218141045.38947-1-purvayeshi550@gmail.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
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; |