[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <20240229125449.3268-1-dkirjanov@suse.de>
Date: Thu, 29 Feb 2024 07:54:49 -0500
From: Denis Kirjanov <kirjanov@...il.com>
To: stephen@...workplumber.org
Cc: netdev@...r.kernel.org,
Denis Kirjanov <dkirjanov@...e.de>
Subject: [PATCH iproute2 v2] iproute2: move generic_proc_open into lib
the function has the same definition in ifstat and ss
v2: fix the typo in the chnagelog
Signed-off-by: Denis Kirjanov <dkirjanov@...e.de>
---
include/utils.h | 2 ++
lib/utils.c | 14 ++++++++++++++
misc/nstat.c | 29 ++++++++---------------------
misc/ss.c | 13 -------------
4 files changed, 24 insertions(+), 34 deletions(-)
diff --git a/include/utils.h b/include/utils.h
index 9ba129b8..a2a98b9b 100644
--- a/include/utils.h
+++ b/include/utils.h
@@ -393,4 +393,6 @@ int proto_a2n(unsigned short *id, const char *buf,
const char *proto_n2a(unsigned short id, char *buf, int len,
const struct proto *proto_tb, size_t tb_len);
+FILE *generic_proc_open(const char *env, const char *name);
+
#endif /* __UTILS_H__ */
diff --git a/lib/utils.c b/lib/utils.c
index 6c1c1a8d..deb7654a 100644
--- a/lib/utils.c
+++ b/lib/utils.c
@@ -2003,3 +2003,17 @@ int proto_a2n(unsigned short *id, const char *buf,
return 0;
}
+
+FILE *generic_proc_open(const char *env, const char *name)
+{
+ const char *p = getenv(env);
+ char store[128];
+
+ if (!p) {
+ p = getenv("PROC_ROOT") ? : "/proc";
+ snprintf(store, sizeof(store) - 1, "%s/%s", p, name);
+ p = store;
+ }
+
+ return fopen(p, "r");
+}
diff --git a/misc/nstat.c b/misc/nstat.c
index 2c10feaa..63d177d1 100644
--- a/misc/nstat.c
+++ b/misc/nstat.c
@@ -43,35 +43,22 @@ int npatterns;
char info_source[128];
int source_mismatch;
-static int generic_proc_open(const char *env, char *name)
-{
- char store[128];
- char *p = getenv(env);
-
- if (!p) {
- p = getenv("PROC_ROOT") ? : "/proc";
- snprintf(store, sizeof(store)-1, "%s/%s", p, name);
- p = store;
- }
- return open(p, O_RDONLY);
-}
-
-static int net_netstat_open(void)
+static FILE *net_netstat_open(void)
{
return generic_proc_open("PROC_NET_NETSTAT", "net/netstat");
}
-static int net_snmp_open(void)
+static FILE *net_snmp_open(void)
{
return generic_proc_open("PROC_NET_SNMP", "net/snmp");
}
-static int net_snmp6_open(void)
+static FILE *net_snmp6_open(void)
{
return generic_proc_open("PROC_NET_SNMP6", "net/snmp6");
}
-static int net_sctp_snmp_open(void)
+static FILE *net_sctp_snmp_open(void)
{
return generic_proc_open("PROC_NET_SCTP_SNMP", "net/sctp/snmp");
}
@@ -277,7 +264,7 @@ static void load_ugly_table(FILE *fp)
static void load_sctp_snmp(void)
{
- FILE *fp = fdopen(net_sctp_snmp_open(), "r");
+ FILE *fp = net_sctp_snmp_open();
if (fp) {
load_good_table(fp);
@@ -287,7 +274,7 @@ static void load_sctp_snmp(void)
static void load_snmp(void)
{
- FILE *fp = fdopen(net_snmp_open(), "r");
+ FILE *fp = net_snmp_open();
if (fp) {
load_ugly_table(fp);
@@ -297,7 +284,7 @@ static void load_snmp(void)
static void load_snmp6(void)
{
- FILE *fp = fdopen(net_snmp6_open(), "r");
+ FILE *fp = net_snmp6_open();
if (fp) {
load_good_table(fp);
@@ -307,7 +294,7 @@ static void load_snmp6(void)
static void load_netstat(void)
{
- FILE *fp = fdopen(net_netstat_open(), "r");
+ FILE *fp = net_netstat_open();
if (fp) {
load_ugly_table(fp);
diff --git a/misc/ss.c b/misc/ss.c
index 5296cabe..11a0ed57 100644
--- a/misc/ss.c
+++ b/misc/ss.c
@@ -457,19 +457,6 @@ static void filter_merge_defaults(struct filter *f)
}
}
-static FILE *generic_proc_open(const char *env, const char *name)
-{
- const char *p = getenv(env);
- char store[128];
-
- if (!p) {
- p = getenv("PROC_ROOT") ? : "/proc";
- snprintf(store, sizeof(store)-1, "%s/%s", p, name);
- p = store;
- }
-
- return fopen(p, "r");
-}
#define net_tcp_open() generic_proc_open("PROC_NET_TCP", "net/tcp")
#define net_tcp6_open() generic_proc_open("PROC_NET_TCP6", "net/tcp6")
#define net_udp_open() generic_proc_open("PROC_NET_UDP", "net/udp")
--
2.30.2
Powered by blists - more mailing lists