ping: Print data bytes

data bytes is in original ping, therefore it should be in JSON as well.

TODO: IPv4 has more info, but that requires ping_func_set_st *fset
pointer - obviously gather_statistics() is not a good place to print the
"header" data. Maybe move it to ping_print_packet(), which is used only
for non-json code (which is sign of wrong design)?

TODO: what would be proper names these "56 (84) data bytes"?

Signed-off-by: Petr Vorel <pvorel@suse.cz>

Author: pevik

ping/ping_common.c

@@ -834,6 +834,17 @@ int gather_statistics(struct ping_rts *rts, uint8_t *icmph, int icmplen,
 		uint8_t *cp, *dp;
 
 		print_timestamp(rts);
+
+		if (rts->opt_json) {
+			// TODO: improve key name
+			construct_json(rts, PING_JSON_INT, "data_bytes", rts->datalen);
+			/*
+			 * FIXME: Add this require either add ping_func_set_st *fset or move into ping_print_packet().
+			if (fset->ping_data_bytes)
+				construct_json(rts, PING_JSON_INT, "data_bytes_full", fset->ping_data_bytes(rts));
+			*/
+		}
+
 		ping_print_int(rts, _("%d bytes "), "bytes", cc);
 		// TODO: split host IP address and name into separate attributes in JSON?
 		ping_print_str(rts, _("from %s:"), "host", from);