FreeTDS Development Group

["James K. Lowden" <jklowden AT freetds.org>]

Ivo van der Wijk wrote:
> I'm currently using a very recent nightly build (feb 5) of FreeTDS. We
> need the recently added   timeout code and it works (mostly) as a
> charm.

Excellent, thanks for the feedback.  

> However, in certain cases, when a timeout occurs (which I simply test
> by unplugging the utp), FreeTDS (and python, Zope which I use it for
> along with it) dies with the following assertion errror:
> 
> buffering.h:233: failed assertion `buffer_index_valid(buf, idx)'
> 
> Essentially, this happens in dblib.c function dbnextrow(), and I
> suspect because of the following.

You are correct.  Thanks too for the thorough analysis.  I looked at the
code and I think your fix is correct.  At least we know it's worse than
useless to try to read a buffered row based on an out-of-bounds index!  I
applied your suggestion, adding only a log message in case something in
the default case needs closer examination.  It might make tonight's
snapshot; we'll see.  

Please let us know what else you find.  

Regards, 

--jkl