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

refactor: use POSIX standard udphdr/tcphdr #261

Merged
merged 2 commits into from
Sep 7, 2022

Conversation

aloisklink
Copy link
Contributor

Refactors the packet_decoder so that the POSIX standard values for the udphdr and tcphdr struct are used.

On Linux, these structs are defined as a union, so that the POSIX standard can also be used. However, other platforms like FreeBSD only have the POSIX standard.

Refactors the packet_decoder so that the POSIX standard
values for the udphdr and tcphdr struct are used.

On Linux, these structs are defined as a union, so that
the POSIX standard can also be used. However, other platforms like
FreeBSD only have the POSIX standard.
@aloisklink aloisklink added the refactor Refactoring code label Sep 6, 2022
@codecov
Copy link

codecov bot commented Sep 6, 2022

Codecov Report

Merging #261 (b4731ff) into main (0aaf983) will increase coverage by 0.05%.
The diff coverage is 0.00%.

❗ Current head b4731ff differs from pull request most recent head aaeff71. Consider uploading reports for the commit aaeff71 to get more accurate results

@@            Coverage Diff             @@
##             main     #261      +/-   ##
==========================================
+ Coverage   48.51%   48.57%   +0.05%     
==========================================
  Files         113      113              
  Lines       17887    17920      +33     
==========================================
+ Hits         8678     8704      +26     
- Misses       9209     9216       +7     
Impacted Files Coverage Δ
...ure/middlewares/header_middleware/packet_decoder.c 0.00% <0.00%> (ø)
src/radius/wpabuf.h 79.41% <0.00%> (-0.59%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Copy link
Contributor

@mereacre mereacre left a comment

Choose a reason for hiding this comment

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

Great

@aloisklink aloisklink merged commit fa3873b into main Sep 7, 2022
@aloisklink aloisklink deleted the refactor/fix-udphdr-tcphdr-posix-compliance branch September 7, 2022 12:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor Refactoring code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants