netdev - RE: [PATCH] iputils-s20101006.tar.bz2: "-w deadline" starts after "-c" xmitted, and "-c" was exceeded for flood ping

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <7D3B6706FA74174B8C1AC24710890745136FEF3DE6@USNAVSXCHMBSA1.ndc.alcatel-lucent.com>
Date:	Tue, 21 Feb 2012 08:08:46 -0600
From:	"Cloutier, Joseph (Joseph)" <joseph.cloutier@...atel-lucent.com>
To:	"'YOSHIFUJI Hideaki'" <yoshfuji@...ux-ipv6.org>
CC:	"'netdev@...r.kernel.org'" <netdev@...r.kernel.org>,
	"'Linus Torvalds'" <torvalds@...ux-foundation.org>
Subject: RE: [PATCH] iputils-s20101006.tar.bz2: "-w deadline" starts after
 "-c" xmitted, and "-c" was exceeded for flood ping

I tried "ping -f -c 1000 -s 1472 -l 20 135.112.91.184 -W 5" and it seems "-W 5" is not working;  it might be disabled by "-f".

I will submit a change using a new feature "-ww".  I agree it is probably best not to try to redefine an existing option.

Yoshfuji- I am giving a talk to our major Customer Thursday and they are asking about this.  Can I say that we will be adding a new command line option to Linux ping that will give them the feature they need?

Thanks, Joe

-----Original Message-----
From: YOSHIFUJI Hideaki [mailto:yoshfuji@...ux-ipv6.org]
Sent: Tuesday, February 21, 2012 7:22 AM
To: Cloutier, Joseph (Joseph)
Cc: 'netdev@...r.kernel.org'; 'Linus Torvalds'; YOSHIFUJI Hideaki
Subject: Re: [PATCH] iputils-s20101006.tar.bz2: "-w deadline" starts
after "-c" xmitted, and "-c" was exceeded for flood ping


Joseph,

Cloutier, Joseph (Joseph) wrote:
> Hideaki, can you give me any status on this request?  Was it rejected, is it pending?  Thanks, Joe Cloutier

I asked you to tell me if -W is okay for your purpose.

If -W is not suitable, please introduce new option
(maybe -ww?).

Regards,

--yoshfuji

> -----Original Message-----
> From: Cloutier, Joseph (Joseph)
> Sent: Tuesday, January 10, 2012 1:01 PM
> To: 'YOSHIFUJI Hideaki'
> Cc: 'netdev@...r.kernel.org'; 'Linus Torvalds'
> Subject: [PATCH] iputils-s20101006.tar.bz2: "-w deadline" starts after
> "-c" xmitted, and "-c" was exceeded for flood ping
>
>
> From: Joseph Cloutier<jcloutier@...atel-lucent.com>
>
> 2 changes to ping&  ping6.  Only ping_common.c affected.
>    1- changed "-w deadline" to start deadline after "-c" pings have been transmitted, instead of from start of run.
>    2- fixed bug where>  "-c" pings are sent in flood ping mode when "-w deadline" option is also used.
> Reason:
>    Linux flood ping is an invaluable tool to finding duplex bugs, fiber problems, NE config problems-  especially
>    when used to trouble shoot wireless system backhaul problems.  ANY dropped pings are red flag indicators.
>    Current ping problem is that it terminates sometimes when outstanding pings are in transit, giving FALSE red
>    flags.  Change "-w deadline" to start after last "-c" transmitted ping allows cleanup of stragglers.  Second fix is
>    when "-f" option is running, "-c" is not rigorously checked when generating pings  A more rigorous check was
>    added.
>
> Signed-off-by: Joseph Cloutier<jcloutier@...atel-lucent.com>
>
> ---
> --- iputils-s20101006/ping_common.c.orig        2012-01-10 10:37:40.399719858 -0500
> +++ iputils-s20101006/ping_common.c     2012-01-10 11:33:37.084662882 -0500
> @@ -523,16 +523,6 @@ void setup(int icmp_sock)
>
>          gettimeofday(&start_time, NULL);
>
> -       if (deadline) {
> -               struct itimerval it;
> -
> -               it.it_interval.tv_sec = 0;
> -               it.it_interval.tv_usec = 0;
> -               it.it_value.tv_sec = deadline;
> -               it.it_value.tv_usec = 0;
> -               setitimer(ITIMER_REAL,&it, NULL);
> -       }
> -
>          if (isatty(STDOUT_FILENO)) {
>                  struct winsize w;
>
> @@ -553,6 +543,7 @@ void main_loop(int icmp_sock, __u8 *pack
>          int cc;
>          int next;
>          int polling;
> +       int sig_deadline = 0;
>
>          iov.iov_base = (char *)packet;
>
> @@ -567,9 +558,22 @@ void main_loop(int icmp_sock, __u8 *pack
>                  /* Check for and do special actions. */
>                  if (status_snapshot)
>                          status();
> +               if((npackets == ntransmitted)&&  deadline&&
> +                       (sig_deadline == 0)){
> +                       sig_deadline++;
> +                       struct itimerval it;
> +
> +                       it.it_interval.tv_sec = 0;
> +                       it.it_interval.tv_usec = 0;
> +                       it.it_value.tv_sec = deadline;
> +                       it.it_value.tv_usec = 0;
> +                       setitimer(ITIMER_REAL,&it, NULL);
> +               }
>
>                  /* Send probes scheduled to this time. */
>                  do {
> +                       if(npackets == ntransmitted)
> +                               break;
>                          next = pinger();
>                          next = schedule_exit(next);
>                  } while (next<= 0);

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html