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.

Sign up for GitHub

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

Jump to bottom

DATAGRAM Network Function OS_SocketSendTo Segfault #544

Closed
yammajamma opened this issue Jul 17, 2020 · 1 comment · Fixed by #716 or #750
Closed

DATAGRAM Network Function OS_SocketSendTo Segfault #544

yammajamma opened this issue Jul 17, 2020 · 1 comment · Fixed by #716 or #750
Assignees
@jphickey
Labels
bug
Milestone
Caelum

Comments

@yammajamma
Copy link
Contributor

yammajamma commented Jul 17, 2020 •
edited
Loading

Describe the bug
OS_SocketSendTo() Segfaults when passed in NULL for Remote Address (last parameter) and all the other parameters are valid.

System observed on:
cFS Dev Server
OS: Ubuntu 18.04
Versions: OSAL 5.0.11.0

Reporter Info
Y. M. K.
Emergent
@skliper
Copy link
Contributor

skliper commented Dec 9, 2020

Relates to nasa/cFE#520, would be helpful to have the run time assert to check for null pointer.

@skliper skliper added this to the 6.0.0 milestone Dec 9, 2020
@skliper skliper added the bug label Dec 9, 2020
@skliper skliper mentioned this issue Dec 16, 2020
Merged
jphickey added a commit to jphickey/osal that referenced this issue Dec 29, 2020
@jphickey
Fix nasa#544, add pointer check
a8306ce 
Add parameter check to OS_SocketSendTo and adjust coverage test
to validate.
@jphickey jphickey mentioned this issue Dec 29, 2020
Merged
jphickey added a commit to jphickey/osal that referenced this issue Dec 29, 2020
@jphickey
Fix nasa#544, add pointer check
847a6d2 
Add parameter check to OS_SocketSendTo and adjust coverage test
to validate.
astrogeco added a commit that referenced this issue Jan 12, 2021
@astrogeco
Merge pull request #716 from jphickey/fix-544-sendto-pointer
08c1cf1 
Fix #544, add pointer check
@astrogeco astrogeco mentioned this issue Jan 13, 2021
Merged
jphickey pushed a commit to jphickey/osal that referenced this issue Aug 10, 2022
@astrogeco
Merge pull request nasa#544 from nasa/ic-20200304
00ec535 
Integration Candidate 20200304
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jphickey jphickey

Labels
bug
Projects
None yet
Milestone
Caelum
Development

Successfully merging a pull request may close this issue.

Fix #544, add pointer check jphickey/osal
osal Integration candidate: 2021-01-12 nasa/osal
3 participants
@jphickey @skliper @yammajamma