Skip to content

Commit

Permalink
Merge branch 'sctp-dupcookie-fixes'
Browse files Browse the repository at this point in the history
Xin Long says:

====================
sctp: a bunch of fixes for processing dupcookie

After introducing transport hashtable and per stream info into sctp,
some regressions were caused when processing dupcookie, this patchset
is to fix them.
====================

Signed-off-by: David S. Miller <davem@davemloft.net>
  • Loading branch information
David S. Miller committed May 24, 2017
2 parents 12e8b57 + 7e06297 commit 11d3c94
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 10 deletions.
4 changes: 3 additions & 1 deletion net/sctp/associola.c
Original file line number Diff line number Diff line change
Expand Up @@ -1176,7 +1176,9 @@ void sctp_assoc_update(struct sctp_association *asoc,

asoc->ctsn_ack_point = asoc->next_tsn - 1;
asoc->adv_peer_ack_point = asoc->ctsn_ack_point;
if (!asoc->stream) {

if (sctp_state(asoc, COOKIE_WAIT)) {
sctp_stream_free(asoc->stream);
asoc->stream = new->stream;
new->stream = NULL;
}
Expand Down
13 changes: 4 additions & 9 deletions net/sctp/sm_make_chunk.c
Original file line number Diff line number Diff line change
Expand Up @@ -2454,16 +2454,11 @@ int sctp_process_init(struct sctp_association *asoc, struct sctp_chunk *chunk,
* stream sequence number shall be set to 0.
*/

/* Allocate storage for the negotiated streams if it is not a temporary
* association.
*/
if (!asoc->temp) {
if (sctp_stream_init(asoc, gfp))
goto clean_up;
if (sctp_stream_init(asoc, gfp))
goto clean_up;

if (sctp_assoc_set_id(asoc, gfp))
goto clean_up;
}
if (!asoc->temp && sctp_assoc_set_id(asoc, gfp))
goto clean_up;

/* ADDIP Section 4.1 ASCONF Chunk Procedures
*
Expand Down
3 changes: 3 additions & 0 deletions net/sctp/sm_statefuns.c
Original file line number Diff line number Diff line change
Expand Up @@ -2088,6 +2088,9 @@ sctp_disposition_t sctp_sf_do_5_2_4_dupcook(struct net *net,
}
}

/* Set temp so that it won't be added into hashtable */
new_asoc->temp = 1;

/* Compare the tie_tag in cookie with the verification tag of
* current association.
*/
Expand Down

0 comments on commit 11d3c94

Please sign in to comment.