Edgewall Software
Modify

Ticket #4000 (closed defect: fixed)

Opened 4 years ago

Last modified 8 months ago

Assertion failed: is_canonical(base, blen) while view changeset in DIFF/ZIP format

Reported by: anonymous Owned by: cboos
Priority: normal Milestone: 0.10.1
Component: general Version: 0.10
Severity: critical Keywords:
Cc:

Description

  1. set repository_dir to folder inside a repository, for example, d:\svn\repos\folder
  1. view changeset and choose one of the format below "Download in other formats"
  1. apache server dead due to SVN crash

Assertion failed: is_canonical(base, blen), file C:\Projects\subversion-1.4.0\subversion\libsvn_subr\path.c, line 114

There is no such problem is repository_dir is root of repository. All SVN verison and same problem there

Attachments

Change History

comment:1 Changed 4 years ago by cboos

  • Owner changed from jonas to cboos

Unbelievable but true ;)

comment:2 Changed 4 years ago by cboos

  • Status changed from new to assigned
  • Milestone set to 0.10.1

Looks like repos.svn_repos_dir_delta dislikes leading slashes. Well, if only svn could say that in a nicer way...

Some background info:

The follow change fixes the problem:

  • trac/versioncontrol/svn_fs.py

     
    489489            text_deltas = 0 # as this is anyway re-done in Diff.py... 
    490490            entry_props = 0 # "... typically used only for working copy updates" 
    491491            repos.svn_repos_dir_delta(old_root, 
    492                                       _to_svn(self.scope, old_path), '', 
     492                                      _to_svn(self.scope + old_path), '', 
    493493                                      new_root, 
    494494                                      _to_svn(self.scope + new_path), 
    495495                                      e_ptr, e_baton, authz_cb, 

but I need to give it more testing before I commit it.

comment:3 Changed 4 years ago by cboos

  • Status changed from assigned to closed
  • Resolution set to fixed

Patch applied in r4013 (trunk) and r4017 (0.10-stable).

comment:4 follow-ups: ↓ 5 ↓ 7 Changed 4 years ago by Markus Tacker <m@…>

With r4809 (0.10-stable) I get this error.

# trac-admin /var/trac/phpbb-rss resync
Resyncing repository history...
python: subversion/libsvn_subr/path.c:114: svn_path_join: Assertion `is_canonical(base, blen)' failed.

Is it related?

comment:5 in reply to: ↑ 4 ; follow-up: ↓ 6 Changed 4 years ago by cboos

Replying to Markus Tacker <m@tacker.org>:

> # trac-admin /var/trac/phpbb-rss resync

Can you redo it with DEBUG level logging, please?

comment:6 in reply to: ↑ 5 Changed 4 years ago by Markus Tacker <m@…>

Replying to cboos:

Can you redo it with DEBUG level logging, please?

That's all I get:

2007-03-01 10:53:10,652 Trac[loader] DEBUG: Loading egg plugin webadmin.logging from /usr/lib/python2.4/site-packages/TracWebAdmin-0.1.2dev_r4429-py2.4.egg
2007-03-01 10:53:10,654 Trac[loader] DEBUG: Loading egg plugin webadmin.ticket from /usr/lib/python2.4/site-packages/TracWebAdmin-0.1.2dev_r4429-py2.4.egg
2007-03-01 10:53:10,656 Trac[loader] DEBUG: Loading egg plugin webadmin.basics from /usr/lib/python2.4/site-packages/TracWebAdmin-0.1.2dev_r4429-py2.4.egg
2007-03-01 10:53:10,658 Trac[loader] DEBUG: Loading egg plugin webadmin.perm from /usr/lib/python2.4/site-packages/TracWebAdmin-0.1.2dev_r4429-py2.4.egg
2007-03-01 10:53:10,659 Trac[loader] DEBUG: Loading egg plugin webadmin.web_ui from /usr/lib/python2.4/site-packages/TracWebAdmin-0.1.2dev_r4429-py2.4.egg
2007-03-01 10:53:10,660 Trac[loader] DEBUG: Loading egg plugin webadmin.plugin from /usr/lib/python2.4/site-packages/TracWebAdmin-0.1.2dev_r4429-py2.4.egg
2007-03-01 10:53:10,668 Trac[loader] DEBUG: Loading egg plugin TracXMLRPC from /usr/lib/python2.4/site-packages/TracXMLRPC-0.1-py2.4.egg
2007-03-01 10:53:10,710 Trac[loader] DEBUG: Loading egg plugin tracreposearch from /usr/lib/python2.4/site-packages/tracreposearch-0.2-py2.4.egg
2007-03-01 10:53:10,713 Trac[loader] DEBUG: Loading egg plugin batchmod.web_ui from /usr/lib/python2.4/site-packages/BatchModify-0.1.0-py2.4.egg
2007-03-01 10:53:10,716 Trac[loader] DEBUG: Loading egg plugin acct_mgr.htfile from /usr/lib/python2.4/site-packages/TracAccountManager-0.1.3dev_r2063-py2.4.egg
2007-03-01 10:53:10,721 Trac[loader] DEBUG: Loading egg plugin acct_mgr.web_ui from /usr/lib/python2.4/site-packages/TracAccountManager-0.1.3dev_r2063-py2.4.egg
2007-03-01 10:53:10,755 Trac[loader] DEBUG: Loading egg plugin acct_mgr.admin from /usr/lib/python2.4/site-packages/TracAccountManager-0.1.3dev_r2063-py2.4.egg
2007-03-01 10:53:10,757 Trac[loader] DEBUG: Loading egg plugin acct_mgr.api from /usr/lib/python2.4/site-packages/TracAccountManager-0.1.3dev_r2063-py2.4.egg
2007-03-01 10:53:10,758 Trac[loader] DEBUG: Loading egg plugin acct_mgr.http from /usr/lib/python2.4/site-packages/TracAccountManager-0.1.3dev_r2063-py2.4.egg

comment:7 in reply to: ↑ 4 Changed 3 years ago by anonymous

Replying to Markus Tacker <m@tacker.org>:

With r4809 (0.10-stable) I get this error.

This happened because of a missing trac.repository_dir entry in the config file.

View

Add a comment

Modify Ticket

Change Properties
<Author field>
Action
as closed
The resolution will be deleted. Next status will be 'reopened'
to The owner will be changed from cboos. Next status will be 'closed'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.