[sword-devel] Patch for Sword crash with BibleTime on Windows

[Gary Holmlund]

Hi,

I just rediscovered a sword bug that I told you about over a year ago. 
For Bibletime it crashes the program when you try to use the Personal 
Commentary in write mode on Windows.

I upgraded to Sword 1.62 recently and this caused the bug to occur 
again. I had been using a patched version of 1.6.0 for our Windows 
build. Please incorporate the patch this time.

Thanks,

Gary Holmlund

--------------------------------------------------------------------------------
*Gary Holmlund* gary.holmlund at gmail.com 
<mailto:sword-devel%40crosswire.org?Subject=Re:%20Re%3A%20%5Bsword-devel%5D%20Patch%20for%20Sword%20crash%20with%20BibleTime%20on%20Windows&In-Reply-To=%3C4AF0ECE1.6090906%40gmail.com%3E>
/Tue Nov 3 19:54:25 MST 2009/

    * Previous message: [sword-devel] Python client
      <http://www.crosswire.org/pipermail/sword-devel/2009-November/033042.html>
    * Next message: [sword-devel] Python client
      <http://www.crosswire.org/pipermail/sword-devel/2009-November/033044.html>
    * *Messages sorted by:* [ date ]
      <http://www.crosswire.org/pipermail/sword-devel/2009-November/date.html#33043>
      [ thread ]
      <http://www.crosswire.org/pipermail/sword-devel/2009-November/thread.html#33043>
      [ subject ]
      <http://www.crosswire.org/pipermail/sword-devel/2009-November/subject.html#33043>
      [ author ]
      <http://www.crosswire.org/pipermail/sword-devel/2009-November/author.html#33043>


------------------------------------------------------------------------

Hi,

I am working on BibleTime for Windows and we found and fixed a crash in
the sword library. I have attached a patch for the HEAD of sword svn. We
are using the 1.60 version of sword.

The crash occurs when trying to save to a personal commentary for the
first time. Sword is looking for the "incfile" of the personal
commentary. The file does not exist yet.. Here is the call stack and
function at the crash point.

libsword.dll!sword::FileDesc::read(void * buf=0x013eaec0, long count=4)
Line 139
libsword.dll!sword::RawFiles::getNextFilename()  Line 194
libsword.dll!sword::RawFiles::setEntry(const char * inbuf=0x0253d050,
long len=3)  Line 130
bibletime.exe!CSwordModuleInfo::write(CSwordKey * key=0x02e6cd00, const
QString&  newText={...})  Line 705

long FileDesc::read(void *buf, long count) {
    return ::read(getFd(), buf, count);       // crash here
}

Since the file does not exist, getFd() returns a  fd of -1. The read
promptly crashes with the negative fd. Tracing the same problem in linux
shows the same -1 fd, but the read does not crash there. The crash seems
specific to the Visual Studio 2008 runtime libraries, but reading with a
negative fd value is clearly wrong.

The fix is simple. Test for the negative fd and return 0 from
fileDesc::read if fd is negative. Returning 0 is what is happening on linux.

Gary Holmlund



-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.crosswire.org/pipermail/sword-devel/attachments/20110211/83feb354/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: sword_filemgr.diff
Type: text/x-patch
Size: 375 bytes
Desc: not available
URL: <http://www.crosswire.org/pipermail/sword-devel/attachments/20110211/83feb354/attachment.bin>