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

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