v01d opened a new pull request #2302:
URL: https://github.com/apache/incubator-nuttx/pull/2302


   ## Summary
   
   This fixes a regression introduced in #2070 where the outgoing HCI message 
buffer was not freed anymore. At the same time, that PR introduced this buffer 
freeing step for the simulated HCI socket, right after it was sent to the real 
BT stack, since when NuttX's own bluetooth stack is disabled, at this point the 
buffer needs to be freed. These two changes compensated each other when NuttX's 
stack was disabled, but when enabled it resulted in the buffer being released 
at the wrong time.
   
   This reinstates the buffer freeing at the correct place needed for NuttX BLE 
stack and conditions the freeing of the buffer in the simulated HCI socket to 
only be done when NuttX stack is disabled.
   
   ## Impact
   
   Fixes regression, observed when NuttX's BLE stack is used with the sim HCI 
socket.
   
   ## Testing
   
   sim:bthcisock


----------------------------------------------------------------
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:
us...@infra.apache.org


Reply via email to