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

The ofconn_send_error in handle_flow_monitor_request uses the inner layer, which may cause heap overflow #312

Open
cotyleton opened this issue Nov 24, 2023 · 0 comments

Comments

@cotyleton
Copy link

The ofp header has been pulled at the ofputil_decode_flow_monitor_request, b->data points to the inner layer. As a result, if we need to send error reply, the ofconn_send_error would incorrectly processe the inner layer as the message header. An incorrect length would be used when memcpy request to reply.
image

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

No branches or pull requests

1 participant