Skip to content

json_last_error_msg - better message - error position near by #18866

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
+118 −27
Open

json_last_error_msg - better message - error position near by #18866

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
70024b7
first setup
juan-morales Jun 8, 2025
05f2fc5
counts ok
juan-morales Jun 10, 2025
b434301
memory leak fixed with old school
juan-morales Jun 10, 2025
73a671e
POC
juan-morales Jun 12, 2025
34439b8
poc tests passed for json ext
juan-morales Jun 13, 2025
a0764f3
some cleanup
juan-morales Jun 16, 2025
0593c47
comments in php_json_scanner.h
juan-morales Jun 16, 2025
f4f5e08
better position calculation
juan-morales Jun 16, 2025
765d574
fix test json_encode_exceptions.phpt
juan-morales Jun 17, 2025
eb6400c
Update ext/json/json.c - style - missing space
juan-morales Jun 20, 2025
7255a81
code style fixes
juan-morales Jun 20, 2025
0f3a191
style fix
juan-morales Jun 20, 2025
File filter

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
memory leak fixed with old school
  • Loading branch information
@juan-morales
juan-morales committed Jun 10, 2025
commit b4343011829a44af918d764a322bcdf950583030
7 changes: 6 additions & 1 deletion ext/json/json.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -375,6 +375,11 @@ PHP_FUNCTION(json_last_error_msg)
{
ZEND_PARSE_PARAMETERS_NONE();

RETURN_STRING(php_json_get_error_msg(JSON_G(error_code)));
char *msg = php_json_get_error_msg(JSON_G(error_code));
RETVAL_STRING(msg);
if (JSON_G(error_code) == PHP_JSON_ERROR_SYNTAX) {
efree(msg);
}
//original:RETURN_STRING(php_json_get_error_msg(JSON_G(error_code)));
}
/* }}} */