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