[prev in list] [next in list] [prev in thread] [next in thread] 

List:       freebsd-bugs
Subject:    bin/11943: pppd connection time report is wrong
From:       plm () xs4all ! nl
Date:       1999-05-30 14:12:39
[Download RAW message or body]


> Number:         11943
> Category:       bin
> Synopsis:       pppd connection time report is wrong
> Confidential:   no
> Severity:       non-critical
> Priority:       low
> Responsible:    freebsd-bugs
> State:          open
> Quarter:        
> Keywords:       
> Date-Required:
> Class:          sw-bug
> Submitter-Id:   current-users
> Arrival-Date:   Sun May 30 07:20:01 PDT 1999
> Closed-Date:
> Last-Modified:
> Originator:     Peter Mutsaers
> Release:        -current of may 12th 1999
> Organization:
n.a.
> Environment:
FreeBSD muon 4.0-CURRENT FreeBSD 4.0-CURRENT #40: Tue May 11 22:12:00 MET DST 1999    \
plm@muon:/var/arch/fbsd/src/sys/compile/PLM  i386
> Description:
Via syslog/debug.notice pppd reports how long it has been connected 
when the link is shutdown (idle).

When the link is shutdown it reports e.g. 65 minutes connected time when
in fact it was only 4 minutes. This occurs when using dial-on-demand.
> How-To-Repeat:
Start pppd client connection via dial on demand, then shut it down or wait until the \
idle timer expires
> Fix:
In main.c: stime (start time) is set when pppd starts. It should be
set when the connecion fires up instead:

*** main.c-     Sun May 30 16:05:46 1999
--- main.c      Sun May 30 16:07:23 1999
***************
*** 548,553 ****
--- 548,554 ----
         * incoming events (reply, timeout, etc.).
         */
        syslog(LOG_NOTICE, "Connect: %s <--> %s", ifname, devnam);
+       stime = time((time_t *) NULL);
        lcp_lowerup(0);
        lcp_open(0);            /* Start protocol */
        for (phase = PHASE_ESTABLISH; phase != PHASE_DEAD; ) {


> Release-Note:
> Audit-Trail:
> Unformatted:


To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-bugs" in the body of the message


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic