bin/84217: "enable proxy" is ignored in ppp.conf

[Tsurutani Naoki]

>Number:         84217
>Category:       bin
>Synopsis:       "enable proxy" is ignored in ppp.conf
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    freebsd-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Thu Jul 28 11:40:11 GMT 2005
>Closed-Date:
>Last-Modified:
>Originator:     Tsurutani Naoki
>Release:        FreeBSD 5.4-STABLE i386
>Organization:
>Environment:
System: FreeBSD polymer3.scphys.kyoto-u.ac.jp 5.4-STABLE FreeBSD 5.4-STABLE #6: Fri Jul 1 17:32:43 JST 2005 root at polymer3.scphys.kyoto-u.ac.jp:/usr/local/work/usr/obj/usr/src/sys/POLYMER i386


	
>Description:
	When use /usr/sbin/ppp with PoPToP, "enable proxy" in /etc/ppp/ppp.conf
	is ignored.
	
>How-To-Repeat:
	Always using ppp with PoPToP (installed from ports without any Macros).
	I have no serial line, so I don't know PoPToP affects this phenomna.
	my ppp.conf is as follows:
	loop:
	 set log Phase Chat Connect LCP IPCP CCP tun command
	 #set log All
	 set device localhost:pptp
	 set dial
	 set login
	 set ifaddr 10.1.1.1 10.1.1.101-10.1.1.105 255.255.255.128
	 set server /var/tmp/loop "" 0177
	
	loop-in:
	 set timeout 0
	 allow mode direct
	
	pptp:
	 load loop
	 set timeout 0
	 disable CHAP MSCHAP PAP
	 enable MSCHAPv2
	 disable deflate pred1
	 deny deflate pred1
	 set mppe 128 stateless
	 enable MPPE
	 accept MPPE
	 enable chap81
	 enable proxy
	 set dns 10.1.1.1
	 accept dns

	When i write "enable proxy" prior to "set ifaddr ..." line, 
	this does not occur.
	
>Fix:
	I checked when entries in ppp.conf was interpreted with arranging code
	of ppp, and i found that function about proxy is called just after
	"set ifaddr" is treated. At that time, flag to enable proxy is
	not set yet, so if treating "enable proxy" later, nothing can happen.
	As i checked in above way, each entries in ppp.conf is treated
	one by one, but it is not correct way, i think.
	


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