Bug 112930

Summary: vim: seqfaults while svn commits
Product: [openSUSE] SUSE LINUX 10.0 Reporter: Danny Al-Gaaf <dalgaaf>
Component: ConsoleAppsAssignee: Mads Martin Joergensen <mmj>
Status: RESOLVED FIXED QA Contact: E-mail List <qa-bugs>
Severity: Major    
Priority: P5 - None CC: bjacke, chref, dmueller, forgotten_OS1JNCFbCX, kukuk, lmuelle, mc, mge, moussa.sagna, mvidner, suse-beta, thomas.schraitle
Version: Beta 3   
Target Milestone: ---   
Hardware: Other   
OS: All   
Whiteboard:
Found By: Other Services Priority:
Business Priority: Blocker: ---
Marketing QA Status: --- IT Deployment: ---
Attachments: Newly created ~/.viminfo file after trying to run 'svn co'
and again valgrind with debuginfo....
suggested fix
same patch for 9.1

Description Danny Al-Gaaf 2005-08-25 10:28:57 UTC
vim crashes from time to time while svn commit with errormessage: "... revieved 
Signal 11 ...". In this case you must reset the console and if you retry the 
commit it crashes again and again and ...

To get vim back you must remove .viminfo from your home. This is more than ugly 
behavior!
Comment 1 Hendrik Vogelsang 2005-08-25 11:25:09 UTC
The viminfo file is just corrupted. Nothing we can do about that. See

Bug #51666
Bug #53703
Comment 2 Danny Al-Gaaf 2005-08-25 11:34:08 UTC
What kind of solution is 'remove .viminfo'. If vim creates corrupted viminfo 
file this must be fixed!
Comment 3 Hendrik Vogelsang 2005-08-25 11:47:09 UTC
good luck
Comment 4 Mads Martin Joergensen 2005-08-29 09:09:47 UTC
Was the .viminfo created by earlier distributions, or was it created by the vim
in the beta (which one) you were running?
Comment 5 Danny Al-Gaaf 2005-08-29 12:03:24 UTC
no, this was always from a fresh installation. No changes or updates, clean new 
user home. This happend on 9.2, 10.0 Preview and Beta 4.
Comment 6 Mads Martin Joergensen 2005-08-29 15:11:24 UTC
Can I see your .vimrc?
Comment 7 Danny Al-Gaaf 2005-08-29 15:44:24 UTC
I don't have a .vimrc in my home. I changed only in /etc/vimrc "syntax on" to 
get syntax highlighting. If you need this one, let me know.
Comment 8 Mads Martin Joergensen 2005-08-30 08:21:23 UTC
How does subversion invoke vim?
Comment 9 Mads Martin Joergensen 2005-08-30 08:37:55 UTC
Can you try:

1) add "set viminfo+=c" to /etc/vimrc
2) rm ~/.viminfo

And now reproduce.
Comment 10 Danny Al-Gaaf 2005-08-30 08:53:58 UTC
I will try this. I have stored 2 invalid .viminfo here saved. I mail you the 
files directly.
Comment 11 Mads Martin Joergensen 2005-09-01 09:43:58 UTC
Any news? You have to start with a fresh .vimrc after adding the above.
Comment 12 Danny Al-Gaaf 2005-09-01 17:26:22 UTC
Sorry for the lack ... I test this tomorrow again.
Comment 13 Mads Martin Joergensen 2005-09-09 10:47:18 UTC
Are you able to reproduce like said in comment #9?
Comment 14 Danny Al-Gaaf 2005-09-09 19:20:10 UTC
I added this and I could not reproduce this crash currently, but I did currently 
not so many svn commits as before. I must test more with svn to check if this 
happen again.
Comment 15 Mads Martin Joergensen 2005-09-19 07:41:11 UTC
Any news here?
Comment 16 Mads Martin Joergensen 2005-09-21 08:10:55 UTC
*** Bug 118179 has been marked as a duplicate of this bug. ***
Comment 17 Mads Martin Joergensen 2005-09-21 08:19:31 UTC
Can people who can reproduce after taking the steps in comment #9 please post
their viminfo settings from vimrc?
Comment 18 Thomas Schraitle 2005-09-22 08:28:20 UTC
Created attachment 50627 [details]
Newly created  ~/.viminfo file after trying to run 'svn co'

After trying comment #9, it seems the problem has disappeared. 
If you need more info, tell me. :)
Comment 19 Christian Zoz 2005-09-29 10:13:10 UTC
I had these crashes frequently. Always removed .viminfo to proceed.

Now i added set viminfo+=c to my .vimrc.
Let's see if it happens again.
Comment 20 Mads Martin Joergensen 2005-09-29 10:25:58 UTC
If people can reproduce, please try with:

~mmj/Export/vim-10.0-i386 packages
Comment 21 Mads Martin Joergensen 2005-10-03 12:35:01 UTC
So apparantly these packages seems more stable, could someone else please
try them out?
Comment 22 Dirk Mueller 2005-10-12 09:10:47 UTC
the ~mmj package doesn't seem to segfault here during svn commit anymore. but 
it only happens rather rarely anyway. but if it happens, then it usually 
happens 5-10 times in a row.  
 
 
Comment 23 Christian Zoz 2005-11-04 08:50:38 UTC
viminfo+=c did not help. I remove that again and update to vim-6.3.84-2.1.
Comment 24 Christian Zoz 2005-11-04 09:33:03 UTC
Immediately after vim update with old .viminfo it happened again. I remove .viminfo now as always.
Comment 25 Dirk Mueller 2005-12-02 11:18:20 UTC
still happens..

=7457== Conditional jump or move depends on uninitialised value(s)
==7457==    at 0x80D3A2E: ??? (mbyte.c:1570)
==7457==    by 0x80BF185: ??? (message.c:2840)
==7457==    by 0x80B98C9: ??? (memline.c:3733)
==7457==    by 0x80B9FA2: ??? (memline.c:495)
==7457==    by 0x809962D: ??? (fileio.c:634)
==7457==    by 0x804FD13: ??? (buffer.c:131)
==7457==    by 0x80AFD5D: ??? (main.c:1792)
==7457==    by 0x40A6E97: __libc_start_main (in /lib/libc-2.3.6.so)
==7457==
==7457== Conditional jump or move depends on uninitialised value(s)
==7457==    at 0x80D3A2E: ??? (mbyte.c:1570)
==7457==    by 0x80BF038: ??? (message.c:2910)
==7457==    by 0x80B98C9: ??? (memline.c:3733)
==7457==    by 0x80B9FA2: ??? (memline.c:495)
==7457==    by 0x809962D: ??? (fileio.c:634)
==7457==    by 0x804FD13: ??? (buffer.c:131)
==7457==    by 0x80AFD5D: ??? (main.c:1792)
==7457==    by 0x40A6E97: __libc_start_main (in /lib/libc-2.3.6.so)
==7457==
==7457== Invalid read of size 1
==7457==    at 0x80D3DEF: ??? (mbyte.c:2349)
==7457==    by 0x80D500D: ??? (mbyte.c:2551)
==7457==    by 0x80D5075: ??? (mbyte.c:2531)
==7457==    by 0x80E0467: ??? (normal.c:1248)
==7457==    by 0x80AE956: ??? (main.c:2183)
==7457==    by 0x80B00DD: ??? (main.c:2001)
==7457==    by 0x40A6E97: __libc_start_main (in /lib/libc-2.3.6.so)
==7457==  Address 0x4510BD8 is 8 bytes after a block of size 4,096 alloc'd
==7457==    at 0x401B142: malloc (in /usr/lib/valgrind/x86-linux/vgpreload_memcheck.so)
==7457==    by 0x80CD0B7: ??? (misc2.c:843)
==7457==    by 0x80CD359: ??? (misc2.c:742)
==7457==    by 0x80B4F17: ??? (memfile.c:939)
==7457==    by 0x80B5445: ??? (memfile.c:382)
==7457==    by 0x80B668F: ??? (memline.c:2937)
==7457==    by 0x80BA550: ??? (memline.c:329)
==7457==    by 0x804FC04: ??? (buffer.c:86)
==7457==    by 0x80AFD5D: ??? (main.c:1792)
==7457==    by 0x40A6E97: __libc_start_main (in /lib/libc-2.3.6.so)
==7457==
Comment 26 Juergen Weigert 2006-01-12 17:04:08 UTC
Created attachment 63151 [details]
and again valgrind with debuginfo....
Comment 27 Mads Martin Joergensen 2006-01-27 13:58:46 UTC
Any new debuginfo here from beta2? I've compiled it with stack-protector-all.
Comment 28 Hendrik Vogelsang 2006-02-06 15:20:02 UTC
*** Bug 148397 has been marked as a duplicate of this bug. ***
Comment 29 Hendrik Vogelsang 2006-02-23 10:29:21 UTC
*** Bug 149046 has been marked as a duplicate of this bug. ***
Comment 30 Juergen Weigert 2006-03-17 19:20:45 UTC
I just saw it happen on a 9.1 box:
svn runs 'vim -c "set tw=72 et" svn-commit.tmp
User presses 'i' to enter insert mode, and vim dies.

The user was typing very quickly and fumbled a few times, producing strange utf-8 digraphs which he fixed by backspacing. Then he managed to enter the 
svn command and vim crashed.

Not reproducable.
Comment 31 Guenther Deschner 2006-03-17 20:50:32 UTC
I have this segfault daily (sometimes even several times a day) when commiting to svn.samba.org. At least on my 10.0 box a simple rm ~/.viminfo* allows to repeat the crashed commit sucessfully.
Comment 32 Juergen Weigert 2006-03-17 22:35:15 UTC
I can reproduce, and know what is going on.
Tricky. 

A workaround would be:
end your commit message with the cursor in the first column :-)

don't ask.
Comment 33 Juergen Weigert 2006-03-18 00:21:29 UTC
Created attachment 73785 [details]
suggested fix

This patch should fix the problem.
Submitted to stable. 

I am unsure if vim7 needs a similar fix or not.
Please investigate.
Comment 34 Juergen Weigert 2006-03-18 00:22:18 UTC
Created attachment 73786 [details]
same patch for 9.1
Comment 35 Mads Martin Joergensen 2006-03-18 22:38:06 UTC
Thorsten, is this something for SLES9-SP4?
Comment 36 Hendrik Vogelsang 2006-03-20 14:18:42 UTC
fixed in stable