Fix #81509 pg_end_copy still expects a resource

mbeccati · mbeccati · commit 958daa6529c3 · 2021-10-05T18:13:50.000+02:00
diff --git a/NEWS b/NEWS
@@ -17,6 +17,9 @@ PHP                                                                        NEWS
 - PCRE:
   . Unfixed bug #81424 (PCRE2 10.35 JIT performance regression). (cmb)
 
+- PgSQL:
+  . Fixed bug #81509 (pg_end_copy still expects a resource). (Matteo)
+
 - Standard:
   . Fixed bug #81491 (Incorrectly using libsodium for argon2 hashing).
     (Dan Pock)
diff --git a/ext/pgsql/pgsql.c b/ext/pgsql/pgsql.c
@@ -3007,7 +3007,7 @@ PHP_FUNCTION(pg_end_copy)
 	int result = 0;
 	pgsql_link_handle *link;
 
-	if (zend_parse_parameters(ZEND_NUM_ARGS(), "|r!", &pgsql_link, pgsql_link_ce) == FAILURE) {
+	if (zend_parse_parameters(ZEND_NUM_ARGS(), "|O!", &pgsql_link, pgsql_link_ce) == FAILURE) {
 		RETURN_THROWS();
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -3007,7 +3007,7 @@ PHP_FUNCTION(pg_end_copy)`
`3007`	`3007`	`int result = 0;`
`3008`	`3008`	`pgsql_link_handle *link;`
`3009`	`3009`
`3010`		`- if (zend_parse_parameters(ZEND_NUM_ARGS(), "\|r!", &pgsql_link, pgsql_link_ce) == FAILURE) {`
	`3010`	`+ if (zend_parse_parameters(ZEND_NUM_ARGS(), "\|O!", &pgsql_link, pgsql_link_ce) == FAILURE) {`
`3011`	`3011`	`RETURN_THROWS();`
`3012`	`3012`	`}`
`3013`	`3013`