Hi Daniel,

On Sat, Dec 03, 2011 at 01:26:19PM +0100, Daniel Mack wrote:
> In case iptables_init() errors out early, 'table' is still NULL. Calling
> table_cleanup() on it causes a SIGSEGV in the daemon.

I would actually prefer to use 2 separate error labels.
Also, checking for the table pointer not being NULL from table_cleanup
wouldn't hurt.

Cheers,
Samuel.

-- 
Intel Open Source Technology Centre
http://oss.intel.com/
_______________________________________________
connman mailing list
[email protected]
http://lists.connman.net/listinfo/connman

Reply via email to