From 9b44d687df22761433496c72bad56b8bce3d1978 Mon Sep 17 00:00:00 2001 From: Pierre Joye Date: Sat, 17 May 2014 05:41:21 +0200 Subject: [PATCH] fix zend_hash_get_current_data_ptr_ex call --- ext/pdo/pdo_sql_parser.re | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/ext/pdo/pdo_sql_parser.re b/ext/pdo/pdo_sql_parser.re index 48d80e2684..a1742e397a 100644 --- a/ext/pdo/pdo_sql_parser.re +++ b/ext/pdo/pdo_sql_parser.re @@ -422,8 +422,9 @@ int old_pdo_parse_params(pdo_stmt_t *stmt, char *inquery, int inquery_len, char padding = 3; } if(params) { + HashPosition *param_pos; zend_hash_internal_pointer_reset(params); - while (SUCCESS == zend_hash_get_current_data(params, (void**)¶m)) { + while ((param == zend_hash_get_current_data_ptr_ex(params, ¶m_pos)) != NULL) { if(param->parameter) { convert_to_string(param->parameter); /* accommodate a string that needs to be fully quoted @@ -455,9 +456,9 @@ int old_pdo_parse_params(pdo_stmt_t *stmt, char *inquery, int inquery_len, char } /* lookup bind first via hash and then index */ /* stupid keys need to be null-terminated, even though we know their length */ - if((SUCCESS == zend_hash_find(params, s.tok, s.cur-s.tok,(void **)¶m)) + if((NULL != zend_hash_find(params, s.tok, s.cur-s.tok,(void **)¶m)) || - (SUCCESS == zend_hash_index_find(params, bindno, (void **)¶m))) + (NULL != zend_hash_index_find(params, bindno, (void **)¶m))) { char *quotedstr; int quotedstrlen; @@ -494,7 +495,7 @@ int old_pdo_parse_params(pdo_stmt_t *stmt, char *inquery, int inquery_len, char return (int) (s.cur - inquery); } /* lookup bind by index */ - if(SUCCESS == zend_hash_index_find(params, bindno, (void **)¶m)) + if(NULL != zend_hash_index_find(params, bindno, (void **)¶m)) { char *quotedstr; int quotedstrlen; -- 2.50.1