Re: [PATCH] cifs: Trivial comment fixes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 10 Apr 2013 13:16:52 -0400
scott.lovenberg@xxxxxxxxx wrote:

> From: Scott Lovenberg <scott.lovenberg@xxxxxxxxx>
> 
> Two trivial comment fixes.
> 
> Signed-off-by: Scott Lovenberg <scott.lovenberg@xxxxxxxxx>
> ---
>  mount.cifs.c |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/mount.cifs.c b/mount.cifs.c
> index 4548fc6..58b127a 100644
> --- a/mount.cifs.c
> +++ b/mount.cifs.c
> @@ -975,7 +975,7 @@ parse_options(const char *data, struct parsed_mount_info *parsed_info)
>  
>  			fprintf(stderr, "bad option: gid=\"%s\"\n", value);
>  			return EX_USAGE;
> -		/* fmask fall through to file_mode */
> +		/* fmask falls through to file_mode */
>  		case OPT_FMASK:
>  			fprintf(stderr,
>  				"WARNING: CIFS mount option 'fmask' is\
> @@ -1145,7 +1145,7 @@ nocopy:
>  	if (got_cruid) {
>  		word_len = snprintf(txtbuf, sizeof(txtbuf), "%u", cruid);
>  
> -		/* comma + "cruid=" + terminating NULL == 6 */
> +		/* comma + "cruid=" + terminating NULL == 8 */
>  		if (out_len + word_len + 8 > MAX_OPTIONS_LEN) {
>  			fprintf(stderr, "Options string too long\n");
>  			return EX_USAGE;

Looks good. Will merge in a day or two assuming no one objects.

Thanks,
-- 
Jeff Layton <jlayton@xxxxxxxxxx>
--
To unsubscribe from this list: send the line "unsubscribe linux-cifs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux