bugrdiff-backup - Bugs: bug #39476, regression error on encfs...

 
 

bug #39476: regression error on encfs destination with long filename

Submitted by:  None
Submitted on:  Fri 12 Jul 2013 08:41:18 PM UTC  
 
Category: NoneSeverity: 2 - Minor
Item Group: NoneStatus: Ready For Test
Privacy: PublicAssigned to: None
Open/Closed: Open

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Fri 12 Feb 2016 09:48:37 AM UTC, comment #1:

Need to test this patch.
Will do so on the new tree at:
https://github.com/sol1/rdiff-backup

david kempe <davekempe>
Project Administrator
Fri 12 Jul 2013 08:41:18 PM UTC, original submission:

Hello,

After an aborted backup I always get this error:

Previous backup seems to have failed, regressing destination now.
...
AssertionError: Bad index order: ('long_filename_data', '1') >= ('b',)

I append a patch with this bug report. It cures the error but I did not check whether the regression is doing now what it is intended to do. The error is caused by filenames which are to long to be saved on the target directory. Rdiff-backup uses the 'AlternateMirrorName' technique in this case. My target directory is an encfs encrypted directory with filename encryption. Filenames are limited to 189 bytes.

Here are my findings. Line 275 of rorpiter.py throws the error. The line test for the index order. The index order is an alphabetical filename order. Line 255 of longname.py opens a new 'mirror_rp' with 'index' ,,('long_filename_data', '1')'' for regression. This index is used for the index test but it should be tested against the long filename related to this regression.

In my example the long filename is 190 bytes 'a190byteaaaa...aaa'. A second file following the long file name is necessary to throw the error. I use filename 'b'. The error only occurs if both files are alphabetically placed before the string 'long_filename_data'.

my system:
Debian 7.1
rdiff-backup 1.2.8

I know, this is more or less an auto information list. V1.2.8 is from 2009. Should I post the bug report on Debian to get the patch publicized?

Many thanks to the developers of rdiff-backup!!!

Christian

Anonymous

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

Attached Files
file #28557:  long-filename-regression.patch added by None (571B - text/x-diff)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by davekempe (Posted a comment)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 3 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Fri 12 Feb 2016 09:48:37 AM UTCdavekempeSeverity3 - Normal=>2 - Minor
      StatusNone=>Ready For Test
    Fri 12 Jul 2013 08:41:18 PM UTCNoneAttached File-=>Added long-filename-regression.patch, #28557

    Back to the top


    Powered by Savane 3.1-cleanup1