raiden00pl opened a new pull request, #3827:
URL: https://github.com/apache/nuttx-apps/pull/3827

   
   ## Summary
   
   NXMB_RTU_MIN_FRAME_SIZE was 5, but the shortest valid RTU frame is unit_id + 
fc + 2 CRC bytes = 4 bytes, e.g. a Report Server ID (FC17) request. Such 
requests were dropped and never answered. Set the minimum to 4. The ASCII 
transport already accepts the equivalent minimum frame (addr + fc + LRC).
   
   ## Impact
   
   bugfix
   
   ## Testing
   
   detected by automate tests from https://github.com/railab/nuttx-testlab
   
   
   


-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to