[nanomsg] Re: [PATCH] Initialize epbase item

  • From: Martin Sustrik <sustrik@xxxxxxxxxx>
  • To: nanomsg@xxxxxxxxxxxxx
  • Date: Mon, 04 Mar 2013 19:54:15 +0100

On 04/03/13 19:07, Nir Soffer wrote:

But I think that nn_list_item_erase needs some protection from junk
data. Either ignore removal of nil item, or assert that item is not
nil.

Also item->next is not initialized - maybe no code is failing now
because of this, but finally some code will. I don't think saving
initialization time worth the possible trouble.

Fair enough. Couple of nanoseconds is not worth it. I've fixed the trunk to address both points above.

Thanks!
Martin


Other related posts: