Update to 2.4.4b1

Clean up base64_encode a bit more.

Assume that we have libcrypt if we have /usr/include/crypt.h.
This should fix linking on 64-bit architectures.

Stop the charshunt process after running the disconnector,
rather than immediately a signal is received.  With this change
we can send a SIGHUP to pppd and actually have it do the
LCP TermReq/TermAck exchange.

Bring up the link on an explicit call from main() rather than
doing it in link_required().  With the old way, it was restarting
the link in the middle of link_terminated().

Add a bit more space to the output buffer for base64 encoding,
to prevent 4-character usernames getting truncated.

Get the plugins from the right directory when DESTDIR isn't /usr.

Return the message from the radius server to the peer.

Fix the rechallenge behaviour.  Previously, once it sent a rechallenge,
it would ignore the response (except to send a reply with a bogus
message) and retransmit the rechallenge every 3 seconds until it
eventually timed out and took down the link.

Don't set the interface MTU in PPPOESendConfig, in fact get rid of
it altogether, and instead set the wanted MRU and allowed MTU to
1492 before starting negotiation.

Clear the wtmp entry in plogout whether or not USE_PAM is defined,
for symmetry with plogin, which creates it whether or not USE_PAM
is defined.

Fix the return value from setpassfilter and setactivefilter - it
should be 1 if the value was OK or 0 if it wasn't.

Fixed kernel memory leaks reported by Jin Jiang along with some
readability and commenting problems that led to the leaks.

Added support for x64 (AMD Opteron/Athlon and Intel EM64T) on Solaris
using Sun WorkShop or gcc.

Fixed configure breakage in $archvariant support for Solaris
introduced by fix in RCS ID 1.33 -- failed to configure for WorkShop C
compiler correctly because test was changed from -f (file exists) to
"$archvariant" (variable is non-null).

Fixed ccp.c compilation warnings due to missing argument type in RCS
ID 1.48 fix.

Fix for Sun CR 6257917: the right prefix length for an interface token
is 64 bits, not 10.  (10 came from the prefix length of a link-layer
address, but it's not actually used by SIOCSLIFADDR, so it wasn't right
in any case.)

Make the description of the call option more verbose, and
allegedly clearer.  From Marco D'Itri.

From Marco D'Itri: fix for an earlier patch from him.

From Marco D'Itri.

Apparently some French ISPs really send PADO packets from
multicast MAC addresses. :-(

From Marco D'Itri.

This is a fix for #294232.

If pppd recognized the peer not to ask for encryption in
his initial offer, it refused any further negotiation.

This change tells the peer using a ConfNak what
encryption options we're able to accept.
This makes the peer send a new ConfReq, usually with
one of the options we're able to accept.

Do an lcp_close whenever the link terminates, not just if it
terminates because of an error.  This is needed for persist
to work properly.

Call lcp_close(0) in link_required if the channel connect or
establish_ppp functions fail.  It's a bit grotty but it is needed
to get lcp back into closed state so that a future lcp_open will
do what it should.

Don't close pty_slave and real_ttyfd in connect_tty if an error
occurs; link_required calls cleanup_tty if connect_tty returns
an error, and that does the closing.  Doing the closes in connect_tty
meant that the disconnector couldn't run and the tty mode couldn't
be restored.

DLT_PPP_WITH_DIRECTION should be DLT_PPP_WITHDIRECTION.
Thanks to Stefan Petersen for pointing this out.

Get rid of the MS_ChapResponse and MS_Chap2Response structures.
Using a struct to represent an on-the-wire format is basically
broken, since the compiler can add padding between members or
assume alignment for the struct.  Instead we just use arrays
of unsigned char and define offsets in the arrays for the various
fields.

Cut down on spam from Solaris ppp kernel bits: unknown DLPI primitives
(as encountered on Solaris 10) aren't errors, and need to flush out any
queued up (undecoded) data on the read side after pushing ppp_ahdlc.

Added log messages for LCP Identification, Time-Remaining, and other
more obscure protocols.

Use the ipparam value as the PW_CALLING_STATION_ID value
if no value has been set for remote_number.

Add radius to the list of plugins to be compiled by default

make sure plugins get installed in the right place

Make sure we install plugins in the right place

Simplified and cleaned up the radiusclient implementation, and
made it use pppd facilities where relevant.  All the bits of
radiusclient that we actually need are now in the plugins/radius
directory.

Add these files, used with TDB.

Apparently the IBM patent on LZW is still pending.

Correctly escape or unescape hypens in the man pages.
Without this patch "-" is rendered as the endash in unicode
consoles and then bad things happen.  From Marco d'Itri.

Add pppoe-discovery program, from Marco D'Itri

Update

Exit with an error if both demand and notty are given.
Don't require a connect script with demand if we are using
the pty or socket options.
Enforce the connect_delay if we are using the socket option.

Make SIGHUP just take down the link now, not the bundle.
Be a bit more careful about quitting when we get a SIGTERM
or SIGINT.

Update for the recent multilink fixes

Terminate IPCP if the peer refuses to agree to our IP address.

New version of TDB code, borrowed from samba.

Mention multilink and TDB updates

Add a 'treat as reject' parameter to the nakci functions.
Use it to make sure we don't keep requesting the same IP
address over and over when it keeps getting nacked.

Don't allow the client to ask for callback, since we don't
support it.

Create link pid file (from the linkname option value) straight
away when starting pppd.  Suggested by Shun-ichi TAHARA.

Multilink improvements.  This involved moving some logic from the
main loop in main.c into link_required() and link_terminated() in
auth.c and adding code to multilink.c.  We now make a tdb entry
with the list of pppd pids for all the links in the bundle, and the
master pppd uses this to send a SIGHUP to each one when the bundle
is terminated.

We still have one pppd controlling both the bundle and the first link,
but when that link goes down, assuming that other links still exist,
the first link's pppd will clean up after that link but then stay
running until all the links have disconnected.  So it is possible to
lose the first link without losing the bundle.

This requires a small kernel patch which I will be sending to the
kernel maintainers shortly.

squish a compile warning

When using the notty option, close off stdin and stdout once the
charshunt has been started.  Also close off stderr unless it is the
log_to_fd.

Cope with /proc/net/ipx/interface as well as /proc/net/ipx_interface.
Added a couple of unrelated comments as well.

Add __attribute__((__packed__)) to the MS-CHAP response structures
since they may be unaligned and may not be padded.

Add an underscore to the MD5 routine names so they can more
easily be replaced by the openssl versions.

Move sys_init call after we have occupied fds 0, 1, 2.

Arrange for holdoff_specified to be set if the holdoff option
is used; make child-timeout option use OPT_PRIO.

Correct my email address.

Fix an infinite negotiation loop where we would respond to a
configure-reject for the IPCP Addresses option with another
IPCP configure-request containing the Addresses option.

New pppoatm plugin, sent in by David Woodhouse.

Mention winbind plugin

Run ntlm_auth as the user that invoked pppd.
Make the ntlm_auth-helper option privileged.
Use safe_fork to manage fds over the fork.

Extend safe_fork to include the shuffling of fds to get the
ones we want on fds 0, 1, 2.

Don't ask for CHAP if we don't have any digests to use.

Add winbind plugin from Andrew Bartlet.
Changes to chap_ms.[ch] needed by winbind.

Update for 2.4.3b1 release

Update patch level to 2.4.3b1

Fix an fd leak on the discovery socket.
Remove some unused code.

Take out some ancient debugging code.

Remove the requirement that redistributions in binary form reproduce
the copyright conditions in documentation from my copyright notice.
It still remains in CMU's and others' copyright notices.

Fix the error message when a device name and the socket option
are both specified.  Make it clearer how ttyfd gets set.

document the child-timeout option

Don't log messages in signal handlers; defer it to mainline
Logging in signal handlers can cause deadlocks.

Add a timeout when waiting for child processes to exit before pppd
exits.  Send a SIGTERM to the child processes if the timeout expires
or if pppd gets a SIGTERM or SIGINT itself.

remove duplicate $(CDEFS)

Use Makedefs.com rather than solaris/Makedefs, since Makedefs.com
has @DESTDIR@ etc. substituted.
Move installation of kernel modules to a install-modules target
so that it can be compiled and installed in an alternate directory
as non-root for the build farm.

Fix construction of $ksrc/Makefile for Solaris

Only compile in filtering if /usr/include/pcap-bpf.h exists.

Add installcheck targets to keep the autobuilder happy
(see http://build.samba.org/)

Remove dependencies on CHAPMS definition in header files.
Fix bug in filling in mdtype field when we NAK and suggest CHAP.
Ask for/suggest MD5 before MSCHAP{v2,} digest.

Take out the -o root argument to install.  It is the default anyway
when installing as root, and it causes errors in the build farm testing.

Cope better when prototype Makefiles don't exist.
Restore $archvariant stuff for solaris.

Put man pages in /usr/share/man under Linux.

Support --prefix and --sysconfdir on Solaris too

Add --prefix and --sysconfdir options to configure, and put
@DESTDIR@ and @SYSCONF@ tags in various Makefile.linux files.
These tags get expanded by configure.

Patch from Robert Vogelgesang:
This patch enables plugins called via hooks/notifiers triggered
by this call to lcp_close() to see status set here.
Otherwise (i. e. without this patch) the RADIUS plugin has
no chance to set the attribute PW_ACCT_TERMINATE_CAUSE to the
value PW_ACCT_SESSION_TIMEOUT.

Patch from Robert Vogelgesang:
This patch fixes the link statistics for connections that
go through multiple IPCP up-down cycles.  Such connections
happen typically in a setup where pppd is used as a back-end
by a L2TP daemon, in case the PPP session at the other side
of the L2TP tunnel reconnects, but the L2TP daemon at that
side just reuses the old L2TP tunnel instead of creating a
new one.

The patch is most important when RADIUS accounting is in use:
Each IPCP-down initiates a RADIUS-Accounting-Stop packet, which
indicates the end of a session.  Without this patch, the
accounting information in each subsequent RADIUS-Accounting-Stop
packet of the very same connection would contain cumulative
data since the connection start, but not the data of the last
"sub-session"; in other words, the accounting data sent to
the RADIUS server would indicate that the client had used much
more session time and transfered much more data.
NOTE: The problem fixed by this patch exists even when the
radius plugin is not in use; when extracting accounting data
from the syslog, you can work around the bug, because you
can see there that the same instance of pppd had multiple
sessions; you cannot see this in the RADIUS accounting data.

Furthermore, this patch suppresses duplicate printing/syslogging
of identical data.

Patch from Robert Vogelgesang:
This patch does two things:
o It adds some debugging messages.
o "cleanup()" will no longer be added to the link_down_notifier
chain.
The debugging messages are obvious.
The problem with cleanup() in the link_down_notifier chain is only
half-way that there could be cases where the link would go up again
and without a further authentication -- I just don't know if this
can happen.  But this part of the patch is a work-around for a
_real_ problem/bug in the radius plugin (not the radattr plugin):

The radius plugin calls functions registered via the
radius_attributes_hook after each PAP authentication (which is
correct), but only after the _first_ successful CHAP authentication
during a session.  Subsequent CHAP authentications are performed,
but the radius_attributes_hook will not be processed again.

This can happen in a setup where pppd is used as a back-end
        by a L2TP daemon, in case the PPP session at the other side
        of the L2TP tunnel reconnects, but the L2TP daemon at that
        side just reuses the old L2TP tunnel instead of creating a
        new one.  In such situations, an incomming follow-up session
via an existing T2TP tunnel would re-use the same instance of
pppd; the incomming CHAP authentication would first tear down
the old session, which in turn would call the link_down_notifier.
When the _subsequent_ CHAP authentication succeeds, there is
currently no call to the function assigned to
radius_attributes_hook (here: print_attributes(); THIS BUG
REMAINS AND NEEDS TO BE FIXED).
To summarize: The radius plugin calls the function registered
via the radius_attributes_hook after _each_ successful PAP
authentication, but only after the _first_ successful CHAP
authentication; radius_attributes_hook _should_ be processed
after _each_ successful CHAP authentication.

I have currently no patch for this bug; furthermore, I should
first contact the author of the radius plugin and ask him,
_why_ he has programmed a special handling of subsequent
CHAP authentications.

With the following patch, the follow-up session can re-use the
radattr-file left over from the previous session, which is OK
in our application, but may cause problems in others.
Note: This is only a problem when CHAP is used; subsequent
sessions authenticated with PAP are OK, with and without this
patch.

Remove compile warning resulting from chap_verify_hook prototype change.

Patch from Robert Vogelgesang:
        This patch avoids duplicate session IDs in RADIUS accounting,
        when the same pppd instance has multiple sessions during
        the same second.  This can happen when you have a really
        fast RADIUS server and fast clients, e. g. when using pppd
        as a back-end for PPPoE (either directly or via L2TP).

Get rid of an unnecessary chmod (it wasn't being used on Linux anyway).

Fix some places where we weren't checking the received
packets carefully enough.

Make the filtering stuff work with recent versions of libpcap.

Don't prepend /dev/ to a possible device name if it already begins
with '/' (i.e. just check for / instead of /dev/ as before).
This allows /udev/blah to be used as a tty device name.
Requested by Pawel Sakowski.

Don't use unsigned long in the SHA1 code; we want 32-bit variables
and unsigned long is 64 bits on 64-bit platforms.  Use unsigned int
or u_int32_t instead.  Pointed out by Oleg Makarenko.

Fix use-after-free bug where we were freeing the per-user options
set in the secrets file before they were used.  Patch from
Michael Tokarev.

Tolerate EINTR on tcsetattr in set_up_tty - just retry.

Close the device fd in device_script() if the channel plugin doesn't
have a close function.  Change suggested by Alan Hourihane.

Allow pppoe to be used on bridging interfaces (br*).
Get rid of OldDevnameHook, which was never used.

Handle PFC on rx side, efficiently, thanks to Jan Dubiec.

fix typos noticed by James Cameron.

Tested with MS-CHAP and CBCP options on Solaris and added options
to makefile.
Repaired support for use of gcc on Solaris x86 -- 32 bit modules also
need -fno-builtin.
MPPE changes broke plain MS-CHAP; repaired errors and cleaned up
compilation warnings due to char/unsigned char differences with non-gcc
compilers.

Set up the mask of signals we handle in one place not too
Remove unnecessary sigaddset in kill_my_pg