d/freebsd-nq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Use TAILQ_EMPTY() for testing if a tail queue is empty.

Browse Source 
Set whoFrom to NULL after freeing whoFrom.
This commit is contained in:
Michael Tuexen 2010-09-15 21:53:10 +00:00
parent 225f5b4adc
commit 24f52bbd9b
1 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
sys/netinet/sctp_indata.c
View File

@ -708,9 +708,10 @@ protocol_error:
control->data = NULL;
asoc->size_on_all_streams -= control->length;
sctp_ucount_decr(asoc->cnt_on_all_streams);
if (control->whoFrom)
if (control->whoFrom) {
sctp_free_remote_addr(control->whoFrom);
control->whoFrom = NULL;
control->whoFrom = NULL;
}
sctp_free_a_readq(stcb, control);
return;
} else {
@ -4845,7 +4846,7 @@ sctp_handle_sack(struct mbuf *m, int offset_seg, int offset_dup,
if (asoc->pr_sctp_cnt != 0)
asoc->pr_sctp_cnt--;
}
if ((TAILQ_FIRST(&asoc->sent_queue) == NULL) &&
if (TAILQ_EMPTY(&asoc->sent_queue) &&
(asoc->total_flight > 0)) {
#ifdef INVARIANTS
panic("Warning flight size is postive and should be 0");
@ -5818,7 +5819,7 @@ sctp_handle_forward_tsn(struct sctp_tcb *stcb,
*/
sctp_slide_mapping_arrays(stcb);
if (TAILQ_FIRST(&asoc->reasmqueue)) {
if (!TAILQ_EMPTY(&asoc->reasmqueue)) {
/* now lets kick out and check for more fragmented delivery */
/* sa_ignore NO_NULL_CHK */
sctp_deliver_reasm_check(stcb, &stcb->asoc);