diff options
| author | Thorsten Blum <[email protected]> | 2025-04-13 09:34:34 +0000 |
|---|---|---|
| committer | Jakub Kicinski <[email protected]> | 2025-04-15 15:21:46 +0000 |
| commit | def9d0958befc7dfbda5fc1f22e17fbab912bc62 (patch) | |
| tree | 93e798c347e9a7b74afcecd53c651c2073d46b37 /net/mptcp/protocol.c | |
| parent | mptcp: sched: split validation part (diff) | |
| download | kernel-def9d0958befc7dfbda5fc1f22e17fbab912bc62.tar.gz kernel-def9d0958befc7dfbda5fc1f22e17fbab912bc62.zip | |
mptcp: pm: Return local variable instead of freed pointer
Commit e4c28e3d5c090 ("mptcp: pm: move generic PM helpers to pm.c")
removed an unnecessary if-check, which resulted in returning a freed
pointer.
This still works due to the implicit boolean conversion when returning
the freed pointer from mptcp_remove_anno_list_by_saddr(), but it can be
confusing and potentially error-prone. To improve clarity, add a local
variable to explicitly return a boolean value instead.
Signed-off-by: Thorsten Blum <[email protected]>
Reviewed-by: Matthieu Baerts (NGI0) <[email protected]>
Signed-off-by: Matthieu Baerts (NGI0) <[email protected]>
Link: https://patch.msgid.link/20250413-net-next-mptcp-sched-mib-sft-misc-v2-3-0f83a4350150@kernel.org
Signed-off-by: Jakub Kicinski <[email protected]>
Diffstat (limited to 'net/mptcp/protocol.c')
0 files changed, 0 insertions, 0 deletions