Check incoming IPv6 multicast scope packets #10995
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
If we receive an IPv6 packet with interface scope multicast address FF01:: then we must drop it as those addresses are reserved for local network traffic only. Fixes zephyrproject-rtos#10959 Signed-off-by: Jukka Rissanen <jukka.rissanen@linux.intel.com>
Test that we drop the received packet if the destination address is interface scope multicast address. Signed-off-by: Jukka Rissanen <jukka.rissanen@linux.intel.com>
The zero scope is a reserved value so it should not be used so drop it. Fixes zephyrproject-rtos#10958 Signed-off-by: Jukka Rissanen <jukka.rissanen@linux.intel.com>
Test that we drop the received packet if the destination address is zero scope (ff00::) multicast address. Signed-off-by: Jukka Rissanen <jukka.rissanen@linux.intel.com>
If we receive an IPv6 packet with site scope multicast address FF05:: then we must drop it as those addresses are reserved for site network traffic only. Fixes zephyrproject-rtos#10960 Signed-off-by: Jukka Rissanen <jukka.rissanen@linux.intel.com>
Test that we drop the received packet if the destination address is site scope (ff05::) multicast address. Signed-off-by: Jukka Rissanen <jukka.rissanen@linux.intel.com>
If we receive an IPv6 packet with organisation scope multicast address FF08:: then we must drop it as those addresses are reserved for organisation network traffic only. Fixes zephyrproject-rtos#10961 Signed-off-by: Jukka Rissanen <jukka.rissanen@linux.intel.com>
Test that we drop the received packet if the destination address is organisation scope (ff08::) multicast address. Signed-off-by: Jukka Rissanen <jukka.rissanen@linux.intel.com>
Codecov Report
@@ Coverage Diff @@
## master #10995 +/- ##
=========================================
Coverage ? 53.25%
=========================================
Files ? 217
Lines ? 26692
Branches ? 5923
=========================================
Hits ? 14215
Misses ? 10059
Partials ? 2418
Continue to review full report at Codecov.
|
tbursztyka
approved these changes
Nov 2, 2018
Yes, I will do that. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.