iproute: Add route showdump command
Subsystems (2)
PATCH 1/1 iproute: Add route showdump command
Some time ago the save+restore commands were added to ip route (git id f4ff11e3, Add ip route save/restore). These two save the raw rtnl stream into a file and restore one (reading it from stdin).
The problem is that there's no way to get the contents of the dump file in a human readable form. How about adding a command that reads the rtnl stream from stdin and prints the data in a way the usual "ip route list" does?
---
ip/iproute.c +14 -0
@@ -60,6 +60,7 @@ static void usage(void)fprintf (stderr,"Usage: ip route { list | flush } SELECTOR\n" );fprintf (stderr," ip route save SELECTOR\n" );fprintf (stderr," ip route restore\n" ); +fprintf (stderr," ip route showdump\n" );fprintf (stderr," ip route get ADDRESS [ from ADDRESS iif STRING ]\n" );fprintf (stderr," [ oif STRING ] [ tos TOS ]\n" );fprintf (stderr," [ mark NUMBER ]\n" );
@@ -1526,6 +1527,17 @@ int iproute_restore(void)exit (rtnl_from_file (stdin, &restore_handler,NULL )); } +static int show_handler (const struct sockaddr_nl *nl,struct nlmsghdr *n,void *arg) +{ +print_route (nl, n, stdout); +return 0 ; +} + +static int iproute_showdump (void ) +{ +exit (rtnl_from_file (stdin, &show_handler,NULL )); +} +void iproute_reset_filter () {memset (&filter,0 ,sizeof (filter));
@@ -1570,6 +1582,8 @@ int do_iproute(int argc, char **argv)return iproute_list_flush_or_save (argc-1 , argv+1 ,IPROUTE_SAVE );if (matches (*argv,"restore" ) ==0 )return iproute_restore (); +if (matches (*argv,"showdump" ) ==0 ) +return iproute_showdump ();if (matches (*argv,"help" ) ==0 )usage ();fprintf (stderr,"Command \"%s\" is unknown, try \"ip route help\".\n" , *argv);
Quoted 10 lines · On Thu, 26 Jul 2012 12:53:39 +0400 Pavel Emelyanov <[email protected]> wrote:
Some time ago the save+restore commands were added to ip route (git id
f4ff11e3, Add ip route save/restore). These two save the raw rtnl stream into a file and restore one (reading it from stdin).The problem is that there's no way to get the contents of the dump file in a human readable form. How about adding a command that reads the rtnl stream from stdin and prints the data in a way the usual "ip route list" does?
Signed-off-by Pavel Emelyanov <[email protected]>
Being able to decode a dump is great idea.
Are the user's smart enough not to try it out at the command prompt and get totally lost? Maybe another isatty() check is needed.
Another solution would be to put a small header on the save file with a magic number that could be checked. This would mean changing save/restore/showdump and ideally updating the magic file in distributions.
Quoted 14 lines · Stephen Hemminger
Quoted 10 lines · On Thu, 26 Jul 2012 12:53:39 +0400 Pavel Emelyanov <[email protected]> wrote:
Some time ago the save+restore commands were added to ip route (git id
f4ff11e3, Add ip route save/restore). These two save the raw rtnl stream into a file and restore one (reading it from stdin).The problem is that there's no way to get the contents of the dump file in a human readable form. How about adding a command that reads the rtnl stream from stdin and prints the data in a way the usual "ip route list" does?
Signed-off-by Pavel Emelyanov <[email protected]>
Quoted 8 lines
Being able to decode a dump is great idea.
Are the user's smart enough not to try it out at the command prompt and get totally lost? Maybe anotherisatty()check is needed.Another solution would be to put a small header on the save file with a magic number that could be checked. This would mean changing save/restore/showdump and ideally updating the magic file in distributions.
.
Hm... The idea with magic looks reasonable.
I will look at what can be done here, thanks!