On Tue, 21 Jul 2020 18:31:57 +0100 Tom Parkin wrote:
> #if IS_ENABLED(CONFIG_IPV6)
> - if (info->attrs[L2TP_ATTR_IP6_SADDR] &&
> - info->attrs[L2TP_ATTR_IP6_DADDR]) {
> - cfg.local_ip6 = nla_data(
> - info->attrs[L2TP_ATTR_IP6_SADDR]);
> - cfg.peer_ip6 = nla_data(
> - info->attrs[L2TP_ATTR_IP6_DADDR]);
> - } else
> + if (attrs[L2TP_ATTR_IP6_SADDR] && attrs[L2TP_ATTR_IP6_DADDR]) {
> + cfg.local_ip6 = nla_data(attrs[L2TP_ATTR_IP6_SADDR]);
> + cfg.peer_ip6 = nla_data(attrs[L2TP_ATTR_IP6_DADDR]);
> + } else {
> #endif
This no longer builds. Probably because you added the closing backet
which wasn't there?
Please make sure each patch in the submission builds cleanly.
Please split this submission into series of at most 15 patches at a
time, to make sure reviewers don't get overloaded.
Please CC people who are working on the l2tp code (get_maintainers
script is your friend).
