Blame SOURCES/0119-print_fields.h-add-PRINT_FIELD_LEN-macro.patch

d811e0
From 8806a21b612d76c8732d0df2ebaf52b62166ce51 Mon Sep 17 00:00:00 2001
d811e0
From: =?UTF-8?q?=C3=81kos=20Uzonyi?= <uzonyi.akos@gmail.com>
d811e0
Date: Sat, 13 Jun 2020 18:18:31 +0200
d811e0
Subject: [PATCH 119/138] print_fields.h: add PRINT_FIELD_LEN macro
d811e0
d811e0
* print_fields.h (PRINT_FIELD_LEN): New macro.
d811e0
* net.c: (print_get_linger, print_get_ucred, print_tpacket_stats):
d811e0
Rewrite using PRINT_FIELD_LEN.
d811e0
---
d811e0
 net.c          | 90 ++++++----------------------------------------------------
d811e0
 print_fields.h | 15 ++++++++++
d811e0
 2 files changed, 23 insertions(+), 82 deletions(-)
d811e0
d811e0
diff --git a/net.c b/net.c
d811e0
index a58fa92..9ea34b2 100644
d811e0
--- a/net.c
d811e0
+++ b/net.c
d811e0
@@ -579,24 +579,8 @@ print_get_linger(struct tcb *const tcp, const kernel_ulong_t addr,
d811e0
 	if (umoven_or_printaddr(tcp, addr, len, &linger))
d811e0
 		return;
d811e0
 
d811e0
-	if (len < sizeof(linger.l_onoff)) {
d811e0
-		tprints("{l_onoff=");
d811e0
-		print_quoted_string((void *) &linger.l_onoff,
d811e0
-				    len, QUOTE_FORCE_HEX);
d811e0
-	} else {
d811e0
-		PRINT_FIELD_D("{", linger, l_onoff);
d811e0
-
d811e0
-		if (len > offsetof(struct linger, l_linger)) {
d811e0
-			len -= offsetof(struct linger, l_linger);
d811e0
-			if (len < sizeof(linger.l_linger)) {
d811e0
-				tprints(", l_linger=");
d811e0
-				print_quoted_string((void *) &linger.l_linger,
d811e0
-						    len, QUOTE_FORCE_HEX);
d811e0
-			} else {
d811e0
-				PRINT_FIELD_D(", ", linger, l_linger);
d811e0
-			}
d811e0
-		}
d811e0
-	}
d811e0
+	PRINT_FIELD_LEN("{", linger, l_onoff, len, PRINT_FIELD_D);
d811e0
+	PRINT_FIELD_LEN(", ", linger, l_linger, len, PRINT_FIELD_D);
d811e0
 	tprints("}");
d811e0
 }
d811e0
 
d811e0
@@ -617,38 +601,9 @@ print_get_ucred(struct tcb *const tcp, const kernel_ulong_t addr,
d811e0
 	if (umoven_or_printaddr(tcp, addr, len, &uc))
d811e0
 		return;
d811e0
 
d811e0
-	if (len < sizeof(uc.pid)) {
d811e0
-		tprints("{pid=");
d811e0
-		print_quoted_string((void *) &uc.pid,
d811e0
-				    len, QUOTE_FORCE_HEX);
d811e0
-	} else {
d811e0
-		PRINT_FIELD_D("{", uc, pid);
d811e0
-
d811e0
-		if (len > offsetof(struct ucred, uid)) {
d811e0
-			len -= offsetof(struct ucred, uid);
d811e0
-			if (len < sizeof(uc.uid)) {
d811e0
-				tprints(", uid=");
d811e0
-				print_quoted_string((void *) &uc.uid,
d811e0
-						    len, QUOTE_FORCE_HEX);
d811e0
-			} else {
d811e0
-				PRINT_FIELD_UID(", ", uc, uid);
d811e0
-
d811e0
-				if (len > offsetof(struct ucred, gid) -
d811e0
-					  offsetof(struct ucred, uid)) {
d811e0
-					len -= offsetof(struct ucred, gid) -
d811e0
-					       offsetof(struct ucred, uid);
d811e0
-					if (len < sizeof(uc.gid)) {
d811e0
-						tprints(", gid=");
d811e0
-						print_quoted_string((void *) &uc.gid,
d811e0
-								    len,
d811e0
-								    QUOTE_FORCE_HEX);
d811e0
-					} else {
d811e0
-						PRINT_FIELD_UID(", ", uc, gid);
d811e0
-					}
d811e0
-				}
d811e0
-			}
d811e0
-		}
d811e0
-	}
d811e0
+	PRINT_FIELD_LEN("{", uc, pid, len, PRINT_FIELD_D);
d811e0
+	PRINT_FIELD_LEN(", ", uc, uid, len, PRINT_FIELD_UID);
d811e0
+	PRINT_FIELD_LEN(", ", uc, gid, len, PRINT_FIELD_UID);
d811e0
 	tprints("}");
d811e0
 }
d811e0
 
d811e0
@@ -688,38 +643,9 @@ print_tpacket_stats(struct tcb *const tcp, const kernel_ulong_t addr,
d811e0
 	if (umoven_or_printaddr(tcp, addr, len, &stats))
d811e0
 		return;
d811e0
 
d811e0
-	if (len < sizeof(stats.tp_packets)) {
d811e0
-		tprints("{tp_packets=");
d811e0
-		print_quoted_string((void *) &stats.tp_packets,
d811e0
-				    len, QUOTE_FORCE_HEX);
d811e0
-	} else {
d811e0
-		PRINT_FIELD_U("{", stats, tp_packets);
d811e0
-
d811e0
-		if (len > offsetof(struct tp_stats, tp_drops)) {
d811e0
-			len -= offsetof(struct tp_stats, tp_drops);
d811e0
-			if (len < sizeof(stats.tp_drops)) {
d811e0
-				tprints(", tp_drops=");
d811e0
-				print_quoted_string((void *) &stats.tp_drops,
d811e0
-						    len, QUOTE_FORCE_HEX);
d811e0
-			} else {
d811e0
-				PRINT_FIELD_U(", ", stats, tp_drops);
d811e0
-
d811e0
-				if (len > offsetof(struct tp_stats, tp_freeze_q_cnt) -
d811e0
-					  offsetof(struct tp_stats, tp_drops)) {
d811e0
-					len -= offsetof(struct tp_stats, tp_freeze_q_cnt) -
d811e0
-					       offsetof(struct tp_stats, tp_drops);
d811e0
-					if (len < sizeof(stats.tp_freeze_q_cnt)) {
d811e0
-						tprints(", tp_freeze_q_cnt=");
d811e0
-						print_quoted_string((void *) &stats.tp_freeze_q_cnt,
d811e0
-								    len,
d811e0
-								    QUOTE_FORCE_HEX);
d811e0
-					} else {
d811e0
-						PRINT_FIELD_U(", ", stats, tp_freeze_q_cnt);
d811e0
-					}
d811e0
-				}
d811e0
-			}
d811e0
-		}
d811e0
-	}
d811e0
+	PRINT_FIELD_LEN("{", stats, tp_packets, len, PRINT_FIELD_U);
d811e0
+	PRINT_FIELD_LEN(", ", stats, tp_drops, len, PRINT_FIELD_U);
d811e0
+	PRINT_FIELD_LEN(", ", stats, tp_freeze_q_cnt, len, PRINT_FIELD_U);
d811e0
 	tprints("}");
d811e0
 }
d811e0
 #endif /* PACKET_STATISTICS */
d811e0
diff --git a/print_fields.h b/print_fields.h
d811e0
index 02c56bf..70dbbff 100644
d811e0
--- a/print_fields.h
d811e0
+++ b/print_fields.h
d811e0
@@ -277,4 +277,19 @@
d811e0
 			       (size_), (hwtype_));			\
d811e0
 	} while (0)
d811e0
 
d811e0
+# define PRINT_FIELD_LEN(prefix_, where_, field_, 			\
d811e0
+			len_, print_func_, ...)				\
d811e0
+	do {								\
d811e0
+		unsigned int start = offsetof(typeof(where_), field_);	\
d811e0
+		unsigned int end = start + sizeof(where_.field_);	\
d811e0
+		if (len_ >= end) {					\
d811e0
+			print_func_(prefix_, where_, field_,		\
d811e0
+					##__VA_ARGS__);			\
d811e0
+		} else if (len_ > start) {				\
d811e0
+			tprintf("%s%s=", prefix_, #field_);		\
d811e0
+			print_quoted_string((void *)&where_.field_,	\
d811e0
+					len_ - start, QUOTE_FORCE_HEX);	\
d811e0
+		}							\
d811e0
+	} while (0)
d811e0
+
d811e0
 #endif /* !STRACE_PRINT_FIELDS_H */
d811e0
-- 
d811e0
2.1.4
d811e0