linux-kernel - Re: [BUG] copy file result with zero

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] [thread-next>] [day] [month] [year] [list]

Message-Id: <CA140AD1-B715-46BF-81B3-EA2069CBC131@dilger.ca>
Date:	Sun, 2 Oct 2011 01:02:04 -0700
From:	Andreas Dilger <aedilger@...il.com>
To:	Dave Young <hidave.darkstar@...il.com>
Cc:	Jeff liu <jeff.liu@...cle.com>, Ted Ts'o <tytso@....edu>,
	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
	linux-ext4 development <linux-ext4@...r.kernel.org>
Subject: Re: [BUG] copy file result with zero

On 2011-10-01, at 11:41 PM, Jeff liu wrote:
>> On Sat, Oct 1, 2011 at 10:39 PM, Ted Ts'o <tytso@....edu> wrote:
>>> On Sat, Oct 01, 2011 at 10:01:35PM +0800, Dave Young wrote:
>>>> Weird problem, when I build app from source,
>>>> make; make install
>>>> run the command, but got "cannot execute binary file"
>>>> 
>>>> hexdump shows the installed binary is full of zero
>>>> 
>>>> Is it related to ext4 fiemap problem described below?
>>>> http://lwn.net/Articles/429349/
>>> 
>>> There is general agreement that /bin/cp should not have been relying
>>> on FIEMAP, and I believe the more recent versions of /bin/cp have
>>> removed that code by default pending implementation of
>>> SEEK_HOLE/SEEK_DATA.  That being said, ext4 had a workaround to its
>>> FIEMAP implementation that landed in 2.6.39, and you're using
>>> 3.1.0-rc6.
> 
> Actually, upstream cp(1) using FIEMAP only if the source file is sparse,  or else, it will do normal copy, i.e, block based.

Are there any distros that are shipping with a version of cp that depends on FIEMAP?  That would dramatically increase the severity of this problem, since orders of magnitude more users will hit the problem.

Dave, what distro were you seeing this problem on, and had you installed/upgraded your coreutils and/or kernel yourself?

>> Do you means It should work in 3.1.0-rc6 even with cp which depends fiemap?
>> 
>>> 
>>>> I finally managed to find the way to reproduce this:
>>>> just cp a elf binary A  to file B, then cp B to file C,  then you will get:
>>>> A == B != C
>>>> 
>>>> ie.
>>>> cp /bin/ls ls1
>>>> cp ls1 ls2
>>>> 
>>>> ls2 will be filled with zero
>>> 
>>> If you add a "sync" between the two copies, does that work around the
>>> problem?  I bet it will...
>> 
>> Yes, it works
>> 
>>> 
>>> My suggestion is to upgrade to a newer version of coreutils that
>>> doesn't try to use FIEMAP.
>> 
>> Thanks, will try
>> 
>>> 
>>>                                       - Ted
>>> 
>> 
>> 
>> 
>> -- 
>> Regards
>> Dave
>> --
>> 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
> 
> --
> 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


Cheers, Andreas





--
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/