tony2001 Thu Dec 1 14:59:40 2005 EDT Modified files: /php-src/ext/pdo pdo_stmt.c Log: fix #35508 (PDO fails when unknown fetch mode specified) http://cvs.php.net/diff.php/php-src/ext/pdo/pdo_stmt.c?r1=1.132&r2=1.133&ty=u Index: php-src/ext/pdo/pdo_stmt.c diff -u php-src/ext/pdo/pdo_stmt.c:1.132 php-src/ext/pdo/pdo_stmt.c:1.133 --- php-src/ext/pdo/pdo_stmt.c:1.132 Sun Nov 27 15:56:46 2005 +++ php-src/ext/pdo/pdo_stmt.c Thu Dec 1 14:59:36 2005 @@ -18,7 +18,7 @@ +----------------------------------------------------------------------+ */ -/* $Id: pdo_stmt.c,v 1.132 2005/11/27 20:56:46 tony2001 Exp $ */ +/* $Id: pdo_stmt.c,v 1.133 2005/12/01 19:59:36 tony2001 Exp $ */ /* The PDO Statement Handle Class */ @@ -1068,6 +1068,11 @@ static int pdo_stmt_verify_mode(pdo_stmt_t *stmt, int mode, int fetch_all TSRMLS_DC) /* {{{ */ { int flags = mode & PDO_FETCH_FLAGS; + + if (mode < 0 || mode > PDO_FETCH__MAX) { + pdo_raise_impl_error(stmt->dbh, stmt, "HY000", "invalid fetch mode" TSRMLS_CC); + return 0; + } mode = mode & ~PDO_FETCH_FLAGS; @@ -1657,10 +1662,12 @@ mode = Z_LVAL_PP(args[skip]); if (!pdo_stmt_verify_mode(stmt, mode, 0 TSRMLS_CC)) { + efree(args); return FAILURE; } switch (mode & ~PDO_FETCH_FLAGS) { + case PDO_FETCH_USE_DEFAULT: case PDO_FETCH_LAZY: case PDO_FETCH_ASSOC: case PDO_FETCH_NUM: @@ -1740,6 +1747,7 @@ } else { pdo_raise_impl_error(stmt->dbh, stmt, "22003", "mode is out of range" TSRMLS_CC); } + efree(args); return FAILURE; }
-- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php