tree e91f44a1a698088f69523df7869ba08cedb10941
parent 0aff5c273daa16e5af234a904ba4a9cf6dc414a6
author Lorenzo Colitti <lorenzo@google.com> 1363880541 +0900
committer Lorenzo Colitti <lorenzo@google.com> 1365737741 +0900

Pass around packet data instead of fds

The current code calls all the translation functions one after
another, accumulating the translated packet into local variables
on the stack and calling writev() at the end. This does not allow
calling the translation functions re-entrantly, which is needed,
for example, to translate ICMP errors (which contain the
packet that caused them). Define a clat_packet type to wrap the
array of iovecs and an enum of packet positions.

Also clean up the code a bit: get rid of a fair bit of duplicated
code (though there is still some left), get rid of some redundant
memcpy statements, fix style issues, etc.

Bug: 8276725
Change-Id: Ib58d2348894e82275234fc67dbdb1f82753f204f