On Thu, Jun 27, 2019 at 12:50:00PM +0200, Arturo Borrero Gonzalez wrote: > In the current setup, nft (the frontend object) is using the xzalloc() function > from libnftables, which does not makes sense, as this is typically an internal > helper function. > > In order to don't use this public libnftables symbol (a later patch just > removes it), let's introduce a new allocation function in the nft frontend. > This results in a bit of code duplication, but given the simplicity of the code, > I don't think it's a big deal. > > Other possible approach would be to have xzalloc() become part of libnftables > public API, but that is a much worse scenario I think. Could you replace the call to xzalloc() in main.c by calloc()? Also check for the error if calloc() fails, then print error and exit(). Just like other error path in main.c Thanks!
