[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

segfault in propget-cmd.c

From: Mattias Rönnblom <hofors_at_lysator.liu.se>
Date: 2001-11-08 09:46:50 CET

Hi,

I've just built the latest version of subversion (revision 423, if
I've understood everything correctly), and tried this:

[98%] matro@mortain$~> mkdir -p svnrepro/svntest
[98%] matro@mortain$~> svnadmin create /home/matro/svnrepro/svntest/
[98%] matro_at_mortain$~> svn co file:///home/matro/svnrepro/svntest/
[98%] matro@mortain$~> cd svntest/
[98%] matro@mortain$~/svntest> touch some-file
[98%] matro@mortain$~/svntest> svn add some-file
A some-file
[98%] matro@mortain$~/svntest> svn propset some-key some-value some-file
property `some-key' set on 'some-file'
[98%] matro@mortain$~/svntest> svn propget some-key some-file
some-file - some-key : some-value
Segmentation fault (core dumped)
[98%] matro@mortain$~/svntest>

I guess this isn't the desired behaviour. This tiny patch seems to fix
the problem.

Index: subversion/clients/cmdline/propget-cmd.c
===================================================================
--- subversion/clients/cmdline/.svn/text-base/propget-cmd.c Wed Nov 7 16:17:08 2001
+++ subversion/clients/cmdline/propget-cmd.c Thu Nov 8 09:44:04 2001
@@ -76,7 +76,7 @@
       SVN_ERR (svn_client_propget (&props, propname, target,
                                    opt_state->recursive, pool));
 
- for (hi = apr_hash_first(pool, props); hi; apr_hash_next(hi))
+ for (hi = apr_hash_first(pool, props); hi; hi = apr_hash_next(hi))
         {
           const char * filename;
           const svn_string_t *propval;

Kind regards,
        Mattias

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Oct 21 14:36:48 2006

This is an archived mail posted to the Subversion Dev mailing list.