On Mon, Jul 20, 2015 at 01:43:40PM -0700, Stephen Hemminger wrote: > On Mon, 20 Jul 2015 22:46:23 +0300 > Vadim Kochan <[email protected]> wrote: > > > + if (s->name) > > + free(s->name); > > Please don't add unnecessary conditional. > free(NULL) is defined to do nothing.
OK, I will re-send, this was from original version. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
