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

sendUdp uses gethostbyname #5

Closed
skliper opened this issue Jun 24, 2019 · 3 comments · Fixed by #26
Closed

sendUdp uses gethostbyname #5

skliper opened this issue Jun 24, 2019 · 3 comments · Fixed by #26
Assignees
@avan989
Labels
enhancement New feature or request
Milestone
Aquila

Comments

@skliper
Copy link
Contributor

skliper commented Jun 24, 2019

Originated by glimes (33 on babelfish):

cppcheck identifies gethostbyname as nonreentrant and obsolete, and recommends gethostbyname_r in the first case and getaddrinfo in the second.
@skliper skliper added the enhancement New feature or request label Jun 24, 2019
@skliper
Copy link
Contributor Author

skliper commented Jul 2, 2019

set gitaddrinfo(3) man page client program for example

@skliper
Copy link
Contributor Author

skliper commented Aug 26, 2019

@avan989 Take a look at this on

@avan989
Copy link
Contributor

avan989 commented Aug 26, 2019

Can you clarify the example?

skliper pushed a commit that referenced this issue Sep 6, 2019
@avan989 @skliper
Fix #5, Replace gethostbyname use
fac2dc5
skliper added a commit that referenced this issue Sep 6, 2019
@skliper
CCB 2019-09-04: Merge #24 #4 #12 #6 #11 #5 #7 #14 #1
ae911ab 
Reviewed and approved at 2019-09-04 CCB
@skliper skliper mentioned this issue Sep 6, 2019
Merged
@skliper skliper added this to the 2.1.0 milestone Sep 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@avan989 avan989

Labels
enhancement New feature or request
Projects
None yet
Milestone
Aquila
Development

Successfully merging a pull request may close this issue.

Integration Candidate 20190906 nasa/cFS-GroundSystem
2 participants
@avan989 @skliper