-
Notifications
You must be signed in to change notification settings - Fork 153
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
FreeRTOS+TCP Zynq: check frame type before calculating ICMP checksum … #19
Merged
AniruddhaKanhere
merged 2 commits into
FreeRTOS:master
from
htibosch:Zynq__check_frame_type_before_calculating_ICMP_checksum_(v4)
Oct 21, 2020
Merged
FreeRTOS+TCP Zynq: check frame type before calculating ICMP checksum … #19
AniruddhaKanhere
merged 2 commits into
FreeRTOS:master
from
htibosch:Zynq__check_frame_type_before_calculating_ICMP_checksum_(v4)
Oct 21, 2020
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
…ICMP_checksum_(v4)
AniruddhaKanhere
approved these changes
Oct 21, 2020
aggarg
approved these changes
Oct 21, 2020
AniruddhaKanhere
added a commit
that referenced
this pull request
Oct 27, 2020
* Repair asynchronous DNS lookup (#37) Co-authored-by: Hein Tibosch <hein@htibosch.net> * WIFINetworkParams_t is changing. Update to conform (#36) * WIFINetworkParams_t is changing. Update to conform * uncrustify (v0.66) Co-authored-by: Aniruddha Kanhere <60444055+AniruddhaKanhere@users.noreply.github.com> * added a network interface for the LPC54018 ethernet (#39) * added a network interface for the LPC54018 ethernet Co-authored-by: Joseph Julicher <jjulicher@mac.com> * FreeRTOS+TCP Zynq: check frame type before calculating ICMP checksum (v4) (#19) Co-authored-by: Hein Tibosch <hein@htibosch.net> Co-authored-by: Aniruddha Kanhere <60444055+AniruddhaKanhere@users.noreply.github.com> * A network driver for Xilinx UltraScale+ 64-bits(v2) (#22) * A network driver for Xilinx UltraScale+ 64-bits(v2) * Uncrustify Co-authored-by: Hein Tibosch <hein@htibosch.net> Co-authored-by: AniruddhaKanhere <kanherea@amazon.com> Co-authored-by: Aniruddha Kanhere <60444055+AniruddhaKanhere@users.noreply.github.com> * Adding a network interface for STM32H7xx(v2) (#21) * Adding a network interface for STM32H7xx(v2) * Uncrustify * Uncrustify v2 * Remove trailing whitespace Co-authored-by: Hein Tibosch <hein@htibosch.net> Co-authored-by: AniruddhaKanhere <kanherea@amazon.com> Co-authored-by: Aniruddha Kanhere <60444055+AniruddhaKanhere@users.noreply.github.com> * FreeRTOS+TCP compatibility with 64-bit platforms (v2) (#20) * FreeRTOS+TCP compatibility with 64-bit platforms (v2) * Uncrustify Co-authored-by: Hein Tibosch <hein@htibosch.net> Co-authored-by: Aniruddha Kanhere <60444055+AniruddhaKanhere@users.noreply.github.com> Co-authored-by: AniruddhaKanhere <kanherea@amazon.com> * buffer fixes to the NetworkInterface for the LCP54018 (#41) * added a network interface for the LPC54018 ethernet * fixed the buffers so DHCP is now working * network buffers fixed * network buffers fixed Co-authored-by: Aniruddha Kanhere <60444055+AniruddhaKanhere@users.noreply.github.com> * Cleanup as a result of Hein's code review (#43) * added a network interface for the LPC54018 ethernet * fixed the buffers so DHCP is now working * network buffers fixed * network buffers fixed * added a network interface for the LPC54018 ethernet * improvements from Hein's comments, thanks * final changes based upon Hein's review * uncrustifying the NetworkInterface.c for LPC54018 Co-authored-by: Aniruddha Kanhere <60444055+AniruddhaKanhere@users.noreply.github.com> * Add spell check (#23) * Add the spell-check files * Add lexicon.txt file after generating it. * Update the allowed name list * Added some more words to the lexicon * final revision of additions * Update ci.yml * Update ci.yml * Delete lexicon.txt * Rename temp.txt to lexicon.txt * Ignore the portable directory * Update ci.yml * Update ci.yml * Update ci.yml * Update find-unknown-comment-words * Update ci.yml * Make all letters lowercase * Update the lexicon and fix some typos * Add more details and then add lexicon * remove spell directory from tools * Remove duplicate instances and sort the words * Update lexicon after Gary's comments (and test) * Update the words list * Correct word spelling * Update lexicon.txt Co-authored-by: Gary Wicker <14828980+gkwicker@users.noreply.github.com> * Synchronized the MAC address with main.c * Uncrustify LPC54018 network interface * removed extra prints (#48) * Uncrustify (#44) * Create manifest.yml * Remove trailing whitespace * Update manifest.yml * Corrected a typo * Uncrustify C files * Uncrustify header files * Update ci.yml * Update ci.yml * Uncrustify with version 0.67 * Spelling corrections * uncrustify run * - check for pxDuplicateNetworkBufferWithDescriptor returning NULL (#47) * - check for pxDuplicateNetworkBufferWithDescriptor returning NULL * Uncrustify the sources Co-authored-by: Robert Korn <Robert@jrkorn.de> Co-authored-by: Aniruddha Kanhere <60444055+AniruddhaKanhere@users.noreply.github.com> Co-authored-by: AniruddhaKanhere <kanherea@amazon.com> * Added CODEOWNERS to require PRBR review (#50) * Add comments for Doxygen and add doxygen check (#46) * Create manifest.yml * Remove trailing whitespace * Update manifest.yml * Corrected a typo * Add the config file * Update doxygen and Update some files * Add the CI check * Add documentation for doxygen * Uncrustify * temp * Add the files individually * Update * Added 2 c files * Added 2 c files v2 * Added DNS and IP * Added more comments for doxygen * Added more comments * Sockets.c added * Sockets and TCP_IP * Added UDP_IP.c * Remove doxygen output * Remove a typo * Spelling corrections * Some spelling corrections * Update ci.yml * Add the TCP_WIN.c file * Add missing function description * update lexicon and fix some typos * Removed duplicates, sort and make lower-case * Update lexicon * Uncrustify * Minor Uncrustify changes * Update lexicon * removed extra prints (#48) * Uncrustify * Fix conflict resolution issues * Uncrustify * Add missing briefs and update based on @htibosch's comments. * Fix spellings and update lexicon * uncrustify * Minor fixes Co-authored-by: root <root@ip-172-31-10-225.us-west-2.compute.internal> Co-authored-by: Joseph Julicher <jjulicher@mac.com> Co-authored-by: Hein Tibosch <hein_tibosch@yahoo.es> Co-authored-by: Hein Tibosch <hein@htibosch.net> Co-authored-by: David Chalco <59750547+dachalco@users.noreply.github.com> Co-authored-by: Carl Lundin <53273776+lundinc2@users.noreply.github.com> Co-authored-by: Joseph Julicher <jjulicher@mac.com> Co-authored-by: Gary Wicker <14828980+gkwicker@users.noreply.github.com> Co-authored-by: RolinBert <JonnyKorn@gmx.de> Co-authored-by: Robert Korn <Robert@jrkorn.de> Co-authored-by: root <root@ip-172-31-10-225.us-west-2.compute.internal>
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.
Description
This PR will replace the earlier FreeRTOS PR #177, which will be closed.
The only significant change is in the function
xNetworkInterfaceOutput()
, in which the frame-type is checked before checking the protocol type.I the current driver, outgoing ARP frames get a size warning. It doesn't cause incorrect behaviour, except for the size warnings in the logging.
Test Steps
Define
ipconfigDRIVER_INCLUDED_TX_IP_CHECKSUM
as1
and send an outgoing ping. The ARP before the PING may not trigger a size warning, and the ICMP checksum shall be calculated correctly.Related Issue
None.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.