Fragment questions

[Emeric POUPON]

Hello,

I noticed two questionable things in the fragmentation code:
- in ip_fragment, we do not copy the flowid from the original mbuf to the fragmented mbuf. Therefore we may output very desynchronized fragments (first fragment emitted far later the second fragment, etc.)
- in the ip_newid macro, we do "htons(V_ip_id++))" if we do not use randomized id. In multi core systems, we may emit successive packets with the same id.

Both problems combined lead to bad packet reassembly on the remote host.

What do you think?

Emeric