Julia Lawall <julia.lawall@xxxxxxx> wrote: > The function xt_find_table_lock defined in net/netfilter/x_tables.c is > preceeded by a comment that says that it returns ERR_PTR() on error. But > looking at the definition, I only see occurrences of return NULL and > returns of pointers that have previously been dereferenced. Is it the > code or the documentation that is incorrect? The call sites seem to be > using IS_ERR_OR_NULL. Is there a plan to return ERR_PTR values in the > future? It used to return ERR_PTR, see: commit 7926dbfa4bc14e27f4e18a6184a031a1c1e077dc netfilter: don't use mutex_lock_interruptible() So the comment isn't correct anymore and callers could test vs NULL. -- 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