rtag assertion in recurse.c:642

[Mark E. Hamilton]

Hi,

We've been using cvs-1.11.6 for a long time, so I thought I'd better 
look into upgrading. I installed and built 1.11.17 and started running 
some of our tests. I ran into this assert when trying to rtag a 
repository. I created a simple test repository with one top-level 
direcotry and one file. When I try to tag it with 1.11.6 it works fine; 
the same command using 1.11.17 fails.

Our main CVS server is a SourceForge Enterprise Edition system, which is 
currently running cvs-1.11.5. As it happens we have a 1.11.17 binary 
over there we'd like to upgrade to as well. However, this exhibits the 
same problem; when using the 1.11.5 binary the rtag works, but the 
1.11.17 binary asserts (though with a slightly longer message.)

Has the behavior of 'cvs rtag .' been changed, or is this in fact a bug 
(or, have I been misusing rtag for a long time?)

sahp4737% /usr/netpub/cvs-1.11.6/bin/cvs -d /var/tmp/mhamilt/repos rtag 
'test_tag' .
cvs rtag: Tagging .
cvs rtag: Tagging CVSROOT
cvs rtag: Tagging blarg

sahp4737% /usr/netpub/cvs-1.11.17/bin/cvs -d /var/tmp/mhamilt/repos rtag 
'test_tag' .
cvs: recurse.c:642: do_recursion: Assertion `strstr (repository, "/./") 
== ((void *)0)' failed.
cvs [rtag aborted]: received abort signal


sahp4737% export CVS_SERVER=/sourceforge/cvs/bin/cvs
sahp4737% /usr/netpub/cvs-1.11.17/bin/cvs -d 
:ext:sourceforge.sandia.gov:/cvsroot/playground rtag 'test_tag' .
cvs rtag: Tagging .
cvs rtag: Tagging CVSROOT
cvs rtag: Tagging CVSROOT/Emptydir
cvs rtag: Tagging playdoh
<snip>

sahp4737% export CVS_SERVER=/sourceforge/cvs/bin/cvs.1.11.17
sahp4737% /usr/netpub/cvs-1.11.17/bin/cvs -d 
:ext:sourceforge.sandia.gov:/cvsroot/playground rtag 'test_tag' .
cvs.1.11.17 [rtag aborted]: received abort signal
cvs.1.11.17: recurse.c:642: do_recursion: Assertion `(__extension__ 
(__builtin_constant_p ("/./") && ((size_t)(const void *)(("/./") + 1) - 
(size_t)(const void *)("/./") == 1) ? (((const char *) ("/./"))[0] == 
'\0' ? (char *) (size_t) (repository) : (((const char *) ("/./"))[1] == 
'\0' ? (__extension__ (__builtin_constant_p (((const char *) 
("/./"))[0]) && (((const char *) ("/./"))[0]) == '\0' ? (char *) 
__rawmemchr (repository, ((const char *) ("/./"))[0]) : strchr 
(repository, ((const char *) ("/./"))[0]))) : strstr (repository, 
"/./"))) : strstr (repository, "/./"))) == ((void *)0)' failed.


--
----------------
Mark E. Hamilton
Orion International Technologies, Inc.
Sandia National Laboratory, NM.
844-7666