Date: Wed, 28 Feb 2007 12:00:05 -0800 From: Doug Barton <dougb@FreeBSD.org> To: Gabor Kovesdan <gabor@FreeBSD.org> Cc: Dominic Fandrey <lon_kamikaze@gmx.de>, cvs-all@FreeBSD.org, ports-committers@FreeBSD.org, cvs-ports@FreeBSD.org, erwin@freebsd.org, Christian Lackas <delta@lackas.net> Subject: Re: cvs commit: ports/security/vpnc Makefile ports/security/vpnc/files vpnc.in vpnc.sh Message-ID: <45E5DF45.6010407@FreeBSD.org> In-Reply-To: <45E5C97A.6060401@FreeBSD.org> References: <200702261857.l1QIvVMT051664@repoman.freebsd.org> <45E3701E.9000900@FreeBSD.org> <45E5C97A.6060401@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
This is a multi-part message in MIME format. --------------090309060604090302030002 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Gabor Kovesdan wrote: > Hello, > > thanks for pointing these out. Could you check the patch at > http://gabor.t-hosting.hu/patches/security-vpnc.diff to make sure I got > all your points correctly? While here, I changed the wrapping a bit to > make it easier to read. > Christian, do you approve these changes suggested by Doug? No problem, glad to help. I have just a couple small suggestions in the attached patch. First, it's ok (and you probably should) have the shutdown KEYWORD in there. It's the FreeBSD KEYWORD that is obsolete. The few other changes are mostly whitespace, and the removal of now obsolete code at the end of _start(). Please test this thoroughly before committing. Regards, Doug -- This .signature sanitized for your protection --------------090309060604090302030002 Content-Type: text/plain; name="vpnc.dougb.diff" Content-Transfer-Encoding: 7bit Content-Disposition: inline; filename="vpnc.dougb.diff" --- vpnc.in.patched Wed Feb 28 10:11:37 2007 +++ vpnc.in Wed Feb 28 10:10:53 2007 @@ -12,6 +12,7 @@ # PROVIDE: vpnc # REQUIRE: LOGIN +# KEYWORD: shutdown # Default settings - don't change this. : ${vpnc_enable="NO"} @@ -38,12 +39,10 @@ for config in $vpnc_conf; do - # The current configuration file. - + # The current configuration file. current="$vpnc_conf_dir/$config" - # Start vpnc. - + # Start vpnc. if ! $command $current $vpnc_flags; then status=$? echo "Running 'vpnc $current $vpnc_flags' failed." @@ -60,12 +59,7 @@ # Wait for the system to catch up. /bin/sleep 1 - done - else - # No configuration files given, run unmanaged. - $command $vpnc_flags - return $? - fi + done } vpnc_stop() { @@ -94,8 +88,7 @@ # Run the disconnect command. $command-disconnect - - done + done # Remove the connection record. --------------090309060604090302030002--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?45E5DF45.6010407>