linux-ext4 - Re: [REPOST][RFC][PATCH] ext4: unify each meaning of the offset in ext4_check_dir

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 for Android: free password hash cracker in your pocket

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

Message-id: <F9B87217-D6BA-4BF6-BD13-1B960DB4B20C@sun.com>
Date:	Thu, 14 Jan 2010 10:18:53 -0500
From:	Andreas Dilger <adilger@....com>
To:	Toshiyuki Okajima <toshi.okajima@...fujitsu.com>
Cc:	tytso@....edu, linux-ext4@...r.kernel.org
Subject: Re: [REPOST][RFC][PATCH] ext4: unify each meaning of the offset in
 ext4_check_dir_entry calling from some functions.

On 2010-01-14, at 01:57, Toshiyuki Okajima wrote:
> From: Toshiyuki Okajima <toshi.okajima@...fujitsu.com>
>
> "offset" of the error message of ext4_check_dir_entry() might change  
> the
> meaning by the caller. There are 2 meanings:
> - "File offset"
>  called by:
>   ext4_readdir, htree_dirblock_to_tree, search_dirblock,  
> ext4_dx_find_entry,
>   empty_dir
> - "Buffer offset"
>  called by:
>   add_dirent_to_buf, ext4_delete_entry

This makes sense to fix.

> The best way to solve this problem is to change the meaning of
> "Buffer offset" into "File offset" but it is not easy.
> However, we can solve this problem easily if we unify the meanings  
> into
> "Buffer offset". So, instead of "File Offset" meaning, we add the  
> block number information to this message.
>
> --- Examples ---
> Error message which is changed by this patch:
> EXT4-fs error (device loop0): htree_dirblock_to_tree: bad entry in  
> directory
> #12: rec_len is too small for name_len - block_nr=78:offset=0,  
> inode=216,
> rec_len=12, name_len=11

I personally would prefer to see "block=78, offset=0" instead of the  
above.

> @@ -84,9 +84,10 @@ int ext4_check_dir_entry(const char *fun
>
> 	if (error_msg != NULL)
> 		ext4_error(dir->i_sb, function,
> +			"bad entry in directory #%lu: %s - block_nr=%llu:"
> 			"offset=%u, inode=%u, rec_len=%d, name_len=%d",
> +			dir->i_ino, error_msg,
> +			(u64)bh->b_blocknr, (u32)(offset%bh->b_size),

One problem here is that "(u64)" is not necessarily a "long long".  On  
some 64-bit platforms it will be a "long" and this will generate a  
compiler warning.  Better to use "(unsigned long long)", and you may  
as well use "(unsigned)" for the offset value.

Cheers, Andreas
--
Andreas Dilger
Sr. Staff Engineer, Lustre Group
Sun Microsystems of Canada, Inc.

--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html