On Tue, Jul 26, 2022 at 08:25:05PM +0200, Morten Brørup wrote: > > From: Bruce Richardson [mailto:bruce.richard...@intel.com] > > Sent: Monday, 25 July 2022 18.36 > > To: dev@dpdk.org > > Cc: Bruce Richardson; Ciara Power; Keith Wiles > > Subject: [PATCH v2 02/13] telemetry: fix escaping of invalid json > > characters > > > > For string values returned from telemetry, escape any values that > > cannot > > normally appear in a json string. According to the json spec[1], the > > characters than need to be handled are control chars (char value < > > 0x20) > > and '"' and '\' characters. > > > > To handle this, we replace the snprintf call with a separate string > > copying and encapsulation routine which checks each character as it > > copies it to the final array. > > > > [1] https://www.rfc-editor.org/rfc/rfc8259.txt > > > > Fixes: 6dd571fd07c3 ("telemetry: introduce new functionality") > > Bugzilla ID: 1037 > > > > Signed-off-by: Bruce Richardson <bruce.richard...@intel.com> > > --- > > Patchwork didn't pick up my reply to the 00/13 of the series, so I'll try > again here... > > Series-Acked-by: Morten Brørup <m...@smartsharesystems.com> > Unfortunately, patchwork doesn't work with series acks, only individual ones. However, the maintainers recognise when they are present, and if I do a V3, I'll split your ack across the set so patchwork does recognise it.
/Bruce
