search

FirebirdSQL / firebird

Firebird server, client and tools
https://www.firebirdsql.org/
1.26k stars 217 forks source link

Array with BOOLEAN. Bug in sdl_desc [sdl.cpp] [CORE3412] #3775

Open firebird-automations opened 13 years ago

firebird-automations commented 13 years ago

Submitted by: @ibprovider

Hi (#⁠3)

Please add the support for blr_sql_bool into sdl_desc [sdl.cpp]

Thanks, Kovalenko Dmitry

Commits: FirebirdSQL/firebird@b8d5f6f24a7278e6ec11b59edebc5d0a6c5a3637

firebird-automations commented 13 years ago

Commented by: @dyemanov

It would be faster to resolve it you could contribute the patch :-)

firebird-automations commented 13 years ago

Commented by: @ibprovider

sdl.cpp

case blr\_sql\_time:
    desc\-\>dsc\_dtype = dtype\_sql\_time;
    desc\-\>dsc\_length = sizeof\(ULONG\);
    break;

//-------------------- [begin patch] case blr_bool: desc->dsc_dtype = dtype_boolean; desc->dsc_length = sizeof(char); // <---- This is right? break; //-------------------- [end patch] default: fb_assert_continue(FALSE); return NULL; }

switch \(desc\-\>dsc\_dtype\)
\{
case dtype\_short:
firebird-automations commented 13 years ago

Commented by: @dyemanov

Done, please test.

firebird-automations commented 13 years ago

Commented by: @ibprovider

tested. fix works without problem :)