WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#54: 
new file mode 100644

WARNING: line length of 100 exceeds 80 columns
#129: FILE: tools/testing/selftests/ovpn/float-test.sh:25:
+			ip link add veth${p} netns peer0 type veth peer name veth${p} netns peer${p}

WARNING: line length of 91 exceeds 80 columns
#147: FILE: tools/testing/selftests/ovpn/float-test.sh:43:
+			ip netns exec peer0 $OVPN_CLI new_multi_peer tun0 1 $UDP_PEERS_FILE

WARNING: line length of 120 exceeds 80 columns
#150: FILE: tools/testing/selftests/ovpn/float-test.sh:46:
+			#	ip netns exec peer0 $OVPN_CLI new_peer tun0 ${p} ${p} 10.10.${p}.2 1 5.5.5.$((${p} + 1))

WARNING: line length of 97 exceeds 80 columns
#151: FILE: tools/testing/selftests/ovpn/float-test.sh:47:
+				ip netns exec peer0 $OVPN_CLI new_key tun0 ${p} $ALG 0 data64.key

WARNING: line length of 103 exceeds 80 columns
#154: FILE: tools/testing/selftests/ovpn/float-test.sh:50:
+			ip netns exec peer${1} $OVPN_CLI new_peer tun${1} 1 ${1} 10.10.${1}.1 1 5.5.5.1

WARNING: line length of 95 exceeds 80 columns
#155: FILE: tools/testing/selftests/ovpn/float-test.sh:51:
+			ip netns exec peer${1} $OVPN_CLI new_key tun${1} ${1} $ALG 1 data64.key

WARNING: line length of 90 exceeds 80 columns
#159: FILE: tools/testing/selftests/ovpn/float-test.sh:55:
+			(ip netns exec peer$1 $OVPN_CLI listen tun0 1 $TCP_PEERS_FILE && {

WARNING: line length of 105 exceeds 80 columns
#161: FILE: tools/testing/selftests/ovpn/float-test.sh:57:
+					ip netns exec peer0 $OVPN_CLI new_key tun0 ${p} $ALG 0 data64.key

WARNING: line length of 100 exceeds 80 columns
#166: FILE: tools/testing/selftests/ovpn/float-test.sh:62:
+			ip netns exec peer${1} $OVPN_CLI connect tun${1} ${1} 10.10.${1}.1 1 5.5.5.1

WARNING: line length of 95 exceeds 80 columns
#167: FILE: tools/testing/selftests/ovpn/float-test.sh:63:
+			ip netns exec peer${1} $OVPN_CLI new_key tun${1} ${1} $ALG 1 data64.key

WARNING: line length of 88 exceeds 80 columns
#177: FILE: tools/testing/selftests/ovpn/float-test.sh:73:
+		ip netns exec peer${p} ${OVPN_CLI} del_iface tun${p} 2>/dev/null || true

WARNING: line length of 100 exceeds 80 columns
#248: FILE: tools/testing/selftests/ovpn/netns-test.sh:25:
+			ip link add veth${p} netns peer0 type veth peer name veth${p} netns peer${p}

WARNING: line length of 91 exceeds 80 columns
#266: FILE: tools/testing/selftests/ovpn/netns-test.sh:43:
+			ip netns exec peer0 $OVPN_CLI new_multi_peer tun0 1 $UDP_PEERS_FILE

WARNING: line length of 120 exceeds 80 columns
#269: FILE: tools/testing/selftests/ovpn/netns-test.sh:46:
+			#	ip netns exec peer0 $OVPN_CLI new_peer tun0 ${p} ${p} 10.10.${p}.2 1 5.5.5.$((${p} + 1))

WARNING: line length of 97 exceeds 80 columns
#270: FILE: tools/testing/selftests/ovpn/netns-test.sh:47:
+				ip netns exec peer0 $OVPN_CLI new_key tun0 ${p} $ALG 0 data64.key

WARNING: line length of 103 exceeds 80 columns
#273: FILE: tools/testing/selftests/ovpn/netns-test.sh:50:
+			ip netns exec peer${1} $OVPN_CLI new_peer tun${1} 1 ${1} 10.10.${1}.1 1 5.5.5.1

WARNING: line length of 95 exceeds 80 columns
#274: FILE: tools/testing/selftests/ovpn/netns-test.sh:51:
+			ip netns exec peer${1} $OVPN_CLI new_key tun${1} ${1} $ALG 1 data64.key

WARNING: line length of 89 exceeds 80 columns
#278: FILE: tools/testing/selftests/ovpn/netns-test.sh:55:
+			(ip netns exec peer0 $OVPN_CLI listen tun0 1 $TCP_PEERS_FILE && {

WARNING: line length of 105 exceeds 80 columns
#280: FILE: tools/testing/selftests/ovpn/netns-test.sh:57:
+					ip netns exec peer0 $OVPN_CLI new_key tun0 ${p} $ALG 0 data64.key

WARNING: line length of 100 exceeds 80 columns
#285: FILE: tools/testing/selftests/ovpn/netns-test.sh:62:
+			ip netns exec peer${1} $OVPN_CLI connect tun${1} ${1} 10.10.${1}.1 1 5.5.5.1

WARNING: line length of 95 exceeds 80 columns
#286: FILE: tools/testing/selftests/ovpn/netns-test.sh:63:
+			ip netns exec peer${1} $OVPN_CLI new_key tun${1} ${1} $ALG 1 data64.key

WARNING: line length of 86 exceeds 80 columns
#296: FILE: tools/testing/selftests/ovpn/netns-test.sh:73:
+		ip netns exec peer${p} $OVPN_CLI del_iface tun${p} 2>/dev/null || true

WARNING: __always_unused or __maybe_unused is preferred over __attribute__((__unused__))
#547: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:200:
+static int ovpn_nl_cb_error(struct sockaddr_nl (*nla)__attribute__((unused)),

WARNING: __always_unused or __maybe_unused is preferred over __attribute__((__unused__))
#582: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:235:
+static int ovpn_nl_cb_finish(struct nl_msg (*msg)__attribute__((unused)),

WARNING: Missing a blank line after declarations
#789: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:442:
+	int opt = 1;
+	ret = setsockopt(s, SOL_SOCKET, SO_REUSEADDR, &opt, sizeof(opt));

WARNING: line length of 82 exceeds 80 columns
#803: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:456:
+		if (setsockopt(s, IPPROTO_IPV6, IPV6_V6ONLY, &opt, sizeof(opt))) {

WARNING: line length of 86 exceeds 80 columns
#940: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:593:
+			fprintf(stderr, "Invalid family for remote socket address\n");

WARNING: line length of 87 exceeds 80 columns
#943: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:596:
+		NLA_PUT(ctx->nl_msg, OVPN_A_PEER_SOCKADDR_REMOTE, alen, &ovpn->remote);

CHECK: Please don't use multiple blank lines
#946: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:599:
+
+

WARNING: line length of 83 exceeds 80 columns
#953: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:606:
+		NLA_PUT(ctx->nl_msg, OVPN_A_PEER_VPN_IPV6, sizeof(struct in6_addr),

WARNING: line length of 83 exceeds 80 columns
#1037: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:690:
+		inet_ntop(AF_INET, nla_data(attrs_peer[OVPN_A_PEER_VPN_IPV4]), buf,

WARNING: Missing a blank line after declarations
#1037: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:690:
+		char buf[INET_ADDRSTRLEN];
+		inet_ntop(AF_INET, nla_data(attrs_peer[OVPN_A_PEER_VPN_IPV4]), buf,

WARNING: line length of 84 exceeds 80 columns
#1044: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:697:
+		inet_ntop(AF_INET6, nla_data(attrs_peer[OVPN_A_PEER_VPN_IPV6]), buf,

WARNING: Missing a blank line after declarations
#1044: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:697:
+		char buf[INET6_ADDRSTRLEN];
+		inet_ntop(AF_INET6, nla_data(attrs_peer[OVPN_A_PEER_VPN_IPV6]), buf,

WARNING: line length of 85 exceeds 80 columns
#1065: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:718:
+					  nla_data(attrs_peer[OVPN_A_PEER_LOCAL_IP]),

WARNING: line length of 87 exceeds 80 columns
#1071: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:724:
+			fprintf(stderr, "\tRemote: %s:%u\n", buf, ntohs(in->sin_port));

WARNING: line length of 85 exceeds 80 columns
#1077: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:730:
+					  nla_data(attrs_peer[OVPN_A_PEER_LOCAL_IP]),

WARNING: line length of 82 exceeds 80 columns
#1084: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:737:
+				ntohs(in6->sin6_port), ntohl(in6->sin6_scope_id));

WARNING: line length of 81 exceeds 80 columns
#1090: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:743:
+			nla_get_u32(attrs_peer[OVPN_A_PEER_KEEPALIVE_INTERVAL]));

WARNING: line length of 85 exceeds 80 columns
#1330: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:983:
+			    grp->group, nla_len(tb_mcgrp[CTRL_ATTR_MCAST_GRP_NAME])))

WARNING: Block comments use * on subsequent lines
#1393: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:1046:
+		/*if (!attrs[OVPN_A_DEL_PEER_REASON]) {
+			fprintf(stderr, "no reason in DEL_PEER message\n");

WARNING: line length of 123 exceeds 80 columns
#1517: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:1170:
+	fprintf(stderr, "* connect <peer_id> <raddr> <rport> <vpnaddr>: start connecting peer of TCP-based VPN session\n");

WARNING: line length of 101 exceeds 80 columns
#1523: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:1176:
+	fprintf(stderr, "* listen <lport> <peers_file>: listen for incoming peer TCP connections\n");

WARNING: line length of 91 exceeds 80 columns
#1525: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:1178:
+	fprintf(stderr, "\tpeers_file: file containing one peer per line: Line format:\n");

WARNING: line length of 98 exceeds 80 columns
#1528: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:1181:
+	fprintf(stderr, "* new_peer <lport> <peer-id> <raddr> <rport> <vpnaddr>: add new peer\n");

WARNING: line length of 93 exceeds 80 columns
#1529: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:1182:
+	fprintf(stderr, "\tpeer-id: peer ID to be used in data packets to/from this peer\n");

WARNING: line length of 103 exceeds 80 columns
#1535: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:1188:
+	fprintf(stderr, "* new_multi_peer <lport> <file>: add multiple peers as listed in the file\n");

WARNING: line length of 90 exceeds 80 columns
#1537: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:1190:
+	fprintf(stderr, "\tfile: text file containing one peer per line. Line format:\n");

WARNING: line length of 85 exceeds 80 columns
#1553: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:1206:
+	fprintf(stderr, "\tpeer-id: peer ID of the peer to configure the key for\n");

WARNING: line length of 82 exceeds 80 columns
#1560: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:1213:
+	fprintf(stderr, "* del_key <peer-id>: erase existing data channel key\n");

WARNING: line length of 88 exceeds 80 columns
#1563: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:1216:
+	fprintf(stderr, "* swap_keys <peer-id>: swap primary and seconday key slots\n");

WARNING: line length of 91 exceeds 80 columns
#1566: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:1219:
+	fprintf(stderr, "* listen_mcast: listen to ovpn-dco netlink multicast messages\n");

WARNING: line length of 90 exceeds 80 columns
#1569: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:1222:
+static int ovpn_parse_remote(struct ovpn_ctx *ovpn, const char *host, const char *service,

WARNING: line length of 106 exceeds 80 columns
#1585: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:1238:
+		if (!(result->ai_family == AF_INET && result->ai_addrlen == sizeof(struct sockaddr_in)) &&

WARNING: line length of 108 exceeds 80 columns
#1586: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:1239:
+		    !(result->ai_family == AF_INET6 && result->ai_addrlen == sizeof(struct sockaddr_in6))) {

WARNING: line length of 98 exceeds 80 columns
#1598: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:1251:
+	if (!(result->ai_family == AF_INET && result->ai_addrlen == sizeof(struct sockaddr_in)) &&

WARNING: line length of 100 exceeds 80 columns
#1599: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:1252:
+	    !(result->ai_family == AF_INET6 && result->ai_addrlen == sizeof(struct sockaddr_in6))) {

WARNING: line length of 93 exceeds 80 columns
#1613: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:1266:
+static int ovpn_parse_new_peer(struct ovpn_ctx *ovpn, const char *peer_id, const char *raddr,

WARNING: Prefer strscpy, strscpy_pad, or __nonstring over strncpy - see: https://github.com/KSPP/linux/issues/90
#1662: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:1315:
+		strncpy(ovpn.ifname, argv[2], IFNAMSIZ - 1);

ERROR: space required after that close brace '}'
#1702: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:1355:
+	}else if (!strcmp(argv[1], "del_iface")) {

WARNING: line length of 89 exceeds 80 columns
#1752: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:1405:
+			ret = ovpn_parse_new_peer(&peer_ctx, peer_id, NULL, NULL, vpnip);

WARNING: line length of 99 exceeds 80 columns
#1760: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:1413:
+				fprintf(stderr, "cannot add peer to VPN: %s %s\n", peer_id, vpnip);

WARNING: line length of 85 exceeds 80 columns
#1772: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:1425:
+		ret = ovpn_parse_new_peer(&ovpn, argv[3], argv[4], argv[5], argv[6]);

WARNING: line length of 85 exceeds 80 columns
#1802: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:1455:
+		ret = ovpn_parse_new_peer(&ovpn, argv[4], argv[5], argv[6], argv[7]);

WARNING: line length of 94 exceeds 80 columns
#1841: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:1494:
+		while ((n = fscanf(fp, "%s %s %s %s\n", peer_id, raddr, rport, vpnip)) == 4) {

WARNING: line length of 91 exceeds 80 columns
#1848: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:1501:
+			ret = ovpn_parse_new_peer(&peer_ctx, peer_id, raddr, rport, vpnip);

WARNING: line length of 97 exceeds 80 columns
#1856: FILE: tools/testing/selftests/ovpn/ovpn-cli.c:1509:
+				fprintf(stderr, "cannot add peer to VPN: %s %s %s %s\n", peer_id,

total: 1 errors, 66 warnings, 1 checks, 1924 lines checked

NOTE: For some of the reported defects, checkpatch may be able to
      mechanically convert to the typical style using --fix or --fix-inplace.

Commit 8ddec8c317e4 ("testing/selftest: add test tool and scripts for ovpn module") has style problems, please review.

NOTE: Ignored message types: ALLOC_SIZEOF_STRUCT BAD_REPORTED_BY_LINK CAMELCASE COMMIT_LOG_LONG_LINE GIT_COMMIT_ID MACRO_ARG_REUSE NO_AUTHOR_SIGN_OFF

NOTE: If any of the errors are false positives, please report
      them to the maintainer, see CHECKPATCH in MAINTAINERS.