[PATCH] ping.c : fix min/max packet size usage

[Denis Vlasenko]

On Friday 02 March 2007 01:30, Jason Schoon wrote:
> > Unfortunately, it introduces a regression:
> >
> > unpatched:
> >
> > # ./busybox ping -s 1000 127.0.0.1
> > PING 127.0.0.1 (127.0.0.1): 1000 data bytes
> > 1008 bytes from 127.0.0.1: icmp_seq=0 ttl=64 time=0.0 ms
> > 1008 bytes from 127.0.0.1: icmp_seq=1 ttl=64 time=0.0 ms
> > 1008 bytes from 127.0.0.1: icmp_seq=2 ttl=64 time=0.0 ms
> > 1008 bytes from 127.0.0.1: icmp_seq=3 ttl=64 time=0.0 ms
> >
> > patched:
> >
> > # ./busybox ping -s 1000 127.0.0.1
> > PING 127.0.0.1 (127.0.0.1): 1000 data bytes
> > ^C
> > --- 127.0.0.1 ping statistics ---
> > 4 packets transmitted, 0 packets received, 100% packet loss
> 
> 
> Agreed, I saw similar behavior using the original patch against 1.2.2.1.

Try revision 17999 (or current svn).
--
vda