inconsistent filename checking between 'add' and 'smart_add'

[Michael Hudson-Doyle]

This doesn't seem quite right:

mwh@mithril-inside:tmp$ mkdir blah
mwh@mithril-inside:tmp$ cd blah
mwh@mithril-inside:blah$ bzr init
mwh@mithril-inside:blah$ touch a..b
mwh@mithril-inside:blah$ touch c..d
mwh@mithril-inside:blah$ PYTHONPATH=~/src/bzr/bzr.dev python2.4
Python 2.4.4 (#2, Apr 12 2007, 21:03:11)
[GCC 4.1.2 (Ubuntu 4.1.2-0ubuntu4)] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import bzrlib.workingtree
>>> wt = bzrlib.workingtree.WorkingTree.open('.')
>>> wt.smart_add(['a..b'])
([u'a..b'], {})
>>> wt.add(['c..d'])
Traceback (most recent call last):
  File "<stdin>", line 1, in ?
  File "/home/mwh/src/bzr/bzr.dev/bzrlib/mutabletree.py", line 50, in tree_write_locked
    return unbound(self, *args, **kwargs)
  File "/home/mwh/src/bzr/bzr.dev/bzrlib/mutabletree.py", line 126, in add
    self._add(files, ids, kinds)
  File "/home/mwh/src/bzr/bzr.dev/bzrlib/mutabletree.py", line 50, in tree_write_locked
    return unbound(self, *args, **kwargs)
  File "/home/mwh/src/bzr/bzr.dev/bzrlib/workingtree_4.py", line 162, in _add
    assert '..' not in f
AssertionError
>>>

I don't know what the correct behaviour _is_, but I think these two calls should probably succeed on the same filenames...

[John A Meinel (jameinel)]

Well, the check is actually meant to catch cases where someone is trying to do:

tree.add('foo/../bar')

You certainly should be able to add a file named "foo..bar".

So a better check might be "assert /../ not in f"

At least, by that point the path should be properly sanitized such that there are only forward slashes, and not backslashes. Otherwise a lot of the rest of the code would break anyway.-

I believe smart_add() is safe about it, because of how it recurses through the tree and normalizes the user's input. WT.add() is not safe about this, because it expects that the caller has already done it.

If you *really* need to work around it, notice that these are plain 'assert' cases, so running in python -O mode will skip the check.

[Achim Bohnet (allee)]

This bug was marked as dup. of #145207 a showfoto crash. That's wrong.
Please use the correct bug number.
Achim

[Martin Pool (mbp)]

This assertion is no longer present