Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 13 Jan 2001 09:18:20 -0500 (EST)
From:      Garrett Wollman <wollman@khavrinen.lcs.mit.edu>
To:        Samuel Tardieu <sam@inf.enst.fr>
Cc:        freebsd-bugs@FreeBSD.ORG
Subject:   Re: bin/24271: dumpon should check its argument more
Message-ID:  <200101131418.JAA75138@khavrinen.lcs.mit.edu>
In-Reply-To: <2001-01-13-13-41-54%2Btrackit%2Bsam@inf.enst.fr>
References:  <200101121610.f0CGA7a52473@freefall.freebsd.org> <20010113095245.A66238@wantadilla.lemis.com> <20010113004100.A12683@walton.maths.tcd.ie> <2001-01-13-13-41-54%2Btrackit%2Bsam@inf.enst.fr>

next in thread | previous in thread | raw e-mail | index | archive | help
<<On Sat, 13 Jan 2001 13:41:53 +0100, Samuel Tardieu <sam@inf.enst.fr> said:

> +		case 'f':
> +		  force = 1;
> +			break;

> +			(void)snprintf(special, MAXPATHLEN, "%s%s", _PATH_DEV, argv[0]);
> +			if (!setfsent ())

> +				if (!strcmp(fsp->fs_spec, special)) {
> +					if (!strcmp(fsp->fs_type, FSTAB_SW))

> +					errx(EX_USAGE,
> +							 "%s: -f required to use a %s partition",
> +							 special,
> +							 fsp->fs_vfstype);

> +			errx(EX_USAGE,
> +					 "%s: -f required unless using a declared swap partition",
> +					 special);

> +		"usage: dumpon [-f] [-v] special_file\n"

This patch introduces at least eleven new style bugs.

-GAWollman



To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-bugs" in the body of the message




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