Re: SQLSetStmtAttr()+SQL_ATTR_ROWS_FETCHED_PTR

On Wed, Feb 4, 2009 at 3:32 AM, Nick Gorham <[email protected]> wrote:
> George Neill wrote:
>
>> Nick,
>>
>> On Tue, Feb 3, 2009 at 5:10 AM, Nick Gorham <[email protected]> wrote:
>>
>>>
>>> George Neill wrote:
>>>
>>>
>>>>
>>>> On Tue, Feb 3, 2009 at 2:11 AM, Nick Gorham <[email protected]> wrote:
>>>>
>>>>
>>>>>
>>>>> George Neill wrote:
>>>>>
>>>>>
>>>>>>
>>>>>> Hi All,
>>>>>>
>>>>>> I am having a little bit of confusion with SQLSetStmtAttr() and
>>>>>> SQL_ATTR_ROWS_FETCHED_PTR.  From this documentation,
>>>>>> http://www.unixodbc.org/doc/ODBC64.html, it states the Value/ValuePtr
>>>>>> should be of type SQLULEN ... but the actual code seems to use
>>>>>> SQLUINTEGER.  Is this a bug in the docs or the code?
>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>>>
>>>>> I would guess its a problem in the code, which bit of code in
>>>>> particular?
>>>>> so
>>>>> I can fix it.
>>>>>
>>>>>
>>>>
>>>> SQLSetStmtAttr.c:753
>>>>
>>>> 2.2.15pre
>>>>
>>>> Thanks,
>>>> George
>>>>
>>>>
>>>
>>> Ok, try the 2.2.15pre now.
>>>
>>
>> diff -r unixODBC-2.2.15pre.old/DriverManager/SQLSetStmtAttr.c
>> unixODBC-2.2.15pre.new/DriverManager/SQLSetStmtAttr.c
>> 756c756
>> <         statement -> row_ct_ptr = (SQLUINTEGER*) value;
>> ---
>>
>>>
>>>       statement -> row_ct_ptr = (SQLULEN*) value;
>>>
>>
>>
>> All looks good, except I think you may have missed this one!   I will
>> give it a test drive here in a bit.
>>
>> Thanks
>> George.
>> _______________________________________________
>> unixODBC-dev mailing list
>> [email protected]
>> http://mailman.unixodbc.org/mailman/listinfo/unixodbc-dev
>>
>>
>>
>
> Ok, that bits done now.

in SQLSetStmtAttr.c:694
        statement -> fetch_bm_ptr = (SQLUINTEGER*) value;

Also should be changed to SQLULEN?

Later,
George.
_______________________________________________
unixODBC-dev mailing list
[email protected]
http://mailman.unixodbc.org/mailman/listinfo/unixodbc-dev