tunefs: allowing underscores in file system label

Eygene Ryabinkin rea-fbsd at codelabs.ru
Fri May 2 11:41:07 UTC 2008


Dmitry, good day.

Fri, May 02, 2008 at 03:03:59PM +0400, Dmitry Morozovsky wrote:
> any objection to the following simple patch? It would help using glabel be a 
> bit more consistent, as many msdosfs labels contain underscores.
> 
> Index: tunefs.c
> ===================================================================
> RCS file: /home/ncvs/src/sbin/tunefs/tunefs.c,v
> retrieving revision 1.42
> diff -u -r1.42 tunefs.c
> --- tunefs.c	9 Apr 2004 19:58:40 -0000	1.42
> +++ tunefs.c	2 May 2008 11:00:24 -0000
> @@ -139,8 +139,9 @@
>  			found_arg = 1;
>  			name = "volume label";
>  			Lvalue = optarg;
> -			i = -1;
> -			while (isalnum(Lvalue[++i]));
> +			i = 0;
> +			while (isalnum(Lvalue[i]) || Lvalue[i] == '_')
> +				i++;
>  			if (Lvalue[i] != '\0') {
>  				errx(10,
>  				"bad %s. Valid characters are alphanumerics.",

I think that the error message should be changed too: "... Valid
characters are alphanumerics and underscores.".
-- 
Eygene


More information about the freebsd-current mailing list