[sword-devel] Segmentation fault - was Re: SWORD 1.7.0RC3

Wed Sep 11 07:13:20 MST 2013

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

It's not a compiler bug.

Basically the C++ standard does not enforce an initialization order
for globals. So the compiler is free to initialize such stuff in any
order. I think your toolchain just happens to be lucky enough to do
that in an the order which doesn't yield in a crash.

I don't have the time to look this up in the C++ standards myself, but
you can google such stuff:

http://stackoverflow.com/questions/3746238/c-global-initialization-order-ignores-dependencies

Blessings,
Jaak

On 10.09.2013 18:12, Troy A. Griffitts wrote:
> Hey Jaak,
> 
> Thank, yeah, but why?  :)
> 
> My investigation turns up:
> 
> shared build works fine and valgrinds clean static build on g++
> (GCC) 4.8.1 (F19) fails static build on g++ (GCC) 4.4.7 (RHEL 6.4)
> works fine and valgrinds clean
> 
> In the error case, it seems that std::list's c-tor(iterator start, 
> iterator end) doesn't get called appropriately, though all the
> other file globals get initialized just fine.
> 
> I moved the global filter in the utility to a stack variable and
> it alleviates the problem, but still wonder why the problem in the
> first place.
> 
> Any advice would be appreciated.  If I'm doing something wrong, I'd
> like to change it, but right now I'm going with "compiler bug" :)
> 
> Troy
> 
> 
> 
> 
> 
> On 09/10/2013 08:20 AM, Jaak Ristioja wrote: I'm pretty sure that
> the crash is because the global
> 
> const StringList oValues(&choices[0], &choices[2]);
> 
> in ../src/modules/filters/utf8greekaccents.cpp:41 is not
> initialized when it is used in UTF8GreekAccents::UTF8GreekAccents()
> to initialize the global
> 
> UTF8GreekAccents greekAccentsFilter;
> 
> instance in imp2gbs.cpp:62.
> 
> 
> Blessings, Jaak
> 
> On 10.09.2013 02:02, Peter von Kaehne wrote:
>>>> I hate to bring this up, now, when we really all want 1.7
>>>> out, but current svn head produces a segmentation fault in
>>>> imp2gbs. The other utilities insofar as I tested them seem to
>>>> be fine.
>>>> 
>>>> Below the stacktrace. Tell me if you need more. I basically
>>>> use usrinst.sh produced binaries, with the only modification
>>>> that I changed the install directory to /usr/local/bin
>>>> 
>>>> 
>>>> 
>>>> peter at peter-ThinkPad-Twist:~/Source/sword$ gdb 
>>>> /usr/local/bin/imp2gbs GNU gdb (GDB) 7.5-ubuntu Copyright (C)
>>>> 2012 Free Software Foundation, Inc. License GPLv3+: GNU GPL
>>>> version 3 or later <http://gnu.org/licenses/gpl.html> This is
>>>> free software: you are free to change and redistribute it.
>>>> There is NO WARRANTY, to the extent permitted by law.  Type
>>>> "show copying" and "show warranty" for details. This GDB was
>>>> configured as "i686-linux-gnu". For bug reporting
>>>> instructions, please see: 
>>>> <http://www.gnu.org/software/gdb/bugs/>... Reading symbols
>>>> from /usr/local/bin/imp2gbs...done. (gdb) r Starting
>>>> program: /usr/local/bin/imp2gbs
>>>> 
>>>> Program received signal SIGSEGV, Segmentation fault.
>>>> 0x08050b7a in sword::SWBuf::c_str (this=0x8) at
>>>> ../include/swbuf.h:141 141 inline const char *c_str() const{
>>>> return buf; } (gdb) bt #0 0x08050b7a in sword::SWBuf::c_str
>>>> (this=0x8) at ../include/swbuf.h:141 #1  0x0805a790 in 
>>>> sword::SWOptionFilter::setOptionValue (this=0x8115b60 
>>>> <greekAccentsFilter>, ival=0x80c715b "On") at 
>>>> ../src/modules/filters/swoptfilter.cpp:51 #2  0x0805aca2 in 
>>>> sword::UTF8GreekAccents::UTF8GreekAccents (this=0x8115b60 
>>>> <greekAccentsFilter>, __in_chrg=<optimised out>, 
>>>> __vtt_parm=<optimised out>) at 
>>>> ../src/modules/filters/utf8greekaccents.cpp:43 #3  0x0805099d
>>>> in __static_initialization_and_destruction_0
>>>> (__initialize_p=1, __priority=65535) at imp2gbs.cpp:62 #4
>>>> 0x080509df in _GLOBAL__sub_I_imp2gbs.cpp(void) () at
>>>> imp2gbs.cpp:273 #5 0x080c6ae2 in __libc_csu_init () #6
>>>> 0xb77f046a in __libc_start_main () from
>>>> /lib/i386-linux-gnu/libc.so.6 #7 0x0804f931 in _start ()
>>>> (gdb) #0  0x08050b7a in sword::SWBuf::c_str (this=0x8) at
>>>> ../include/swbuf.h:141 #1  0x0805a790 in 
>>>> sword::SWOptionFilter::setOptionValue (this=0x8115b60 
>>>> <greekAccentsFilter>, ival=0x80c715b "On") at 
>>>> ../src/modules/filters/swoptfilter.cpp:51 #2  0x0805aca2 in 
>>>> sword::UTF8GreekAccents::UTF8GreekAccents (this=0x8115b60 
>>>> <greekAccentsFilter>, __in_chrg=<optimised out>, 
>>>> __vtt_parm=<optimised out>) at 
>>>> ../src/modules/filters/utf8greekaccents.cpp:43 #3  0x0805099d
>>>> in __static_initialization_and_destruction_0
>>>> (__initialize_p=1, __priority=65535) at imp2gbs.cpp:62 #4
>>>> 0x080509df in _GLOBAL__sub_I_imp2gbs.cpp(void) () at
>>>> imp2gbs.cpp:273 #5 0x080c6ae2 in __libc_csu_init () #6
>>>> 0xb77f046a in __libc_start_main () from
>>>> /lib/i386-linux-gnu/libc.so.6 #7 0x0804f931 in _start ()
>>>> (gdb) #0  0x08050b7a in sword::SWBuf::c_str (this=0x8) at
>>>> ../include/swbuf.h:141 #1  0x0805a790 in 
>>>> sword::SWOptionFilter::setOptionValue (this=0x8115b60 
>>>> <greekAccentsFilter>, ival=0x80c715b "On") at 
>>>> ../src/modules/filters/swoptfilter.cpp:51 #2  0x0805aca2 in 
>>>> sword::UTF8GreekAccents::UTF8GreekAccents (this=0x8115b60 
>>>> <greekAccentsFilter>, __in_chrg=<optimised out>, 
>>>> __vtt_parm=<optimised out>) at 
>>>> ../src/modules/filters/utf8greekaccents.cpp:43 #3  0x0805099d
>>>> in __static_initialization_and_destruction_0
>>>> (__initialize_p=1, __priority=65535) at imp2gbs.cpp:62 #4
>>>> 0x080509df in _GLOBAL__sub_I_imp2gbs.cpp(void) () at
>>>> imp2gbs.cpp:273 #5 0x080c6ae2 in __libc_csu_init () #6
>>>> 0xb77f046a in __libc_start_main () from
>>>> /lib/i386-linux-gnu/libc.so.6 #7 0x0804f931 in _start ()
>>>> (gdb)
>>>> 
>>>> 
>>>> 
>>>> _______________________________________________ sword-devel
>>>> mailing list: sword-devel at crosswire.org 
>>>> http://www.crosswire.org/mailman/listinfo/sword-devel
>>>> Instructions to unsubscribe/change your settings at above
>>>> page
>>>> 
>> 
>> _______________________________________________ sword-devel
>> mailing list: sword-devel at crosswire.org 
>> http://www.crosswire.org/mailman/listinfo/sword-devel 
>> Instructions to unsubscribe/change your settings at above page
> 
> 
> _______________________________________________ sword-devel mailing
> list: sword-devel at crosswire.org 
> http://www.crosswire.org/mailman/listinfo/sword-devel Instructions
> to unsubscribe/change your settings at above page

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.20 (GNU/Linux)

iQgcBAEBAgAGBQJSMHp7AAoJEEqsYmEt1rCOblpAAIfO/ypou3k0iGOmyo6xOay1
zGDLP0d8zgGJ4qIT48nsFTEgrSdLukeIhl1QO4hhCevw4IY/r7+pH1gQYlqrhquP
9esPk3SwUWn6oKVHD/7FTbDhap1EFtbsqyu273xV/08m9j/UROOSQAlWYPYmcsn1
5WLocCLrQO+RPNV0x2WuHkNXjm9R9ktqGrBDIQuCPArZehY6SVU5nsf+KdFMaNek
b1Esu+K8Qnyua77SI79K9Aj0s4HsSewrOYCe3TdiBqOKY9lJyHSdAuF3GiEIN2XZ
r/vegWmPRSuOXhLqFGXg5WDuGZcPCeeOCZy7AUmUCpDFRjtkcv+wH38vM9GLfdWb
TRlkiIoOIf7zA/IVwTNHqixQp4fmdyHftH3X9+9QKwvNFuVIG8VFhooIrfGIA7/q
YW1efT1FX6FImtJ+CPMLWvvAfQYKlV9NTQ4OBUR10QmCw32kvdvRipr/QevtCkxI
BIrVTjWjfnOi0coxbB9TTEWJqqG6zpIZGU08uHoVrH84kzf+7OQGoy52dITz72WR
wWwIOTuYWMXLOukym+4rviPRDnkFdG+7EeldN7+qQhm7/W8/aWy7vq8ucEjEwKCu
i+1ASmwlpSU2w1JH18DMcH13oXgClIN5IUuccsxim+BxLQHi+Z63A8p6JCxm96nH
h34/ZLVdb4MLpTY0gvXeIWB+a1e+lEoHQCc/6ukofnKNG2+BXICwwYvb6/FtxRKn
BqLnI58gmuzQwBemgsi+9naAOQSBsLHBtew7zrVAXe6aESkvC9tH8q+XefxWo/4H
c7FkqX0y+ojCrh5QHw8QvkWVV34MzCCMP10j+kBSQJpwBDwkooEgtBT/CUdyuark
9kR62Amd9ZgOGmQsh3qiyn/l1rX13vrsRWZD0luGpAIFC2t9bmGwnLrqf+LHD1ue
KG5qIaLzsoc15uuUJqEIN+6GBYlFQfrI6olFnEtlKfaElhoTfORZR+BBLwEmeIq2
eKOQALwr4coN/VIu0C+kekGaED3VlcxWvIXcCvv91xb6aJ2CvQz8+9wYffiqRQAZ
z4h/un7km8QvQPher/NxCIAcoPZhvh56uCCNYbGeVDvywZvuzCgoGEQYZw/jBY/9
lmqLd1beRif28MLVdiYbxt+BWEwSzsfVUDMkKClrRDIF4L5bKgBVpK8VxlhbQ4vj
ZGE6u57uFig79RQiQKDDxdG7nXoS7IU3lgT7Qcu8xbZqPeDJDYiic+V9hU5OS9Db
AuMlUMLPcVCEId7JdC8u8vcsiamU70gBnxunUZL4ZnWVY3ppzJeOBqxz+ESdt+QR
H3o3yptUnIYDFKoK3Xx+Ca0W3KrOgtswfr+bVXGQBlRvrqupVJO/7IOwbaarqdmo
2AkJrXAWMaYP1nUe0OQEjudamOhObPeVEAWQ8Yzm4Gslfg7aIgJ37HeeUFF7KY1T
fQEW+8CQ+dwO82tlJQwQ29iyEm1jqDVyNqVJNjAHQrhqANeFPZx0TkXU1kuDRbAI
U++TFLXf555ynpshtf4p6YqzN69Pa5SokRurHKlg7ShB3wl33i9HtdHZbv8nTaiT
aoMnBTHh3em1eca88FIp1FjzGiPpEb751WBc5IqPdg8QLLNXI53OX6GbUeZQ0B8K
qDZiUyC5qYYXtMK4oUYONRo3OSFN82O21XP2KiBZhAQ4wDe+XQILE1yXkud8GtWt
fjtITXARZx5lnqkg5G/o3AeWdIp4Z3H7s68b5UTlwfrvBS11L/39TAsuzmRpkLnz
1U3+qLy9luLQCA9ssfhhrLDxYyvmzd2HEghp9vDqNmUGFIOyfdMMy99fkV+gdddd
x54nlthSRr3+venk2r/9RyJAkmwrR7Ktq67B0jWiC7qthL8LVG2vP9sMzPl8jSS/
vDGUef6M60Z1hzFxS/uDERBpJAZkSjwnv7f9+o85xAYPLTG7o7txFTVV0gZpAJX0
6qVic+7M3+gLrqoYFTt1o1LuGQ0WnbRUzI6fm5/BEKu6VVSkZKX6nZ2s5UW8OZet
xN3kiWDsBSQlGkumrlvrdkoSJN6QtaMszXvMyRZn/XvjSTCG/MrRHH7BrMNhH4Hf
RmJ5ot7HBAlkbiSVpEx3S7+Ne0neBxoX6YrQm1USYtfZrBgMfxrzCEdN4dyRRZP2
eSEnxVfaxg5HYwYUMtwhDtGo6cThEoHIM6TXgbCuvejdhFtEjv1xOu8IBJ/SDaxH
f5m6xUsmVNswO3CiMbmLxLaMiUwMD8h3TC1ojO7huZqRvWH/jxzDWGH2Uj9yMCXI
OHh3DLi31iJRSnjRsPRmqlvqEqwHy4xeMSBAnEbWVM/AhbppMArYmH4TXlFOrff4
mUndN2XIh+W9zntL4LMHgWudWPWHBeMHojyHjX4HA0mC+Aljkzc/J+UcEipya9RV
saOJrh3Inzmb4EIxcqMGkWDcrUQpZS71LMM7Wx0LJfolDOh6EhdIg37dtxgw8+St
GhuGtHv0gRl+d/HS2tnwCej8rgO/8KA7Y8g9XYVA8iPmJH8qvImisioTOQsJm9mX
3qFzLbZrP/bPRpckGwACbwuYs1xBCyu7i2Fl/9ec3xX7lEvZkoT9vG0Vv4vMSgwW
ThhOFc4y5KKUfKdik8u25j72+Bv4Rp/NuqqeC/P11JfEVzWn30x8gSrFiqcORlQ6
waGNah30EatCy2ZZOUEqfH2MCb70AsIZSfmPjsRrtmB4MNuLcZaDOuXiey2XS3b/
swjcbbC0bQSas+7uOfqc
=I09B
-----END PGP SIGNATURE-----