[Bug 11866] rsync fails (failed to re-stat) when using double fuzzy + link-dest on renamed files

2017-10-08 Thread just subscribed for rsync-qa from bugzilla via rsync
https://bugzilla.samba.org/show_bug.cgi?id=11866

--- Comment #5 from Ben RUBSON  ---
Thank you very much for the merge Wayne !

-- 
You are receiving this mail because:
You are the QA Contact for the bug.

-- 
Please use reply-all for most replies to avoid omitting the mailing list.
To unsubscribe or change options: https://lists.samba.org/mailman/listinfo/rsync
Before posting, read: http://www.catb.org/~esr/faqs/smart-questions.html

[Bug 11866] rsync fails (failed to re-stat) when using double fuzzy + link-dest on renamed files

2017-10-08 Thread just subscribed for rsync-qa from bugzilla via rsync
https://bugzilla.samba.org/show_bug.cgi?id=11866

Wayne Davison  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|NEW |RESOLVED

--- Comment #4 from Wayne Davison  ---
Thanks for figuring out the issue and providing a patch. I've committed it to
git.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.

-- 
Please use reply-all for most replies to avoid omitting the mailing list.
To unsubscribe or change options: https://lists.samba.org/mailman/listinfo/rsync
Before posting, read: http://www.catb.org/~esr/faqs/smart-questions.html

[Bug 11866] rsync fails (failed to re-stat) when using double fuzzy + link-dest on renamed files

2017-10-04 Thread just subscribed for rsync-qa from bugzilla via rsync
https://bugzilla.samba.org/show_bug.cgi?id=11866

--- Comment #3 from Ben RUBSON  ---
Hi,

Could it be possible to merge this please ?
It's really tiny (one character) and easily understandable :)

And it avoids silent data loss !

Thank you very much !

Ben

-- 
You are receiving this mail because:
You are the QA Contact for the bug.

-- 
Please use reply-all for most replies to avoid omitting the mailing list.
To unsubscribe or change options: https://lists.samba.org/mailman/listinfo/rsync
Before posting, read: http://www.catb.org/~esr/faqs/smart-questions.html

[Bug 11866] rsync fails (failed to re-stat) when using double fuzzy + link-dest on renamed files

2017-07-04 Thread just subscribed for rsync-qa from bugzilla via rsync
https://bugzilla.samba.org/show_bug.cgi?id=11866

--- Comment #2 from Ben RUBSON  ---
Created attachment 13342
  --> https://bugzilla.samba.org/attachment.cgi?id=13342=edit
rsync_double_fuzzy_11866

Bug found, patch attached.
Wayne could you please review and commit please ?
Thank you very much !

-- 
You are receiving this mail because:
You are the QA Contact for the bug.

-- 
Please use reply-all for most replies to avoid omitting the mailing list.
To unsubscribe or change options: https://lists.samba.org/mailman/listinfo/rsync
Before posting, read: http://www.catb.org/~esr/faqs/smart-questions.html

[Bug 11866] rsync fails (failed to re-stat) when using double fuzzy + link-dest on renamed files

2017-07-04 Thread just subscribed for rsync-qa from bugzilla via rsync
https://bugzilla.samba.org/show_bug.cgi?id=11866

--- Comment #1 from Ben RUBSON  ---
I can easily reproduce this dangerous bug.
Both sides running rsync 3.1.2.

# cd /tmp/ && mkdir -p test && cd test && rm -f *
# ssh $usr@$srv "rm -rf /tmp/dst*"

# for i in `seq 0 9`
  do
  echo content_$i > $i
  done

# rsync -a . $usr@$srv:/tmp/dst1/

# rsync -a --link-dest=../dst1 . $usr@$srv:/tmp/dst2/

# ssh $usr@$srv "ls -lin /tmp/dst1 /tmp/dst2"
/tmp/dst1:
total 45
660319 -rw-r--r--  2 501  0  10  4 Jul 12:55 0
660320 -rw-r--r--  2 501  0  10  4 Jul 12:55 1
660321 -rw-r--r--  2 501  0  10  4 Jul 12:55 2
660322 -rw-r--r--  2 501  0  10  4 Jul 12:55 3
660323 -rw-r--r--  2 501  0  10  4 Jul 12:55 4
660324 -rw-r--r--  2 501  0  10  4 Jul 12:55 5
660325 -rw-r--r--  2 501  0  10  4 Jul 12:55 6
660326 -rw-r--r--  2 501  0  10  4 Jul 12:55 7
660327 -rw-r--r--  2 501  0  10  4 Jul 12:55 8
660328 -rw-r--r--  2 501  0  10  4 Jul 12:55 9
/tmp/dst2:
total 45
660319 -rw-r--r--  2 501  0  10  4 Jul 12:55 0
660320 -rw-r--r--  2 501  0  10  4 Jul 12:55 1
660321 -rw-r--r--  2 501  0  10  4 Jul 12:55 2
660322 -rw-r--r--  2 501  0  10  4 Jul 12:55 3
660323 -rw-r--r--  2 501  0  10  4 Jul 12:55 4
660324 -rw-r--r--  2 501  0  10  4 Jul 12:55 5
660325 -rw-r--r--  2 501  0  10  4 Jul 12:55 6
660326 -rw-r--r--  2 501  0  10  4 Jul 12:55 7
660327 -rw-r--r--  2 501  0  10  4 Jul 12:55 8
660328 -rw-r--r--  2 501  0  10  4 Jul 12:55 9

// Perfect, --link-dest correctly works.

# for i in `seq 0 9`
  do
  cp -a $i $i.orig
  echo content_$i >> $i
  done

# rsync -a -y --progress --stats --link-dest=../dst1 . $usr@$srv:/tmp/dst2/
./
0
 20 100%0.00kB/s0:00:00 (xfr#1, to-chk=19/21)
0.orig
 10 100%9.77kB/s0:00:00 (xfr#2, to-chk=18/21)
1
 20 100%   19.53kB/s0:00:00 (xfr#3, to-chk=17/21)
1.orig
 10 100%9.77kB/s0:00:00 (xfr#4, to-chk=16/21)
2
 20 100%   19.53kB/s0:00:00 (xfr#5, to-chk=15/21)
2.orig
 10 100%9.77kB/s0:00:00 (xfr#6, to-chk=14/21)
3
 20 100%   19.53kB/s0:00:00 (xfr#7, to-chk=13/21)
3.orig
 10 100%9.77kB/s0:00:00 (xfr#8, to-chk=12/21)
4
 20 100%   19.53kB/s0:00:00 (xfr#9, to-chk=11/21)
4.orig
 10 100%9.77kB/s0:00:00 (xfr#10, to-chk=10/21)
5
 20 100%9.77kB/s0:00:00 (xfr#11, to-chk=9/21)
5.orig
 10 100%4.88kB/s0:00:00 (xfr#12, to-chk=8/21)
6
 20 100%9.77kB/s0:00:00 (xfr#13, to-chk=7/21)
6.orig
 10 100%4.88kB/s0:00:00 (xfr#14, to-chk=6/21)
7
 20 100%9.77kB/s0:00:00 (xfr#15, to-chk=5/21)
7.orig
 10 100%4.88kB/s0:00:00 (xfr#16, to-chk=4/21)
8
 20 100%9.77kB/s0:00:00 (xfr#17, to-chk=3/21)
8.orig
 10 100%4.88kB/s0:00:00 (xfr#18, to-chk=2/21)
9
 20 100%9.77kB/s0:00:00 (xfr#19, to-chk=1/21)
9.orig
 10 100%3.26kB/s0:00:00 (xfr#20, to-chk=0/21)
Number of files: 21 (reg: 20, dir: 1)
Number of created files: 10 (reg: 10)
Number of deleted files: 0
Number of regular files transferred: 20
Total file size: 300 bytes
Total transferred file size: 300 bytes
Literal data: 190 bytes
Matched data: 110 bytes
File list size: 0
File list generation time: 0.002 seconds
File list transfer time: 0.000 seconds
Total bytes sent: 1,422
Total bytes received: 549
sent 1,422 bytes  received 549 bytes  1,314.00 bytes/sec
total size is 300  speedup is 0.15

# ssh $usr@$srv "ls -lin /tmp/dst2"
total 90
660330 -rw-r--r--  1 501  0  20  4 Jul 12:55 0
660331 -rw-r--r--  1 501  0  10  4 Jul 12:55 0.orig
660332 -rw-r--r--  1 501  0  20  4 Jul 12:55 1
660333 -rw-r--r--  1 501  0  10  4 Jul 12:55 1.orig
660334 -rw-r--r--  1 501  0  20  4 Jul 12:55 2
660335 -rw-r--r--  1 501  0  10  4 Jul 12:55 2.orig
660336 -rw-r--r--  1 501  0  20  4 Jul 12:55 3
660343 -rw-r--r--  1 501  0  10  4 Jul 12:55 3.orig
660344 -rw-r--r--  1 501  0  20  4 Jul 12:55 4
660345 -rw-r--r--  1 501  0  10  4 Jul 12:55 4.orig
660346 -rw-r--r--  1 501  0  20  4 Jul 12:55 5
660347 -rw-r--r--  1 501  0  10  4 Jul 12:55 5.orig
660348 -rw-r--r--  1 501  0  20  4 Jul 12:55 6
660349 -rw-r--r--  1 501  0  10  4 Jul 12:55 6.orig
660350 -rw-r--r--  1 501  0  20  4 Jul 12:55 7
660351 -rw-r--r--  1 501  0  10  4 Jul 12:55 7.orig
660352 -rw-r--r--  1 501  0  20  4 Jul 12:55 8
660353 -rw-r--r--  1 501  0  10  4 Jul 12:55 8.orig
660354 -rw-r--r--  1 501  0  20  4 Jul 12:55 9
660355 -rw-r--r--  1 501  0  10  4 Jul 12:55 9.orig

// Perfect, update correctly done.

# rsync -a -yy --progress --stats --link-dest=../dst1 . $usr@$srv:/tmp/dst3/
sending incremental file list
./
0
 20 100%0.00kB/s0:00:00 (xfr#1, to-chk=19/21)
1
 20 100%   19.53kB/s0:00:00 (xfr#2, to-chk=17/21)
2
 20 100%   19.53kB/s0:00:00 (xfr#3, to-chk=15/21)
3
 20 100%   19.53kB/s0:00:00 (xfr#4,