On Fri, Aug 4, 2017 at 4:09 PM, Luc Van Oostenryck <luc.vanoostenryck@xxxxxxxxx> wrote: > FOR_EACH_PTR(target->users, pu) { > + if (!pu) > + continue; Your previous patch looks fine. Here in my previous email feed back I mean to include this check into FOR_EACH_PTR() so you don't need touch every pseudo_user list looping every where. Just do some thing like if (list->rm && list->list[__nr] == NULL) continue; inside the FOR_EACH_PTR(). and REVERSE as well. It will not have impact on other list because other list will not have list->rm != 0. With this change, the caller side of the change not needed. Chris -- To unsubscribe from this list: send the line "unsubscribe linux-sparse" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html