[GitHub] [incubator-nuttx] masayuki2009 opened a new pull request #3254: Fix arp

[GitBox <gi...@apache.org>]

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


   ## Summary
   
   - This PR consists of the following 2 commits
   - commit 1: net: arp: Fix memory corruption in arp_send()
      - In arp_send(), arp_wait_setup() adds a notify object to g_arp_waiters
         which is removed in arp_wait() in normal case.
      - However, in timeout and error cases, the object was not removed and
        caused memory corruption.
      - This commit fixes this issue.
   - commit 2: Fix a potential bug in arp_notify()
      - In arp_wait_setup() and arp_wait_cancel(), g_arp_waiters
         is protected by a critical section.
      - However, I noticed that arp_notify() does not protect the
         g_arp_waiters that would cause memory corruption
      - This commit fixes the issue.
   
   ## Impact
   
   - None
   
   ## Testing
   
   - Tested with spresense:rndis and spresense:rndis_smp
   


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

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #3254: Fix arp

[GitBox <gi...@apache.org>]

xiaoxiang781216 merged pull request #3254:
URL: https://github.com/apache/incubator-nuttx/pull/3254


   


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