[Ns-bugs] [Bug 247] Checksum calculation crashes tcp-large-transfer

[bugzilla-daemon@nsnam-www.ece.gatech.edu]

http://www.nsnam.org/bugzilla/show_bug.cgi?id=247





------- Comment #6 from mathieu.lacage at sophia.inria.fr  2008-07-10 18:03 -------
(In reply to comment #5)
> The TCP solution (change InitializeChecksum) is interresting and easy to do.
> 
> The UDP header is always 8 bytes (i.e. there are no UDP options in header), am
> I right ? So no need to change it.

I think so, yes.

> 
> But the best would be to have a unique way for UDP/TCP. With that if others
> want to do a new L4 Protocol (such a SCTP), they have a model to follow.

yes.

> 
> Could you explain a little bit your way to proceed with
> Buffer::iterator::GetSize ? I mean in the different classes (*L4Protocol,

Well, if we provided a Buffer::Iterator::GetSize method which were to return
the size of the underlying byte buffer, you could use that in ::Deserialize and
::Serialize, and potentially, get rid of the SetPayloadSize method. 

I can't make myself like this very much but I suspect that it would pretty easy
to get up and running.

> *Header).
> 


-- 
Configure bugmail: http://www.nsnam.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.