netdev - Re: [PATCH iproute2] tc: add print options to fix json output

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <278df9b9-e2f6-fe8a-e7d6-432b29a39697@gmail.com>
Date:   Thu, 29 Oct 2020 20:47:48 -0600
From:   David Ahern <dsahern@...il.com>
To:     "Sharma, Puneet" <pusharma@...mai.com>,
        Stephen Hemminger <stephen@...workplumber.org>
Cc:     "dsahern@...nel.org" <dsahern@...nel.org>,
        "netdev@...r.kernel.org" <netdev@...r.kernel.org>
Subject: Re: [PATCH iproute2] tc: add print options to fix json output

On 10/29/20 6:42 PM, Sharma, Puneet wrote:
> Because basic match is made of multiple keywords and parsed and handle differently
> example:
> 	$ tc filter add dev $eth_dev_name ingress priority 20000 protocol ipv4 basic match '(cmp(u8 at 9 layer network eq 6) or cmp(u8 at 9 layer network eq 17)) and ipset(sg-test-ipv4 src)' action pass
> 
> and if jsonw_string used then it will double-quote every string passed and
> 
> prints something like this:
> 	 "ematch": "("cmp(u8 at 9 layer 1 eq 6,")","OR "cmp(u8 at 9 layer 1 eq 17,")",") ","AND "ipset(sg-test-ipv4 src,")",
> 


I think you need to print that string to a temp buffer to collect the
complete expression and then just call print_string on it.