On Thursday 2015-02-19 14:37, Florian Westphal wrote: >Interesting, --help does mention it: >dst match options: >[!] --dst-len length total length of this header Um, I got confused by your change to dst-opts related code: >>> opts[i] = (parse_opts_num(cp, "opt") & 0xFF) << 8; if (range) { if (opts[i] == 0) xtables_error(PARAMETER_PROBLEM, "PAD0 hasn't got length"); opts[i] |= parse_opts_num(range, "length") & 0xFF; - } else + } else { opts[i] |= (0x00FF); + } <<< All in order. But perhaps separate style changes from real changes commit-wise. -- 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