Skip to content

Commit dda0609

Browse files
committed
fix assertion on invalid data
1 parent b285146 commit dda0609

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

msgpack.c

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -239,6 +239,10 @@ PHP_MSGPACK_API int php_msgpack_unserialize(zval *return_value, char *str, size_
239239
if (off < str_len) {
240240
MSGPACK_WARNING("[msgpack] (%s) Extra bytes", __FUNCTION__);
241241
}
242+
if (Z_ISREF_P(return_value)) {
243+
/* this must not happen, but may happen on unserializing random invalid data */
244+
ZVAL_UNREF(return_value);
245+
}
242246
return SUCCESS;
243247
default:
244248
MSGPACK_WARNING("[msgpack] (%s) Unknown result", __FUNCTION__);

0 commit comments

Comments
 (0)