diff --git a/ext/json/json_encoder.c b/ext/json/json_encoder.c index ae9e086fc05b5..8da2761e99764 100644 --- a/ext/json/json_encoder.c +++ b/ext/json/json_encoder.c @@ -467,7 +467,9 @@ static void php_json_encode_serializable_object(smart_str *buf, zval *val, int o origin_error_code = JSON_G(error_code); if (FAILURE == call_user_function_ex(EG(function_table), val, &fname, &retval, 0, NULL, 1, NULL) || Z_TYPE(retval) == IS_UNDEF) { - zend_throw_exception_ex(NULL, 0, "Failed calling %s::jsonSerialize()", ZSTR_VAL(ce->name)); + if (!EG(exception)) { + zend_throw_exception_ex(NULL, 0, "Failed calling %s::jsonSerialize()", ZSTR_VAL(ce->name)); + } smart_str_appendl(buf, "null", sizeof("null") - 1); zval_ptr_dtor(&fname); return; diff --git a/ext/json/tests/bug73113.phpt b/ext/json/tests/bug73113.phpt new file mode 100644 index 0000000000000..b1d70bc4c7af5 --- /dev/null +++ b/ext/json/tests/bug73113.phpt @@ -0,0 +1,24 @@ +--TEST-- +Bug #73113 (Segfault with throwing JsonSerializable) +Also test that the custom exception is not wrapped by ext/json +--SKIPIF-- + +--FILE-- +getMessage(); +} +--EXPECTF-- +This error is expected