lnstat

lnstat

[David Kaufmann]

[-- Attachment #1: Type: text/plain, Size: 500 bytes --]

Hi!

`lnstat` segfaults (tested on Debian 9, CentOS 6+7, Fedora 27) if it is
started as `lnstat -w 1`

according to gdb the crash is in `build_hdr_string` at lnstat.c:212

as it seems to be an useless value for the option anyway it might make
sense to just handle a single "1" the same as if "0" was specified.
`-w 0,1`, `-w 1,0`, `-w 1,1` and other variations do work.

All the best,
Astra

PS: I did not find any other place to report this, if this is the wrong
place please tell we where to post.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: lnstat

[Stephen Hemminger]

[-- Attachment #1: Type: text/plain, Size: 780 bytes --]

On Sat, 3 Mar 2018 22:56:02 +0100
David Kaufmann <astra@ionic.at> wrote:

> Hi!
> 
> `lnstat` segfaults (tested on Debian 9, CentOS 6+7, Fedora 27) if it is
> started as `lnstat -w 1`
> 
> according to gdb the crash is in `build_hdr_string` at lnstat.c:212
> 
> as it seems to be an useless value for the option anyway it might make
> sense to just handle a single "1" the same as if "0" was specified.
> `-w 0,1`, `-w 1,0`, `-w 1,1` and other variations do work.

Right having one character width is breaking the header building code.
Probably should just catch it in the option parsing.

> 
> All the best,
> Astra
> 
> PS: I did not find any other place to report this, if this is the wrong
> place please tell we where to post.

This the right place.

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]