On Tue, Nov 25, 2014 at 03:59:37PM +0300, Dan Carpenter wrote: > smatch/check_deref.c > 179 static void match_assign(struct expression *expr) > 180 { > 181 struct statement *stmt; > 182 > 183 if (!is_zero(expr->right)) > 184 return; > 185 > 186 FOR_EACH_PTR_REVERSE(big_statement_stack, stmt) { > 187 if (stmt->type == STMT_DECLARATION) > 188 return; > 189 break; Gar. Also you can't call break inside a FOR_EACH_PTR* loop. I'll test a fix for this. > 190 } END_FOR_EACH_PTR_REVERSE(stmt); regards, dan carpenter -- To unsubscribe from this list: send the line "unsubscribe smatch" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html