Skip to content

[Coverity CID :188173] Null pointer dereferences in /subsys/net/ip/route.c #10090

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.

Sign up for GitHub

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

Jump to bottom
Closed
mandarcthorat1 opened this issue Sep 19, 2018 · 0 comments
Closed

[Coverity CID :188173] Null pointer dereferences in /subsys/net/ip/route.c #10090

mandarcthorat1 opened this issue Sep 19, 2018 · 0 comments
Labels
area: Networking bug The issue is a bug, or the PR is fixing a bug Coverity A Coverity detected issue or its fix priority: medium Medium impact/importance bug

Comments

@mandarcthorat1
Copy link
Contributor

Static code scan issues seen in File: /subsys/net/ip/route.c
Category: Null pointer dereferences
Function: net_route_del_by_nexthop_data
Component: Networking
CID: 188173
Please fix or provide comments to square it off in coverity in the link: https://scan9.coverity.com/reports.htm#v32951/p12996
@mandarcthorat1 mandarcthorat1 added area: Networking bug The issue is a bug, or the PR is fixing a bug Coverity A Coverity detected issue or its fix labels Sep 19, 2018
@nashif nashif added the priority: medium Medium impact/importance bug label Sep 19, 2018
jukkar added a commit to jukkar/zephyr that referenced this issue Sep 21, 2018
@jukkar
net: route: Check that neighbor exists before deleting it
b378a1e 
If the neighbour does not exists, then the route to it cannot
be deleted so we can return error to caller in that case.

Coverity-CID: 188173
Fixes zephyrproject-rtos#10090

Signed-off-by: Jukka Rissanen <[email protected]>
@jukkar jukkar closed this as completed in 02e9f9c Sep 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area: Networking bug The issue is a bug, or the PR is fixing a bug Coverity A Coverity detected issue or its fix priority: medium Medium impact/importance bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@nashif @mandarcthorat1