fix: pgsqlSetNoticeCallback: ADDREF on non ref counted

outtersg · outtersg · commit d19e52e9376b · 2020-01-09T19:56:29.000+01:00
Core dumped when using a string callable.
diff --git a/ext/pdo_pgsql/pgsql_driver.c b/ext/pdo_pgsql/pgsql_driver.c
@@ -1186,7 +1186,7 @@ static PHP_METHOD(PDO, pgsqlSetNoticeCallback)
 			H->notice_callback = NULL;
 			RETURN_FALSE;
 		}
-		Z_ADDREF_P(&fc->fci.function_name);
+		Z_TRY_ADDREF_P(&fc->fci.function_name);
 		zend_string_release_ex(cbname, 0);
 
 		H->notice_callback = fc;

Original file line number	Diff line number	Diff line change
`@@ -1186,7 +1186,7 @@ static PHP_METHOD(PDO, pgsqlSetNoticeCallback)`
`1186`	`1186`	`H->notice_callback = NULL;`
`1187`	`1187`	`RETURN_FALSE;`
`1188`	`1188`	`}`
`1189`		`- Z_ADDREF_P(&fc->fci.function_name);`
	`1189`	`+ Z_TRY_ADDREF_P(&fc->fci.function_name);`
`1190`	`1190`	`zend_string_release_ex(cbname, 0);`
`1191`	`1191`
`1192`	`1192`	`H->notice_callback = fc;`