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

Re: Problem with svn_wc_process_committed

From: Philip Martin <philip_at_codematters.co.uk>
Date: 2002-07-13 20:29:23 CEST

Karl Fogel <kfogel@newton.ch.collab.net> writes:

> Philip Martin <philip@codematters.co.uk> writes:
> > I have a patch that implements this. It's quite straightforward and I
> > believe it works as well as the current implementation. Given our
> > current corruption bug, should I commit it?
> I hate to ask you to wait... But, I think it would be best to wait.
> Until we know more, we should avoid further code churn in libsvn_wc,
> except for outright bug fixes.
> How about this: post the patch here first. At least that way it can
> be reviewed while we're working on the bug.

I believe the problem has been identified. Kevin interrupted a
previous commit, which would leave some directories locked.
harvest_committables calls lock_dir to lock directories but doesn't
check that it succeeds. So there should have been an "already locked"
error before starting the commit, but that did not occur, the commit
went ahead and then the NULL lock structure from the failed lock
attempt caused a core dump.

Philip Martin
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Jul 13 21:38:29 2002

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.