Bug 10230 - ppp-watch broken
ppp-watch broken
Product: Red Hat Linux
Classification: Retired
Component: initscripts (Show other bugs)
i386 Linux
medium Severity medium
: ---
: ---
Assigned To: Nalin Dahyabhai
Depends On:
  Show dependency treegraph
Reported: 2000-03-18 19:01 EST by Mikhail Zabaluev
Modified: 2008-05-01 11:37 EDT (History)
1 user (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2000-03-20 11:04:12 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)
Here's that PPP patch (2.10 KB, patch)
2000-03-18 19:03 EST, Mikhail Zabaluev
no flags Details | Diff

  None (edit)
Description Mikhail Zabaluev 2000-03-18 19:01:01 EST
The ppp-watch code in initscripts 4.72 has serious flaws that disable
normal usage of /sbin/ifup and /sbin/ifdown for PPP connections.
I'm using Mandrake and their packages, but it seems like the
problem is on your behalf.
First, it erroneously expects a return status from socket() to be non-zero
for success (actually it have to be non-negative)
when it checks whether an interface is up; this error stops ifup from a
successful return even when a connection is successfully established.
Second, closing the standard descriptors and executing a shell script
afterwards is a bad idea, since that shell runs into problems, for example,
when doing a pipe. You may consider to use /dev/null instead.

I have made a patch that solves both issues, and additionally removes
'--remotename' option from parameters that wvdial is issued with
by ifup-ppp script - the version of wvdial on my system doesn't recognize
it, and it can be placed in /etc/wvdial.conf instead.
Email me if you wanna take a look at the patch.
Comment 1 Mikhail Zabaluev 2000-03-18 19:03:59 EST
Created attachment 157 [details]
Here's that PPP patch
Comment 2 Bill Nottingham 2000-03-20 10:45:59 EST
The return code of socket() has been fixed as of
the current rawhide release.
Comment 3 Bill Nottingham 2000-03-20 11:04:59 EST
ppp-watch is changed to open /dev/null as of initscripts-5.02.

Note You need to log in before you can comment on or make changes to this bug.