Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

- After adding tunnel headers, but before segmentation, packet length ca... #11

Merged
merged 1 commit into from
Dec 11, 2013
Merged

- After adding tunnel headers, but before segmentation, packet length ca... #11

merged 1 commit into from
Dec 11, 2013

Conversation

anandhk-juniper
Copy link
Contributor

...n be

more than 65535. So, pkt_len() should return an integer rather than a short.
Once segmentation happens, we anyway update the outer ip header length. So,
whatever is updated in between (during the course of ip header addition),
really doesn't matter.

  • Mirror bug: If the mirrored packet is TCP and is marked for GSO, we carry
    that gso type to mirror packet which is in-fact a UDP packet. So, correct the
    gso type, if the type is not what is expected for UDP packets

… can be

more than 65535. So, pkt_len() should return an integer rather than a short.
Once segmentation happens, we anyway update the outer ip header length. So,
whatever is updated in between (during the course of ip header addition),
really doesn't matter.

- Mirror bug: If the mirrored packet is TCP and is marked for GSO, we carry
that gso type to mirror packet which is in-fact a UDP packet. So, correct the
gso type, if the type is not what is expected for UDP packets
@ghost ghost assigned srajag Dec 11, 2013
srajag added a commit that referenced this pull request Dec 11, 2013
- After adding tunnel headers, but before segmentation, packet length ca...
@srajag srajag merged commit 2ef54a4 into Juniper:master Dec 11, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants