Patch to fix problems on SQLite Blob Fields

dopidaniel · Post by **dopidaniel** » 29.11.2010, 05:09

Hello everyone,

In order to migrate to Firebird 2.5, I decided to test Zeos 7.0.0 again

My application uses SQLite and Firebird. I noticed that a blob field in my table in SQLite was not correctly read, while the same field, from the same table, but in Firebird (with the same structure and data), was accessed correctly.

Investigating the problem, I noticed that when the field immediately prior to the Blob field is null, the contents of the Blob is despised ...

Debugging the source code, I noticed that in some Units within "dbc" directory, when the Blob contents is readed, the internal variable "LastWasNull" is not being set ... so this cause the problem ...

The patch attached contains modifications to some Units on DBC, adjusting the content of "LastWasNull" on "GetBlob"

Please consider whether the changes are correct, and if it can be integrated with SVN sources

Thanks

zabukowski · Post by **zabukowski** » 04.12.2010, 12:56

+1

Zabukowski

Post by **mdaems** » 05.12.2010, 23:31

Doesn't seem to crash the test suite.
SVN 863 (Testing branch, to be merged into 6.6-patches.

Mark

dopidaniel · Post by **dopidaniel** » 09.12.2010, 11:17

Hi Mark,

Thanks for reviewing my patch ...

I am a bit confused on SVN use ... These changes I made were about the sources of the folder "trunk"

svn://zeos.firmos.at/zeos/trunk

Is that correct? The changes will be applied to this folder?

Post by **mdaems** » 15.12.2010, 21:50

dopidaniel,

Yes, the patch will be merged to trunk at the next merge operation. Which means after at least a week and then when time permits...

Afterwards the patch will also be merged into the 6.6-patches branch (maintenance releases).

Meanwhile you can find the already patched version on svn://zeos.firmos.at/zeos/branches/testing , but this branch should be considered experimental.

Mark