Skip to content

Conversation

@masayuki2009
Copy link
Contributor

Summary

  • This PR reinforces parameter check in getsockname()
  • With this fix, an error in BasicGetSockName test in usrsocktest will be fixed.

Impact

  • This PR affects network programs which use getsockname()
  • However, this is a minor error case, so existing application should work.

Limitations / TODO

  • There still remain errors in usrsocktest and need more investigation.

Testing

  • I tested this PR with spresense:wifi which I added usrsocktest application.

Signed-off-by: Masayuki Ishikawa <Masayuki.Ishikawa@jp.sony.com>
@xiaoxiang781216 xiaoxiang781216 merged commit e9fe1f8 into apache:master Feb 21, 2020
@masayuki2009 masayuki2009 deleted the fix_socket branch March 20, 2020 01:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants