linux-kernel - Re: [PATCH]: Added comments in fat.h

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <4ECE8A15.2010505@xenotime.net>
Date:	Thu, 24 Nov 2011 10:16:53 -0800
From:	Randy Dunlap <rdunlap@...otime.net>
To:	Rahul Bedarkar <rpal143@...il.com>
CC:	linux-kernel@...r.kernel.org
Subject: Re: [PATCH]: Added comments in fat.h

On 11/24/2011 04:46 AM, Rahul Bedarkar wrote:
> Hi,
> 
>     Added comments in fat.h
> 
>     signed off by <rpal143@...il.com>

Improper format for Signed-off-by: line.
See Documentation/SubmittingPatches, please.

and please Cc: the FAT fs maintainer.

> 
> [master b5334fc] Added comments in fat.h
>  1 files changed, 2 insertions(+), 1 deletions(-)
> 
> 
> diff --git a/fs/fat/fat.h b/fs/fat/fat.h
> index 1510a4d..3b64fe0 100644
> --- a/fs/fat/fat.h
> +++ b/fs/fat/fat.h
> @@ -75,7 +75,7 @@ struct msdos_sb_info {
>         struct fat_mount_options options;
>         struct nls_table *nls_disk;  /* Codepage used on disk */
>         struct nls_table *nls_io;    /* Charset used for input and display */
> -       const void *dir_ops;                 /* Opaque; default
> directory operations */

Your email client incorrectly split the above source line into 2 lines.
Maybe Documentation/email-clients.txt can help you.

> +       const void *dir_ops;         /* Opaque; default directory operations */
>         int dir_per_block;           /* dir entries per block */
>         int dir_per_block_bits;      /* log2(dir_per_block) */
> 
> @@ -179,6 +179,7 @@ static inline u8 fat_make_attrs(struct inode *inode)
>         return attrs;
>  }
> 
> +/* Sets the FAT attribute byte for thid inode */

thid should be third?  or this?

>  static inline void fat_save_attrs(struct inode *inode, u8 attrs)
>  {
>         if (fat_mode_can_hold_ro(inode))


-- 
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/