DORMANTno replies

[PATCH iproute2-next master] bridge: fdb get: add missing json init (new_json_obj)

From: Julien Fortin <hidden>
Date: 2020-07-10 00:53:08
Subsystem: the rest · Maintainer: Linus Torvalds

From: Julien Fortin <redacted>

'bridge fdb get' has json support but the json object is never initialized

before patch:

$ bridge -j fdb get 56:23:28:4f:4f:e5 dev vx0
56:23:28:4f:4f:e5 dev vx0 master br0 permanent
$

after patch:

$ bridge -j fdb get 56:23:28:4f:4f:e5 dev vx0 | \
python -c \
'import sys,json;print(json.dumps(json.loads(sys.stdin.read()),indent=4))'
[
    {
        "master": "br0",
        "mac": "56:23:28:4f:4f:e5",
        "flags": [],
        "ifname": "vx0",
        "state": "permanent"
    }
]
$

Signed-off-by: Julien Fortin <redacted>
---
 bridge/fdb.c | 6 ++++++
 1 file changed, 6 insertions(+)
diff --git a/bridge/fdb.c b/bridge/fdb.c
index 198c51d1..aaee4d7c 100644
--- a/bridge/fdb.c
+++ b/bridge/fdb.c
@@ -638,10 +638,16 @@ static int fdb_get(int argc, char **argv)
 	if (rtnl_talk(&rth, &req.n, &answer) < 0)
 		return -2;
 
+	/*
+	 * Initialize a json_writer and open an array object
+	 * if -json was specified.
+	 */
+	new_json_obj(json);
 	if (print_fdb(answer, stdout) < 0) {
 		fprintf(stderr, "An error :-)\n");
 		return -1;
 	}
+	delete_json_obj();
 
 	return 0;
 }
-- 
2.27.0
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help