acassis commented on code in PR #20362: URL: https://github.com/apache/nuttx/pull/20362#discussion_r4120797744
########## 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: You are right Xiang, this is the same not needed fix like #18989. Since this c481374fb8d08ac1eb already added a check, doing it here is redundant. I will close this PR. -- 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]
