Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 11 Dec 2016 00:25:28 +0100
From:      Oliver Pinter <oliver.pinter@hardenedbsd.org>
To:        Devin Teske <dteske@freebsd.org>
Cc:        src-committers@freebsd.org, svn-src-all@freebsd.org,  svn-src-head@freebsd.org
Subject:   Re: svn commit: r307402 - head/usr.sbin/bsdconfig/share
Message-ID:  <CAPQ4ffvne84QnxGQztEjO9g8Ft13SdwxkHaPG_awki66pH54LA@mail.gmail.com>
In-Reply-To: <201610162059.u9GKxSr2065027@repo.freebsd.org>
References:  <201610162059.u9GKxSr2065027@repo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On 10/16/16, Devin Teske <dteske@freebsd.org> wrote:
> Author: dteske
> Date: Sun Oct 16 20:59:28 2016
> New Revision: 307402
> URL: https://svnweb.freebsd.org/changeset/base/307402
>
> Log:
>   Guard against bad service name argument(s) to load_rc_config()
>
>   MFC after:	3 days
>   X-MFC-to:	stable/11 stable/10

Is this MFC still needed? The 3 days already expired.

>
> Modified:
>   head/usr.sbin/bsdconfig/share/sysrc.subr
>
> Modified: head/usr.sbin/bsdconfig/share/sysrc.subr
> ==============================================================================
> --- head/usr.sbin/bsdconfig/share/sysrc.subr	Sun Oct 16 19:50:10
> 2016	(r307401)
> +++ head/usr.sbin/bsdconfig/share/sysrc.subr	Sun Oct 16 20:59:28
> 2016	(r307402)
> @@ -256,7 +256,9 @@ f_sysrc_service_configs()
>  			last_name=
>  			print_name() {
>  				local name="$1"
> -				[ "$name" = "$last_name" ] && return
> +				case "$name" in
> +				""|.|..|*/*|"$last_name") return ;;
> +				esac
>  				echo "$name" >&9
>  				last_name="$name"
>  			}
> _______________________________________________
> svn-src-head@freebsd.org mailing list
> https://lists.freebsd.org/mailman/listinfo/svn-src-head
> To unsubscribe, send any mail to "svn-src-head-unsubscribe@freebsd.org"
>



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAPQ4ffvne84QnxGQztEjO9g8Ft13SdwxkHaPG_awki66pH54LA>