Skip to content

xlate: nft: bf_nfgroup automatically set NLM_F_MULTI flag on multipart messages#50

Merged
qdeslandes merged 1 commit into
facebook:mainfrom
qdeslandes:nft_nfgroup_multipart
Feb 22, 2024
Merged

xlate: nft: bf_nfgroup automatically set NLM_F_MULTI flag on multipart messages#50
qdeslandes merged 1 commit into
facebook:mainfrom
qdeslandes:nft_nfgroup_multipart

Conversation

@qdeslandes
Copy link
Copy Markdown
Contributor

When a group of Netfilter Netlink messages is converted to a response, the response will contain a multipart message if the group contains more than one message. The NLM_F_MULTI flag will be set on all the messages and a final NLMSG_DONE message will be appended to the response.

…t responses

When a group of Netfilter Netlink messages is converted to a response,
the response will contain a multipart message if the group contains more
than one message. The NLM_F_MULTI flag will be set on all the messages
and a final NLMSG_DONE message will be appended to the response.

Signed-off-by: Quentin Deslandes <qde@naccy.de>
@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 22, 2024

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (49c61ba) 22.28% compared to head (87cdbb5) 23.04%.

Files Patch % Lines
src/xlate/nft/nfgroup.c 91.66% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #50      +/-   ##
==========================================
+ Coverage   22.28%   23.04%   +0.76%     
==========================================
  Files          38       38              
  Lines        2675     2703      +28     
==========================================
+ Hits          596      623      +27     
- Misses       2079     2080       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@qdeslandes qdeslandes merged commit 4d4b5ec into facebook:main Feb 22, 2024
@qdeslandes qdeslandes deleted the nft_nfgroup_multipart branch April 27, 2024 05:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants