Commit 53d6eb08 authored by Andrey Ignatov's avatar Andrey Ignatov Committed by Daniel Borkmann

bpftool: Fix bpftool net output

Print `bpftool net` output to stdout instead of stderr. Only errors
should be printed to stderr. Regular output should go to stdout and this
is what all other subcommands of bpftool do, including --json and
--pretty formats of `bpftool net` itself.

Fixes: commit f6f3bac0 ("tools/bpf: bpftool: add net support")
Signed-off-by: default avatarAndrey Ignatov <rdna@fb.com>
Acked-by: default avatarYonghong Song <yhs@fb.com>
Acked-by: default avatarSong Liu <songliubraving@fb.com>
Signed-off-by: default avatarDaniel Borkmann <daniel@iogearbox.net>
parent 1042caa7
...@@ -16,7 +16,7 @@ ...@@ -16,7 +16,7 @@
jsonw_name(json_wtr, name); \ jsonw_name(json_wtr, name); \
jsonw_start_object(json_wtr); \ jsonw_start_object(json_wtr); \
} else { \ } else { \
fprintf(stderr, "%s {", name); \ fprintf(stdout, "%s {", name); \
} \ } \
} }
...@@ -25,7 +25,7 @@ ...@@ -25,7 +25,7 @@
if (json_output) \ if (json_output) \
jsonw_start_object(json_wtr); \ jsonw_start_object(json_wtr); \
else \ else \
fprintf(stderr, "{"); \ fprintf(stdout, "{"); \
} }
#define NET_END_OBJECT_NESTED \ #define NET_END_OBJECT_NESTED \
...@@ -33,7 +33,7 @@ ...@@ -33,7 +33,7 @@
if (json_output) \ if (json_output) \
jsonw_end_object(json_wtr); \ jsonw_end_object(json_wtr); \
else \ else \
fprintf(stderr, "}"); \ fprintf(stdout, "}"); \
} }
#define NET_END_OBJECT \ #define NET_END_OBJECT \
...@@ -47,7 +47,7 @@ ...@@ -47,7 +47,7 @@
if (json_output) \ if (json_output) \
jsonw_end_object(json_wtr); \ jsonw_end_object(json_wtr); \
else \ else \
fprintf(stderr, "\n"); \ fprintf(stdout, "\n"); \
} }
#define NET_START_ARRAY(name, fmt_str) \ #define NET_START_ARRAY(name, fmt_str) \
...@@ -56,7 +56,7 @@ ...@@ -56,7 +56,7 @@
jsonw_name(json_wtr, name); \ jsonw_name(json_wtr, name); \
jsonw_start_array(json_wtr); \ jsonw_start_array(json_wtr); \
} else { \ } else { \
fprintf(stderr, fmt_str, name); \ fprintf(stdout, fmt_str, name); \
} \ } \
} }
...@@ -65,7 +65,7 @@ ...@@ -65,7 +65,7 @@
if (json_output) \ if (json_output) \
jsonw_end_array(json_wtr); \ jsonw_end_array(json_wtr); \
else \ else \
fprintf(stderr, "%s", endstr); \ fprintf(stdout, "%s", endstr); \
} }
#define NET_DUMP_UINT(name, fmt_str, val) \ #define NET_DUMP_UINT(name, fmt_str, val) \
...@@ -73,7 +73,7 @@ ...@@ -73,7 +73,7 @@
if (json_output) \ if (json_output) \
jsonw_uint_field(json_wtr, name, val); \ jsonw_uint_field(json_wtr, name, val); \
else \ else \
fprintf(stderr, fmt_str, val); \ fprintf(stdout, fmt_str, val); \
} }
#define NET_DUMP_STR(name, fmt_str, str) \ #define NET_DUMP_STR(name, fmt_str, str) \
...@@ -81,7 +81,7 @@ ...@@ -81,7 +81,7 @@
if (json_output) \ if (json_output) \
jsonw_string_field(json_wtr, name, str);\ jsonw_string_field(json_wtr, name, str);\
else \ else \
fprintf(stderr, fmt_str, str); \ fprintf(stdout, fmt_str, str); \
} }
#define NET_DUMP_STR_ONLY(str) \ #define NET_DUMP_STR_ONLY(str) \
...@@ -89,7 +89,7 @@ ...@@ -89,7 +89,7 @@
if (json_output) \ if (json_output) \
jsonw_string(json_wtr, str); \ jsonw_string(json_wtr, str); \
else \ else \
fprintf(stderr, "%s ", str); \ fprintf(stdout, "%s ", str); \
} }
#endif #endif
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment