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

Re: "svn diff -rN" dereferences a NIL pointer and crashes with "Segmentation fault".

From: Malcolm Rowe <malcolm-svn-dev_at_farside.org.uk>
Date: 2006-02-20 20:29:01 CET

On Fri, Feb 17, 2006 at 01:54:35PM -0800, Garrett Rooney wrote:
> On 2/17/06, Garrett Rooney <rooneg@electricjellyfish.net> wrote:
> Ok, so here's a "fix" for the bug.

Looks good to me, please commit. With adjustment for bracket placement,
it may even apply cleanly to the 1.3.x branch as well.

> In an ideal world, nonrecursive checkouts would leave some indication
> that the directory in question exists, so we could check for this sort
> of case, but we don't live in an ideal world...
>

Yes, it's pretty poor at the moment - we really should work out the
'right' way to handle non-recursive checkouts, from an entries p-o-v.

> I'm also kind of curious what the "right" behavior is. Do we just
> skip over the missing directory? That's what I've done here, but I'm
> not convinced it's correct.
>

I think so - it's not part of the checked-out wc, so we arguably don't
want to see differences for it at all.

Regards,
Malcolm

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon Feb 20 20:29:28 2006

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.