commit invalidated repository

[Bastian Doetsch]

Am Samstag, den 13.09.2008, 16:46 +0200 schrieb Benoit Boissinot:

> On Sat, Sep 13, 2008 at 4:44 PM, Bastian Doetsch <bastian.doetsch at gmx.de> wrote:
> > Am Samstag, den 13.09.2008, 16:22 +0200 schrieb Benoit Boissinot:
> > > Have you used some extensions on this repo ? can you reproduce it ?
> > > (maybe the user had a \n at the end, so there was a double \n).
> >
> > I've not used any extension like transplant, strip, rebase on the repo. But
> > the username could be the problem - I tested some stuff about reading user
> > from config - and there it might have gotten it by reading the output of
> > showconfig. Filtering it shouldn't be a problem on our side, but I guess
> > this should not cause Mercurial to break, though :-(. I'll try to reproduce
> > it.
> >
> Yes, it shouldn't break. We should validate the input from the config
> file. How did you
> manage to put newlines in the username ?
> 
> regards,
> 
> Benoit
> 


I can reproduce it - and there's definitely a "\n" in the user name. I
got it into the username by using the java ProcessBuilder class and then
calling it. I couldn't do it via the command line, so I guess it's only
possible when directly starting a mercurial process from java - they
probably use some native C-Code to pass the parameters.

Bastian
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.mercurial-scm.org/pipermail/mercurial-devel/attachments/20080913/342ee0a2/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: Dies ist ein digital signierter Nachrichtenteil
URL: <http://lists.mercurial-scm.org/pipermail/mercurial-devel/attachments/20080913/342ee0a2/attachment.asc>