On Fri, Aug 25, 2017 at 01:17:32PM +0200, Phil Sutter wrote: > The common paradigm here is that all parser rules converting string > tokens into symbols must free the string token if it's not used anymore. > This is unrelated to the %destructor directive, since that will apply > only if the parser discards the token, which is not the case then. > > While being at it, simplify error handling in parser rule for listing > conntrack helpers (error() won't return NULL) and drop the unused extra > parameter passed to error() in level_type rule. Applied, thanks Phil. -- To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html