xiaoxiang781216 commented on code in PR #20362: URL: https://github.com/apache/nuttx/pull/20362#discussion_r4104809898
########## net/devif/devif_poll.c: ########## @@ -346,6 +346,17 @@ devif_poll_bluetooth_connections(FAR struct net_driver_s *dev, FAR struct bluetooth_conn_s *bluetooth_conn = NULL; int bstop = 0; + /* PF_BLUETOOTH sockets are polled through struct radio_driver_s, which + * only devices of Bluetooth link type are embedded in. Polling any + * other device here would reinterpret its struct net_driver_s as a + * radio driver and write through fields the device does not have. + */ + + if (dev->d_lltype != NET_LL_BLUETOOTH) Review Comment: but the origin patch state that this change doesn't required -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
