Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

show ip route crash #7

Open
andrnils opened this issue Oct 2, 2014 · 4 comments
Open

show ip route crash #7

andrnils opened this issue Oct 2, 2014 · 4 comments
Assignees

Comments

@andrnils
Copy link
Contributor

andrnils commented Oct 2, 2014

Running show ip route crashes the clicon_cli frontend:

>sh ip route 
Oct  2 11:10:41: quaggapi_exec: write show ip route

Oct  2 11:10:45: UNIX error: read: Bad address
Segmentation fault (core dumped)

Same with show ip route bgp

>sh ip route bgp 
Oct  2 11:11:38: quaggapi_exec: write show ip route bgp

Oct  2 11:11:41: UNIX error: read: Bad address
Segmentation fault (core dumped)

This is with a full bgp:

>sh ip bgp summary 
Oct  2 11:15:26: quaggapi_exec: write show ip bgp summary

BGP router identifier 80.252.167.34, local AS number 12381
RIB entries 927745, using 64 MiB of memory
Peers 1, using 2528 bytes of memory

Neighbor        V    AS MsgRcvd MsgSent   TblVer  InQ OutQ Up/Down  State/PfxRcd
80.252.167.33   4 16150  429172    2661        0    0    0 22:10:34   504929

Total number of neighbors 1

Running show ip route 0.0.0.0/4 longer-prefixes works, whereas show ip route 0.0.0.0/3 longer-prefixes segfaults

@bholmgren bholmgren self-assigned this Oct 6, 2014
@bholmgren
Copy link
Contributor

This should be fixed now but required a change in the way Rost communicates with quagga so quagga needs to be updated as well.
There is quagga patch for 0.99.23 added to the repostitory.
Could you verify that the fix works on your system?

@Fredrik-Widell
Copy link

The fix works, a little, a show ip route or a show ip bgp outputs 65535 chars , which is far from a full bgp table, somewhere around a thusand lines. The segfaults are gone.

@bholmgren
Copy link
Contributor

Turned out cligen clipped individual output messages to 64k.
The output code in Rost has been rearranged and I have been able to view 100k routes at least.
The fix requires an update to clicon and there is an updated quagga patch as well.

@olofhagsand
Copy link
Member

Fixed the cligen 64k limitation as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants