[lwip-devel] [bug #29375] Error in memp_malloc with CVS Head Revision

[Simon Goldschmidt]

Update of bug #29375 (project lwip):

                  Status:                    None => Works For Me           

    _______________________________________________________

Follow-up Comment #1:

Although there is currently no (debug-) code checking if an item to free is
not freed twice, looking at the code, your description strongly suggests there
is a problem with your mbox implementation: TCPIP_MSG_INPKT elements are only
allocated in tcpip_input() and freed in tcpip_input() (when sys_mbox_trypost()
returns != ERR_OK) and tcpip_thread() (after the message has been processed).

The code in tcpip_thread leaves virtually no chance to free a message twice,
but you should make sure that your sys_mbox_trypost() implementation either
enqueues the message passed and returns ERR_OK or does not enqueue the message
and returns an error. Otherwise the message might be freed both by
tcpip_input() and tcpip_thread() when sys_mbox_trypost() enqueues the message
but returns != ERR_OK.

I don't think this is a bug in lwIP, however, I'll leave this open as "works
for me" until you checked that.

    _______________________________________________________

Reply to this item at:

  <http://savannah.nongnu.org/bugs/?29375>

_______________________________________________
  Nachricht geschickt von/durch Savannah
  http://savannah.nongnu.org/