From:             ken-phpbugs at norganna dot com
Operating system: Linux
PHP version:      5.5.3
Package:          MSSQL related
Bug Type:         Bug
Bug description:mssql_fetch_batch does not fetch next batch

Description:
------------
When sending batch requests to the mssql server, the mssql result is having

dbcancel() called prematurely on the handle (via the _free_mssql_result
function).

Because we are processing batches of the result set we need the query to
stay open 
until we are finished fetching all the rows (not just the first batch).

In a non-batch scenario, it is fine to cancel the query after the first
batch is 
called, because all rows have been fetched.

I have attached a patch, which "fixes" the problem, but probably not in the
right 
way. I have no clue what the right way actually would be or what the
repercussions 
of applying the patch are. It is meant for illustrative purposes only.

Test script:
---------------
<?php
// Connect to MSSQL and select the database
$link = mssql_connect('hostpath', 'user', 'pass');
mssql_select_db('db', $link);

// Send a query to a table that has 12 rows
print "Given a table with 12 rows:\n";
$result = mssql_query('SELECT * FROM tablename', $link, 5);

do {
    while ($row = mssql_fetch_assoc($result)) {
        print "Row found\n";
    }

    $remain = mssql_fetch_batch($res);
    print "Remainder $remain\n";
} while ($remain > 0);


Expected result:
----------------
Given a table with 12 rows:
Row found
Row found
Row found
Row found
Row found
Remainder 5
Row found
Row found
Row found
Row found
Row found
Remainder 2
Row found
Row found
Remainder 0


Actual result:
--------------
Given a table with 12 rows:
Row found
Row found
Row found
Row found
Row found
Remainder 1
Row found
Remainder 0


-- 
Edit bug report at https://bugs.php.net/bug.php?id=65693&edit=1
-- 
Try a snapshot (PHP 5.4):   
https://bugs.php.net/fix.php?id=65693&r=trysnapshot54
Try a snapshot (PHP 5.3):   
https://bugs.php.net/fix.php?id=65693&r=trysnapshot53
Try a snapshot (trunk):     
https://bugs.php.net/fix.php?id=65693&r=trysnapshottrunk
Fixed in SVN:               https://bugs.php.net/fix.php?id=65693&r=fixed
Fixed in release:           https://bugs.php.net/fix.php?id=65693&r=alreadyfixed
Need backtrace:             https://bugs.php.net/fix.php?id=65693&r=needtrace
Need Reproduce Script:      https://bugs.php.net/fix.php?id=65693&r=needscript
Try newer version:          https://bugs.php.net/fix.php?id=65693&r=oldversion
Not developer issue:        https://bugs.php.net/fix.php?id=65693&r=support
Expected behavior:          https://bugs.php.net/fix.php?id=65693&r=notwrong
Not enough info:            
https://bugs.php.net/fix.php?id=65693&r=notenoughinfo
Submitted twice:            
https://bugs.php.net/fix.php?id=65693&r=submittedtwice
register_globals:           https://bugs.php.net/fix.php?id=65693&r=globals
PHP 4 support discontinued: https://bugs.php.net/fix.php?id=65693&r=php4
Daylight Savings:           https://bugs.php.net/fix.php?id=65693&r=dst
IIS Stability:              https://bugs.php.net/fix.php?id=65693&r=isapi
Install GNU Sed:            https://bugs.php.net/fix.php?id=65693&r=gnused
Floating point limitations: https://bugs.php.net/fix.php?id=65693&r=float
No Zend Extensions:         https://bugs.php.net/fix.php?id=65693&r=nozend
MySQL Configuration Error:  https://bugs.php.net/fix.php?id=65693&r=mysqlcfg

Reply via email to