- check for pxDuplicateNetworkBufferWithDescriptor returning NULL #47
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
|
Thanks a lot, @RolinBert ! |
aggarg
previously approved these changes
Oct 26, 2020
AniruddhaKanhere
previously approved these changes
Oct 26, 2020
AniruddhaKanhere
dismissed stale reviews from aggarg and themself
via
edfad86
AniruddhaKanhere
approved these changes
Oct 26, 2020
aggarg
approved these changes
Oct 26, 2020
|
Thank you @RolinBert for spotting this issue and creating a PR to fix it! P.S. I had to push a couple of commits to rebase your PR and uncrustify the sources so that this PR can be merged. |
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.
Check for pxDuplicateNetworkBufferWithDescriptor returning NULL
Description
pxDuplicateNetworkBufferWithDescriptor in FreeRTOS_IP.c might return a NULL pointer => we need to check for this case and avoid dereferencing a NULL pointer.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.