[PATCH iproute2] json: do not escape single quotes

[PATCH iproute2] json: do not escape single quotes

[Andrea Claudi]

ECMA-404 standard does not include single quote character among the json
escape sequences. This means single quotes does not need to be escaped.

Indeed the single quote escape produces an invalid json output:

$ ip link add "john's" type dummy
$ ip link show "john's"
9: john's: <BROADCAST,NOARP> mtu 1500 qdisc noop state DOWN mode DEFAULT group default qlen 1000
    link/ether c6:8e:53:f6:a3:4b brd ff:ff:ff:ff:ff:ff
$ ip -j link | jq .
parse error: Invalid escape at line 1, column 765

This can be fixed removing the single quote escape in jsonw_puts.
With this patch in place:

$ ip -j link | jq .[].ifname
"lo"
"john's"

Fixes: fcc16c2287bf ("provide common json output formatter")
Signed-off-by: Andrea Claudi <aclaudi@redhat.com>
---
 lib/json_writer.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/lib/json_writer.c b/lib/json_writer.c
index 88c5eb88..2f3936c2 100644
--- a/lib/json_writer.c
+++ b/lib/json_writer.c
@@ -80,9 +80,6 @@ static void jsonw_puts(json_writer_t *self, const char *str)
 		case '"':
 			fputs("\\\"", self->out);
 			break;
-		case '\'':
-			fputs("\\\'", self->out);
-			break;
 		default:
 			putc(*str, self->out);
 		}
-- 
2.38.1

Re: [PATCH iproute2] json: do not escape single quotes

[patchwork-bot+netdevbpf]

Hello:

This patch was applied to iproute2/iproute2.git (main)
by Stephen Hemminger <stephen@networkplumber.org>:

On Thu,  3 Nov 2022 18:39:25 +0100 you wrote:
> ECMA-404 standard does not include single quote character among the json
> escape sequences. This means single quotes does not need to be escaped.
> 
> Indeed the single quote escape produces an invalid json output:
> 
> $ ip link add "john's" type dummy
> $ ip link show "john's"
> 9: john's: <BROADCAST,NOARP> mtu 1500 qdisc noop state DOWN mode DEFAULT group default qlen 1000
>     link/ether c6:8e:53:f6:a3:4b brd ff:ff:ff:ff:ff:ff
> $ ip -j link | jq .
> parse error: Invalid escape at line 1, column 765
> 
> [...]

Here is the summary with links:
  - [iproute2] json: do not escape single quotes
    https://git.kernel.org/pub/scm/network/iproute2/iproute2.git/commit/?id=49c63bc775d6

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html