Well Lou added the code. The commit log referenced some memory issue that was found that this fixes. Might be worthwhile getting some feedback on that.
I'm not sure we should go carte-blanch back to what we have :) donald On Tue, May 3, 2016 at 2:03 PM, Christian Franke < ch...@opensourcerouting.org> wrote: > On 05/03/2016 07:46 PM, Donald Sharp wrote: > > You are right for the upstream behavior. The version that we have in > > the cumulus tree is this: > > > > 106 void * > > 107 zrealloc (int type, void *ptr, size_t size) > > 108 { > > 109 void *memory; > > 110 > > 111 memory = realloc (ptr, size); > > 112 if (memory == NULL) > > 113 zerror ("realloc", type, size); > > 114 if (ptr == NULL) > > 115 alloc_inc (type); > > 116 > > 117 return memory; > > 118 } > > > > > > Which would not work too well with your patch :) > > > > ah well. My mistake. > > > > acked-by: Donald Sharp <sha...@cumulusnetworks.com > > <mailto:sha...@cumulusnetworks.com>> > > Then let's stick with your version. Imho, either is fine and it's > probably good to keep merge conflicts low. > > -Christian > >
_______________________________________________ Quagga-dev mailing list Quagga-dev@lists.quagga.net https://lists.quagga.net/mailman/listinfo/quagga-dev