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

Bugfix: Bump up libOpenflow version #5157

Merged

Conversation

wenyingd
Copy link
Contributor

@wenyingd wenyingd commented Jun 20, 2023

This is to resolve an issue when unmarshaling IP packet which may lead to incorrect field values in PacketIn meesages. The issue is because the existing code uses a pointer reference to set byte slice fields when unmarshaling the mesages, and libOpenlow reuses the buffer to process the next messages.

@wenyingd
Copy link
Contributor Author

/test-all

@tnqn
Copy link
Member

tnqn commented Jun 20, 2023

We don't need the fixes in ofnet?

@wenyingd
Copy link
Contributor Author

We don't need the fixes in ofnet?

No, the changes in ofnet is for PacketIn2 only, but we do not introduce PacketIn2 in antrea v1.11

@luolanzone
Copy link
Contributor

@wenyingd there is a typo in the PR and commit messages: PackintIn -> PacketIn

This is to resolve an issue when unmarshaling IP packet which may lead to
incorrect field values in PacketIn meesages. The issue is because the existing
code uses a pointer reference to set byte slice fields when unmarshaling the
mesages, and libOpenlow reuses the buffer to process the next messages.

Signed-off-by: wenyingd <wenyingd@vmware.com>
@wenyingd wenyingd force-pushed the unmarshal_byteslice_release1.11 branch from 1d9f36a to 035f8d2 Compare June 21, 2023 02:28
@wenyingd
Copy link
Contributor Author

/test-all

@tnqn tnqn merged commit d1c29cb into antrea-io:release-1.11 Jun 21, 2023
@wenyingd wenyingd deleted the unmarshal_byteslice_release1.11 branch April 3, 2024 03:30
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.

3 participants