GNU bug report logs - #52410
"mv -T --backup=numbered hello world/" renameat2 infinite loop

Previous Next

Full log

View this message in rfc822 format

From: Paul Eggert <eggert <at> cs.ucla.edu>
To: Vincent Vermilya <vincentvermilya <at> gmail.com>
Cc: Gnulib bugs <bug-gnulib <at> gnu.org>, 52410 <at> debbugs.gnu.org
Subject: bug#52410: "mv -T --backup=numbered hello world/" renameat2 infinite loop
Date: Fri, 10 Dec 2021 14:15:02 -0800

[Message part 1 (text/plain, inline)]

On 12/10/21 02:22, Vincent Vermilya wrote:
> If DEST directory has a trailing slash in "mv -T --backup=numbered", mv
> gets stuck in an infinite loop. This does not happen without the trailing
> slash. strace shows renameat2 as the call that gets looped on.

Thanks for the bug report. This is due to a Gnulib bug, so I'm cc'ing 
bug-gnulib.

I installed the first three attached patches into Gnulib. The first one 
should fix the bug there, and the other two are refactoring cleanups.

I installed the last two attached patches into coreutils to propagate 
this fix into coreutils and add a regression test.

[0001-backupfile-fix-numbered-backups-for-XXX.patch (text/x-patch, attachment)]

[0002-backupfile-prefer-signed-integers.patch (text/x-patch, attachment)]

[0003-backupfile-assume-C99-decls.patch (text/x-patch, attachment)]

[0001-build-update-gnulib-submodule-to-latest.patch (text/x-patch, attachment)]

[0002-mv-Bug-52410-fix.patch (text/x-patch, attachment)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.