[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-edu-devel
Subject: Re: [kde-edu]: kalzium molecular viewer segfaults
From: Saro Engels <ps_ml () gmx ! de>
Date: 2007-11-18 5:45:22
Message-ID: 473FD172.7060501 () gmx ! de
[Download RAW message or body]
Benoit Jacob schrieb:
>> but a full backtrace would definitely help making sure. Apparently this
>> OpenBabel calls another OpenBabel function, "SetOutFormat", which
>> segfaults. By the way, looking at the above line, I realize that it feels
>> strange to have "inFormat" passed twice so perhaps there's also something
>> to fix here. I really don't know.
The problem here lies really within the two times inFormat: in the line
above inFormat is defined as
inFormat = Conv.FormatFromExt( QFile::encodeName(filename) );
which results although filename isn't empty in a zero pointer inFormat.
The Problem within openbabel is now that it checks for it in SetInFormat
but not in SetOutFormat (which leads to the segfault when trying to call
inFormat->Flags()).
So right now it seems to be a problem of the openbabel lib.
>
> I think: why the HECK are we setting at all the Out format? We only read
> molecules in Kalzium, we don't write to them (in kde 4.0) so we could (and
> probably should) only set the In format. Since your crash occurs in
> SetOutFormat, this would also quite possibly fix it!!
>
> So please try this: on line 51 of
> kalzium/compoundviewer/openbabel2wrapper.cpp, please apply this diff:
> - Conv.SetInAndOutFormats( inFormat,inFormat );
> + Conv.SetInFormat(inFormat);
No, that doesn't work -
warning: HEAP[kalzium.exe]:
warning: Invalid Address specified to RtlFreeHeap( 003D0000, 6FDCC4B8 )
Program received signal SIGTRAP, Trace/breakpoint trap.
0x7c911231 in ntdll!DbgUiConnectToDbg () from ntdll.dll
(gdb) quit
greetings
SaroEngels
_______________________________________________
kde-edu mailing list
kde-edu@mail.kde.org
https://mail.kde.org/mailman/listinfo/kde-edu
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic