RE: [iproute PATCH v2 7/7] ip/tcp_metrics: Simplify process_msg a bit

From: Jakub Sitnicki

Sent: 22 June 2016 12:34

...

quoted

-	a = attrs[TCP_METRICS_ATTR_ADDR_IPV4];
-	if (a) {
+	if ((a = attrs[TCP_METRICS_ATTR_ADDR_IPV4])) {

Copy the pointer inside the branch?

Same gain on indentation while keeping checkpatch happy.

Or as below (hacked from the mails):

	a = attrs[TCP_METRICS_ATTR_ADDR_IPV4];
	if (a) {
 		if (f.daddr.family && f.daddr.family != AF_INET)
 			return 0;
 		memcpy(&daddr.data, RTA_DATA(a), 4);
 		daddr.bytelen = 4;
 		family = AF_INET;
 		atype = TCP_METRICS_ATTR_ADDR_IPV4;
 		dlen = RTA_PAYLOAD(a);
	} else {
		a = attrs[TCP_METRICS_ATTR_ADDR_IPV6];
		if (!a)
			return 0;
		memcpy(&daddr.data, RTA_DATA(a), 16);
		daddr.bytelen = 16;
		family = AF_INET6;
		atype = TCP_METRICS_ATTR_ADDR_IPV6;
		dlen = RTA_PAYLOAD(a);
	}

	David