-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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
net: l2: ethernet: check dev->api and get_capabilities for being NULL #77841
net: l2: ethernet: check dev->api and get_capabilities for being NULL #77841
Conversation
8ff0eb4
to
996807d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good but the commit should split to three parts as you are also changing two other callbacks.
996807d
to
ff6750e
Compare
… NULL Adds missing checks for get_capabilities method not being NULL. Fixes crash with netusb and possibly other drivers. Signed-off-by: Maciej Panek <[email protected]>
ff6750e
to
248af28
Compare
adds a check for "send" method in Ethernet driver's API not being NULL in ethernet_send Signed-off-by: Maciej Panek <[email protected]>
adds a check for Ethernet driver's API not being NULL in ethernet_mcast_monitor_cb Signed-off-by: Maciej Panek <[email protected]>
248af28
to
49bb666
Compare
Hi @panekmaciej! To celebrate this milestone and showcase your contribution, we'd love to award you the Zephyr Technical Contributor badge. If you're interested, please claim your badge by filling out this form: Claim Your Zephyr Badge. Thank you for your valuable input, and we look forward to seeing more of your contributions in the future! 🪁 |
Adds missing checks for get_capabilities method not being NULL. Fixes crash with netusb and possibly other drivers.
See #77838