Merge pull request #29 from diegosardina/patch-2

Martin Köditz · web-flow · commit b41ee37b0adc · 2022-02-17T11:12:01.000+01:00
Fixed two wrong statements. Set statement resource to NULL on reset.
diff --git a/ibase_query.c b/ibase_query.c
@@ -156,7 +156,7 @@ static void _php_ibase_free_result(zend_resource *rsrc) /* {{{ */
 		_php_ibase_free_xsqlda(ib_result->out_sqlda);
 		if (ib_result->stmt_res != NULL) {
 			zend_list_delete(ib_result->stmt_res);
-			ib_result->stmt_res == NULL;
+			ib_result->stmt_res = NULL;
 		}
 		efree(ib_result);
 	}
@@ -175,7 +175,7 @@ static void _php_ibase_free_query(ibase_query *ib_query) /* {{{ */
 	}
 	if (ib_query->stmt_res != NULL) {
 		zend_list_delete(ib_query->stmt_res);
-		ib_query->stmt_res == NULL;
+		ib_query->stmt_res = NULL;
 	}
 	if (ib_query->in_array) {
 		efree(ib_query->in_array);

Original file line number	Diff line number	Diff line change
`@@ -156,7 +156,7 @@ static void _php_ibase_free_result(zend_resource rsrc) / {{{ */`
`156`	`156`	`_php_ibase_free_xsqlda(ib_result->out_sqlda);`
`157`	`157`	`if (ib_result->stmt_res != NULL) {`
`158`	`158`	`zend_list_delete(ib_result->stmt_res);`
`159`		`- ib_result->stmt_res == NULL;`
	`159`	`+ ib_result->stmt_res = NULL;`
`160`	`160`	`}`
`161`	`161`	`efree(ib_result);`
`162`	`162`	`}`
`@@ -175,7 +175,7 @@ static void _php_ibase_free_query(ibase_query ib_query) / {{{ */`
`175`	`175`	`}`
`176`	`176`	`if (ib_query->stmt_res != NULL) {`
`177`	`177`	`zend_list_delete(ib_query->stmt_res);`
`178`		`- ib_query->stmt_res == NULL;`
	`178`	`+ ib_query->stmt_res = NULL;`
`179`	`179`	`}`
`180`	`180`	`if (ib_query->in_array) {`
`181`	`181`	`efree(ib_query->in_array);`