Skip to content
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: shell: Add IPv6 and IPv4 checks for ping command #11108

Merged
merged 1 commit into from Nov 6, 2018

Conversation

jukkar
Copy link
Member

@jukkar jukkar commented Nov 5, 2018

If IPv6 is disabled, then we can skip IPv6 checks and avoid
Coverity warnings. Same thing for IPv4.

Coverity-CID: 189506
Fixes #11100

Signed-off-by: Jukka Rissanen jukka.rissanen@linux.intel.com

If IPv6 is disabled, then we can skip IPv6 checks and avoid
Coverity warnings. Same thing for IPv4.

Coverity-CID: 189506
Fixes zephyrproject-rtos#11100

Signed-off-by: Jukka Rissanen <jukka.rissanen@linux.intel.com>
@codecov-io
Copy link

Codecov Report

Merging #11108 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master   #11108   +/-   ##
=======================================
  Coverage    53.1%    53.1%           
=======================================
  Files         218      218           
  Lines       26853    26853           
  Branches     5950     5950           
=======================================
  Hits        14259    14259           
  Misses      10162    10162           
  Partials     2432     2432

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7a18efe...75c1956. Read the comment docs.

@jukkar jukkar merged commit a13cee2 into zephyrproject-rtos:master Nov 6, 2018
@jukkar jukkar deleted the coverity branch November 6, 2018 09:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants