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

open5gs/lib/pfcp/message.h:line 1094 #510

Closed
lijhnihaoa opened this issue Jul 23, 2020 · 4 comments
Closed

open5gs/lib/pfcp/message.h:line 1094 #510

lijhnihaoa opened this issue Jul 23, 2020 · 4 comments
Labels
Housekeeping:ToClose Issues reviewed and closed. Old requests, issues which are not bug, feature or documentation request

Comments

@lijhnihaoa
Copy link

I think the member type of this structure should be ogs_pfcp_tlv_application_id_s_pfds_t,not ogs_pfcp_tlv_pfd_context_t. What do you think?

@lijhnihaoa
Copy link
Author

One more question, the macro in the program defines the ED series. Why are 8 bits in bytes affected by byte order?

acetcom added a commit that referenced this issue Jul 23, 2020
@acetcom
Copy link
Member

acetcom commented Jul 23, 2020

Hi @lijhnihaoa

  • PFD management request
    You're right!. I've fixed it in the master branch.
    Thank you so much!

  • How Endianness Effects Bitfield Packing
    I'm not sure if the answer to your question can be replaced by the link http://mjfrazer.org/mjfrazer/bitfields/. If you are trying to suggest something better than current ED method I implemented, please let me know.

Thanks again!
Sukchan

@lijhnihaoa
Copy link
Author

Thank you for understanding the effect of byte order on bit fields.

@pobk pobk added the Housekeeping:ToClose Issues reviewed and closed. Old requests, issues which are not bug, feature or documentation request label Feb 17, 2023
@github-actions
Copy link

This issue has been closed automatically due to lack of activity. This has been done to try and reduce the amount of noise. Please do not comment any further. The Open5GS Team may choose to re-open this issue if necessary.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Housekeeping:ToClose Issues reviewed and closed. Old requests, issues which are not bug, feature or documentation request
Projects
None yet
Development

No branches or pull requests

3 participants