You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@nuttx.apache.org by GitBox <gi...@apache.org> on 2020/04/30 07:09:59 UTC

[GitHub] [incubator-nuttx-apps] masayuki2009 opened a new pull request #221: wireless: gs2200m: Fix to release a bound socket

masayuki2009 opened a new pull request #221:
URL: https://github.com/apache/incubator-nuttx-apps/pull/221


   ## Summary
   
   - During testing https://github.com/apache/incubator-nuttx/pull/917, I found a bug in gs2200m daemon that a server socket is not released in close(). For example, kill tcpserver and then restarting tcpserver fails due to the internal cid is not released.
   - This PR introduces BOUND state in gs2200m_main.c to release in close_request().
   - Also, I did refactoring in close_request() so that it can handle both TCP/UDP socket in same logic.
   
   ## Impact
   
   - This PR affects gs2200m daemon only. However it affects all sockets. (i.e. TCP and UDP, client and server).
   
   ## Testing
   
   - I checked UDP server and client as well as TCP server and client.
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org