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

gnrc_sock_udp: fix possibly dereferenced NULL pointer #6203

Merged
merged 1 commit into from
Dec 13, 2016

Conversation

miri64
Copy link
Member

@miri64 miri64 commented Dec 12, 2016

Fixes bug reported and offers a less intrusive alternative to #6196.

@miri64 miri64 added Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) GNRC Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer Area: network Area: Networking labels Dec 12, 2016
@kaspar030 kaspar030 added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Dec 12, 2016
Copy link
Contributor

@kaspar030 kaspar030 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's fix this independently of the discussion in #6196. ACK.

@kaspar030 kaspar030 merged commit 50b1e7b into RIOT-OS:master Dec 13, 2016
@miri64 miri64 deleted the gnrc_sock_udp/fix/sock-null-ptr branch December 13, 2016 15:54
bergzand pushed a commit to bergzand/RIOT that referenced this pull request Dec 31, 2016
kb2ma pushed a commit to kb2ma/RIOT that referenced this pull request Jan 14, 2017
neiljay pushed a commit to neiljay/RIOT that referenced this pull request Jan 16, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: network Area: Networking CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants