[Box Backup] Bug fix - don't die on lstat failure

Ben Summers boxbackup@fluffy.co.uk
Mon, 13 Dec 2004 22:04:10 +0000


On 13 Dec 2004, at 21:58, Joe Krahn wrote:

> Ben Summers wrote:
>> On 13 Dec 2004, at 18:12, Joe Krahn wrote:
>>> I found why I had problems getting my home directory backed up.
>>> If bbackupd get a bad result of lstat(), it aborts.
>>>
>>> In my case, the problem is actually from having a DVD .ISO file.
>>> Files greater than 4GB in size fail with stat; they need stat64.
>>> So, stat64() support is worth adding.
>>>
>>> But, it should also not abort the whole backup run due to one bad 
>>> stat call. It's better to just skip the item for which fstat failed 
>>> by chaning BackupClientDirectoryRecord.cpp from:
>> Isn't enabling linux LFS supposed to make it call that function 
>> instead of lstat itself?
>> Failures reading a file don't cause the backup to abort, but it does 
>> assume (at the moment) that getting file information is going to 
>> work.
>> I'll make it more tolerant.
> Yes, I think you can just define _FILE_OFFSET_BITS=64 and 
> automatically get 64-bit versions of stat structs/functions. But, I 
> don't see this in the boxbackup sources.

Which version of Box Backup are you using? v0.09 should run a test to 
see if this does anything useful, and then put that define in all the 
makefiles if it does.

Can you run ./configure again, and see if it says

   Linux LFS: yes

in the tests at the beginning? If it says yes, you should see the 
required define on the compiler command line as make does it's magic.

Ben