[Xorp-hackers] Bug report.

[Pavlin Radoslavov]

I agree the "show" output doesn't look right.
[BTW, after the change is committed the "show" output looks normal.]
Though, from your fix it looks like the "for ..." loop is not
needed at all, and it should be replaced with:

    result = "\"" + value + "\"";
    return result;

Please submit a Bugzilla entry so the issue can be tracked and
the fix can be tested properly.

Thanks,
Pavlin

杨小帅 <eshe168 at gmail.com> wrote:

>      when you entry a "xxx xxxx" value into txt node. The xorpsh will
> pass it. But when you use show -all in CLI, the information in your
> screen is wrong. Apparently, text should not be "\"abc cba\"".
> 
> exp:
> yxs at Lenny621# set vlans interface 312 text "abc cba"
> yxs at Lenny621# show
>     vlans {
>         interface 312 {
> >           text: "\"abc cba\""
>         }
>     }
> 
> BUG FIXED:
> string
> ConfigTreeNode::quoted_value(const string& value) const
> {
>     string result;
>     ...
> 	if (x == '"')
> >	    escaped += "\\\""; //should be escaped += "";
> 	else
> 	    escaped += x;
>     ...
>     return result;
> }
> 
> 
> B.R.
> Xiaoshuai Yang
> 
> _______________________________________________
> Xorp-hackers mailing list
> Xorp-hackers at icir.org
> http://mailman.ICSI.Berkeley.EDU/mailman/listinfo/xorp-hackers