Skip to content

sctp: a bunch of fixes by holding transport

Xin Long linux-sctp netdev 2 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v2 Latest3 patches 5 repliesIn review
  2. v13 patches 8 repliesSuperseded

Changes in v2 from the cover letter

Fix the changelog of patch 2/3

Patches in v2 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/3sctp: hold transport instead of assoc in sctp_diagnet/sctp/socket.c+1 -41 reply
2/3sctp: return back transport in __sctp_rcv_init_lookupnet/sctp/input.c+1 -21 reply
3/3sctp: hold transport instead of assoc when lookup assoc in rx pathinclude/net/sctp/sctp.h net/sctp/input.c net/sctp/ipv6.c+18 -181 reply

Reviews trailers per patch

All 3 patches have a Reviewed-by or Acked-by of their own; the series has one for all of them.

PatchNeil HormanMarcelo Ricardo Leitner
SeriesA-b A-b
1/3A-b
2/3A-b
3/3A-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v2 thread

Cover letter [PATCHv2 net 0/3] sctp: a bunch of fixes by holding transport

There are several places where it holds assoc after getting transport by searching from transport rhashtable, it may cause use-after-free issue.

This patchset is to fix them by holding transport instead.

v1->v2:
Fix the changelog of patch 2/3

Xin Long (3):
sctp: hold transport instead of assoc in sctp_diag
sctp: return back transport in __sctp_rcv_init_lookup
sctp: hold transport instead of assoc when lookup assoc in rx path

 include/net/sctp/sctp.h |  2 +-
 net/sctp/input.c        | 35 +++++++++++++++++------------------
 net/sctp/ipv6.c         |  2 +-
 net/sctp/socket.c       |  5 +----
 4 files changed, 20 insertions(+), 24 deletions(-)
Signature · 1 line
2.1.0