Use ZVAL_STR instead of ZVAL_STRINGL when copying a zend_string

Girgias · Girgias · commit 4b60eace1eef · 2021-03-22T04:23:31.000Z
diff --git a/ext/pgsql/pgsql.c b/ext/pgsql/pgsql.c
@@ -4696,7 +4696,7 @@ PHP_PGSQL_API int php_pgsql_convert(PGconn *pg_link, const char *table_name, con
 								err = 1;
 							}
 							else {
-								ZVAL_STRINGL(&new_val, Z_STRVAL_P(val), Z_STRLEN_P(val));
+								ZVAL_STR(&new_val, Z_STR_P(val));
 							}
 #undef REGEX0
 						}
@@ -4746,7 +4746,7 @@ PHP_PGSQL_API int php_pgsql_convert(PGconn *pg_link, const char *table_name, con
 								}
 							}
 							else {
-								ZVAL_STRING(&new_val, Z_STRVAL_P(val));
+								ZVAL_STR(&new_val, Z_STR_P(val));
 							}
 #undef REGEX0
 #undef REGEX1
@@ -4850,7 +4850,7 @@ PHP_PGSQL_API int php_pgsql_convert(PGconn *pg_link, const char *table_name, con
 								err = 1;
 							}
 							else {
-								ZVAL_STRINGL(&new_val, Z_STRVAL_P(val), Z_STRLEN_P(val));
+								ZVAL_STR(&new_val, Z_STR_P(val));
 								convert_to_long(&new_val);
 							}
 						}
@@ -4896,7 +4896,7 @@ PHP_PGSQL_API int php_pgsql_convert(PGconn *pg_link, const char *table_name, con
 								err = 1;
 							}
 							else {
-								ZVAL_STRINGL(&new_val, Z_STRVAL_P(val), Z_STRLEN_P(val));
+								ZVAL_STR(&new_val, Z_STR_P(val));
 								php_pgsql_add_quotes(&new_val, 1);
 							}
 #undef REGEX0
@@ -4965,7 +4965,7 @@ PHP_PGSQL_API int php_pgsql_convert(PGconn *pg_link, const char *table_name, con
 								err = 1;
 							}
 							else {
-								ZVAL_STRINGL(&new_val, Z_STRVAL_P(val), Z_STRLEN_P(val));
+								ZVAL_STR(&new_val, Z_STR_P(val));
 								php_pgsql_add_quotes(&new_val, 1);
 							}
 #undef REGEX0
@@ -4998,7 +4998,7 @@ PHP_PGSQL_API int php_pgsql_convert(PGconn *pg_link, const char *table_name, con
 								err = 1;
 							}
 							else {
-								ZVAL_STRINGL(&new_val, Z_STRVAL_P(val), Z_STRLEN_P(val));
+								ZVAL_STR(&new_val, Z_STR_P(val));
 								php_pgsql_add_quotes(&new_val, 1);
 							}
 #undef REGEX0
@@ -5151,7 +5151,7 @@ PHP_PGSQL_API int php_pgsql_convert(PGconn *pg_link, const char *table_name, con
 								err = 1;
 							}
 							else {
-								ZVAL_STRINGL(&new_val, Z_STRVAL_P(val), Z_STRLEN_P(val));
+								ZVAL_STR(&new_val, Z_STR_P(val));
 								php_pgsql_add_quotes(&new_val, 1);
 							}
 #undef REGEX0

Original file line number	Diff line number	Diff line change
`@@ -4696,7 +4696,7 @@ PHP_PGSQL_API int php_pgsql_convert(PGconn pg_link, const char table_name, con`
`4696`	`4696`	`err = 1;`
`4697`	`4697`	`}`
`4698`	`4698`	`else {`
`4699`		`- ZVAL_STRINGL(&new_val, Z_STRVAL_P(val), Z_STRLEN_P(val));`
	`4699`	`+ ZVAL_STR(&new_val, Z_STR_P(val));`
`4700`	`4700`	`}`
`4701`	`4701`	`#undef REGEX0`
`4702`	`4702`	`}`
`@@ -4746,7 +4746,7 @@ PHP_PGSQL_API int php_pgsql_convert(PGconn pg_link, const char table_name, con`
`4746`	`4746`	`}`
`4747`	`4747`	`}`
`4748`	`4748`	`else {`
`4749`		`- ZVAL_STRING(&new_val, Z_STRVAL_P(val));`
	`4749`	`+ ZVAL_STR(&new_val, Z_STR_P(val));`
`4750`	`4750`	`}`
`4751`	`4751`	`#undef REGEX0`
`4752`	`4752`	`#undef REGEX1`
`@@ -4850,7 +4850,7 @@ PHP_PGSQL_API int php_pgsql_convert(PGconn pg_link, const char table_name, con`
`4850`	`4850`	`err = 1;`
`4851`	`4851`	`}`
`4852`	`4852`	`else {`
`4853`		`- ZVAL_STRINGL(&new_val, Z_STRVAL_P(val), Z_STRLEN_P(val));`
	`4853`	`+ ZVAL_STR(&new_val, Z_STR_P(val));`
`4854`	`4854`	`convert_to_long(&new_val);`
`4855`	`4855`	`}`
`4856`	`4856`	`}`
`@@ -4896,7 +4896,7 @@ PHP_PGSQL_API int php_pgsql_convert(PGconn pg_link, const char table_name, con`
`4896`	`4896`	`err = 1;`
`4897`	`4897`	`}`
`4898`	`4898`	`else {`
`4899`		`- ZVAL_STRINGL(&new_val, Z_STRVAL_P(val), Z_STRLEN_P(val));`
	`4899`	`+ ZVAL_STR(&new_val, Z_STR_P(val));`
`4900`	`4900`	`php_pgsql_add_quotes(&new_val, 1);`
`4901`	`4901`	`}`
`4902`	`4902`	`#undef REGEX0`
`@@ -4965,7 +4965,7 @@ PHP_PGSQL_API int php_pgsql_convert(PGconn pg_link, const char table_name, con`
`4965`	`4965`	`err = 1;`
`4966`	`4966`	`}`
`4967`	`4967`	`else {`
`4968`		`- ZVAL_STRINGL(&new_val, Z_STRVAL_P(val), Z_STRLEN_P(val));`
	`4968`	`+ ZVAL_STR(&new_val, Z_STR_P(val));`
`4969`	`4969`	`php_pgsql_add_quotes(&new_val, 1);`
`4970`	`4970`	`}`
`4971`	`4971`	`#undef REGEX0`
`@@ -4998,7 +4998,7 @@ PHP_PGSQL_API int php_pgsql_convert(PGconn pg_link, const char table_name, con`
`4998`	`4998`	`err = 1;`
`4999`	`4999`	`}`
`5000`	`5000`	`else {`
`5001`		`- ZVAL_STRINGL(&new_val, Z_STRVAL_P(val), Z_STRLEN_P(val));`
	`5001`	`+ ZVAL_STR(&new_val, Z_STR_P(val));`
`5002`	`5002`	`php_pgsql_add_quotes(&new_val, 1);`
`5003`	`5003`	`}`
`5004`	`5004`	`#undef REGEX0`
`@@ -5151,7 +5151,7 @@ PHP_PGSQL_API int php_pgsql_convert(PGconn pg_link, const char table_name, con`
`5151`	`5151`	`err = 1;`
`5152`	`5152`	`}`
`5153`	`5153`	`else {`
`5154`		`- ZVAL_STRINGL(&new_val, Z_STRVAL_P(val), Z_STRLEN_P(val));`
	`5154`	`+ ZVAL_STR(&new_val, Z_STR_P(val));`
`5155`	`5155`	`php_pgsql_add_quotes(&new_val, 1);`
`5156`	`5156`	`}`
`5157`	`5157`	`#undef REGEX0`