[ns] a small bug in delayed ACK ?

[=?ks_c_5601-1987?B?wdbDosjx?= <jchee@dacl3.snu.ac.kr>]

Hi

 

I think there is a small bug in the implementation of delayed ACK in "tcp-sink.cc"

 

At line 300 of tcp-sink.cc of ns 2.1b6, (DelAckSink::recv).

To delay an ACK, packet seq. "th->seqno()" is compared to "acker_->Seqno()"

 

It can cause a problem, because received packet can be "next expected"

and simultaneously "out-of-order".

Hence the comparison should be changed to between packet seq. "th->seqno()" and

_max seq._ "maxseen_".

 

Thanks

 

-------------------------------------------------------------
Changhee Joo                           jchee@netlab.snu.ac.kr
School of Electrical Engineering, Seoul National Univ., Korea
-------------------------------------------------------------