You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by "g2gps (via GitHub)" <gi...@apache.org> on 2023/06/27 03:05:33 UTC

[GitHub] [nuttx] g2gps opened a new pull request, #9625: net/udp: Populate the udp connection structure with the address family.

g2gps opened a new pull request, #9625:
URL: https://github.com/apache/nuttx/pull/9625

   ## Summary
   
   The udp connection structure contains the field, "domain", which defines which address family it belongs to. Prior to this change, this field was only populated correctly if IPv4 and IPv6 was enabled. As a result, packet information was not processed in udp_recvpktinfo, as expected when the appropriate socket option was enabled.
   
   This PR is actually a reworked version of #9294, which was closed when #9295 was merged. However the additions in #9295 did no solve the current issue.
   
   ## Impact
   
   The `conn->domain` field is correctly populated with the address family when only IPv6 is enabled. Further processing of UDP packets, i.e in `udp_recvpktinfo`, can correctly read the address domain and provide appropriate actions.
   
   ## Testing
   
   In our application:
   
   We are now able to fetch using in6_pktinfo using recvmsg. Previously this wasn't possible if only the IPV6 stack was enabled. When both IPV4 and IPV6 was enabled, this wasn't an issue.
   
   In NuttX:
   
   Using syslog messages to log the value of the udp_conn_s::domain field.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [nuttx] xiaoxiang781216 merged pull request #9625: net/udp: Populate the udp connection structure with the address family.

Posted by "xiaoxiang781216 (via GitHub)" <gi...@apache.org>.
xiaoxiang781216 merged PR #9625:
URL: https://github.com/apache/nuttx/pull/9625


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [nuttx] xiaoxiang781216 commented on pull request #9625: net/udp: Populate the udp connection structure with the address family.

Posted by "xiaoxiang781216 (via GitHub)" <gi...@apache.org>.
xiaoxiang781216 commented on PR #9625:
URL: https://github.com/apache/nuttx/pull/9625#issuecomment-1608906538

   Thanks, LGTM.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org