Linux-Fsdevel Archive on lore.kernel.org
help / color / mirror / Atom feed
From: "Namjae Jeon" <namjae.jeon@samsung.com>
To: "'Tetsuhiro Kohada'" <kohada.t2@gmail.com>
Cc: <kohada.tetsuhiro@dc.mitsubishielectric.co.jp>,
	<mori.takahiro@ab.mitsubishielectric.co.jp>,
	<motai.hirotaka@aj.mitsubishielectric.co.jp>,
	"'Sungjong Seo'" <sj1557.seo@samsung.com>,
	<linux-fsdevel@vger.kernel.org>, <linux-kernel@vger.kernel.org>
Subject: RE: [PATCH 1/2] exfat: add dir-entry set checksum validation
Date: Mon, 10 Aug 2020 15:19:15 +0900	[thread overview]
Message-ID: <003e01d66ede$2c2fe620$848fb260$@samsung.com> (raw)
In-Reply-To: <20200807073049.24959-1-kohada.t2@gmail.com>

> Add checksum validation for dir-entry set when getting it.
> exfat_calc_dir_chksum_with_entry_set() also validates entry-type.
> 
> ** This patch depends on:
>   '[PATCH v3] exfat: integrates dir-entry getting and validation'
> 
> Signed-off-by: Tetsuhiro Kohada <kohada.t2@gmail.com>
> ---
>  fs/exfat/dir.c | 34 ++++++++++++++++++++++------------
>  1 file changed, 22 insertions(+), 12 deletions(-)
> 
> diff --git a/fs/exfat/dir.c b/fs/exfat/dir.c index c9715c7a55a1..2e79ac464f5f 100644
> --- a/fs/exfat/dir.c
> +++ b/fs/exfat/dir.c
> @@ -563,18 +563,27 @@ int exfat_remove_entries(struct inode *inode, struct exfat_chain *p_dir,
>  	return 0;
>  }
> 
> -void exfat_update_dir_chksum_with_entry_set(struct exfat_entry_set_cache *es)
> +static int exfat_calc_dir_chksum_with_entry_set(struct
> +exfat_entry_set_cache *es, u16 *chksum)
>  {
> -	int chksum_type = CS_DIR_ENTRY, i;
> -	unsigned short chksum = 0;
>  	struct exfat_dentry *ep;
> +	int i;
> 
> -	for (i = 0; i < es->num_entries; i++) {
> -		ep = exfat_get_validated_dentry(es, i, TYPE_ALL);
> -		chksum = exfat_calc_chksum16(ep, DENTRY_SIZE, chksum,
> -					     chksum_type);
> -		chksum_type = CS_DEFAULT;
> +	ep = container_of(es->de_file, struct exfat_dentry, dentry.file);
> +	*chksum = exfat_calc_chksum16(ep, DENTRY_SIZE, 0, CS_DIR_ENTRY);
> +	for (i = 0; i < es->de_file->num_ext; i++) {
> +		ep = exfat_get_validated_dentry(es, 1 + i, TYPE_SECONDARY);
> +		if (!ep)
> +			return -EIO;
> +		*chksum = exfat_calc_chksum16(ep, DENTRY_SIZE, *chksum, CS_DEFAULT);
>  	}
> +	return 0;
We can return checksum after removing u16 *chksum argument.
> +}
> +
> +void exfat_update_dir_chksum_with_entry_set(struct
> +exfat_entry_set_cache *es) {
> +	u16 chksum;
> +
> +	exfat_calc_dir_chksum_with_entry_set(es, &chksum);
>  	es->de_file->checksum = cpu_to_le16(chksum);
>  	es->modified = true;
>  }
> @@ -775,6 +784,7 @@ struct exfat_entry_set_cache *exfat_get_dentry_set(struct super_block *sb,
>  	struct exfat_entry_set_cache *es;
>  	struct exfat_dentry *ep;
>  	struct buffer_head *bh;
> +	u16 chksum;
> 
>  	if (p_dir->dir == DIR_DELETED) {
>  		exfat_err(sb, "access to deleted dentry"); @@ -839,10 +849,10 @@ struct
> exfat_entry_set_cache *exfat_get_dentry_set(struct super_block *sb,
>  		goto free_es;
>  	es->de_stream = &ep->dentry.stream;
> 
> -	for (i = 2; i < es->num_entries; i++) {
> -		if (!exfat_get_validated_dentry(es, i, TYPE_SECONDARY))
> -			goto free_es;
> -	}
> +	if (max_entries == ES_ALL_ENTRIES &&
> +	    ((exfat_calc_dir_chksum_with_entry_set(es, &chksum) ||
> +	      chksum != le16_to_cpu(es->de_file->checksum))))
Please add error print log if checksum mismatch error happen.
> +		goto free_es;
> 
>  	return es;
> 
> --
> 2.25.1



  parent reply	other threads:[~2020-08-10  6:19 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <CGME20200807073101epcas1p43ba30d0ff54cb09f90b7dc69c746d3e6@epcas1p4.samsung.com>
2020-08-07  7:30 ` Tetsuhiro Kohada
2020-08-07  7:30   ` [PATCH 2/2] exfat: write only modified part of dir-entry set Tetsuhiro Kohada
2020-08-10  6:19   ` Namjae Jeon [this message]
2020-08-12 15:17     ` [PATCH 1/2] exfat: add dir-entry set checksum validation Tetsuhiro Kohada

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='003e01d66ede$2c2fe620$848fb260$@samsung.com' \
    --to=namjae.jeon@samsung.com \
    --cc=kohada.t2@gmail.com \
    --cc=kohada.tetsuhiro@dc.mitsubishielectric.co.jp \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mori.takahiro@ab.mitsubishielectric.co.jp \
    --cc=motai.hirotaka@aj.mitsubishielectric.co.jp \
    --cc=sj1557.seo@samsung.com \
    --subject='RE: [PATCH 1/2] exfat: add dir-entry set checksum validation' \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).