Adding netstat 1.70 ROSE protocol support

Hallo Bernard,

this patch "1.70" is not from me and not authorized, I did not even know about it. Where do you find it? I currently only maintain patches against HEAD of CVS and yes, your Rose patch was not yet included. I wanted to check the general handling of unexpected kernel values for state engines first, and that somewhat delayed the patch.

Thanks for consolidating of the patches, I guess I will apply them before the "review".

Greetings
Bernd

Quoted 60 lines · Bernard Pidoux

Hi Bernd,

Although I submitted an update for netstat supporting AMPR rose protocole,
netstat 1.70 has been commited in Berlios.de CVS repository without including it.

The following patch has been updated against the present net-tools CVS and includes a safer code for reading and displaying rose state.

I hope it will be included in netstat CVS as this is a very usefull tool to display AMPR proc files such as AX.25, NetRom and Rose.

http://f6bvp.free.fr/logiciels/netstat/netstat-CVS-1.70.patch

Bernard Pidoux
f6bvp
Funding President AMSAT France

Note for hamradio operators :
net-tools can be downloaded from CVS by issuing the following requests :

*cvs -d:pserver:[email protected]:/cvsroot/net-tools login*

Type<return> when asked for a password.
*
cvs -z3 -d:pserver:[email protected]:/cvsroot/net-tools co net-tools*

This will create net-tools sub directory with all net-tools modules including netstat.

Apply the above patch after downloading the file :

patch -p0< netstat-CVS-1.70.patch

A typical command could be :

netstat --rose
or
netstat --route --rose

Same for ax25 or netrom

Quoted 9 lines · Bernd Eckenfels

Quoted 2 lines · On Mon, Aug 16, 2010 at 04:01:49PM +0200, Bernard F6BVP wrote:

The present patch corrects this bug by initializing rose_state[] correctly according to linux/include/net/rose.h

Thanks, can you please also add a "unknown state" option in case the kernel
adds new states? Will integrate that to upstream CVS later this week.

Greetings
Bernd