Skip to content

udp: implement recvfrom(2) in os_sys_calls_impl#6160

Merged
alyssawilk merged 1 commit intoenvoyproxy:masterfrom
cmluciano:cml/sockrecvfrom
Mar 5, 2019
Merged

udp: implement recvfrom(2) in os_sys_calls_impl#6160
alyssawilk merged 1 commit intoenvoyproxy:masterfrom
cmluciano:cml/sockrecvfrom

Conversation

@cmluciano
Copy link
Copy Markdown
Member

Signed-off-by: Christopher M. Luciano cmluciano@us.ibm.com

Description: implement recvfrom(2) in os_sys_calls and use in udp_listener_impl
Risk Level: Low - same functionality but different implementation
Testing: bazel test //test/common/network:udp_listener_impl_test
Docs Changes: N/A
Release Notes: N/A

Signed-off-by: Christopher M. Luciano <cmluciano@us.ibm.com>
@cmluciano
Copy link
Copy Markdown
Member Author

/assign @alyssawilk @conqerAtapple

@conqerAtapple
Copy link
Copy Markdown
Contributor

LGTM.

Copy link
Copy Markdown
Contributor

@alyssawilk alyssawilk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for following up on this!

@alyssawilk alyssawilk merged commit a957da5 into envoyproxy:master Mar 5, 2019
@cmluciano cmluciano deleted the cml/sockrecvfrom branch March 5, 2019 19:08
fredlas pushed a commit to fredlas/envoy that referenced this pull request Mar 5, 2019
Description: implement recvfrom(2) in os_sys_calls and use in udp_listener_impl
Risk Level: Low - same functionality but different implementation
Testing: bazel test //test/common/network:udp_listener_impl_test
Docs Changes: N/A
Release Notes: N/A

Signed-off-by: Christopher M. Luciano <cmluciano@us.ibm.com>
Signed-off-by: Fred Douglas <fredlas@google.com>
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