Skip to content

box: fix interval/decimal/tuple MsgPack decoding of zero length data#11734

Merged
locker merged 1 commit intotarantool:masterfrom
nshy:gh-10361-misc-zero-len-mp-decoding
Aug 11, 2025
Merged

box: fix interval/decimal/tuple MsgPack decoding of zero length data#11734
locker merged 1 commit intotarantool:masterfrom
nshy:gh-10361-misc-zero-len-mp-decoding

Conversation

@nshy
Copy link
Contributor

@nshy nshy commented Aug 6, 2025

Zero length data is possible input for MsgPack validation functions. So let's handle it for decimal, interval and tuple.

Closes #10361

@nshy nshy requested a review from a team as a code owner August 6, 2025 13:40
@nshy nshy requested a review from locker August 6, 2025 13:52
@coveralls
Copy link

coveralls commented Aug 6, 2025

Coverage Status

coverage: 87.58% (-0.009%) from 87.589%
when pulling e6a8e44 on nshy:gh-10361-misc-zero-len-mp-decoding
into 2502633
on tarantool:master
.

@locker locker assigned nshy and unassigned locker Aug 7, 2025
Zero payoload is possible input for MsgPack validation functions.
So let's handle it for decimal, interval and tuple.

Closes tarantool#10361

NO_DOC=bugfix
@nshy nshy force-pushed the gh-10361-misc-zero-len-mp-decoding branch from 7ed02ae to e6a8e44 Compare August 7, 2025 13:01
@nshy nshy assigned locker and unassigned nshy Aug 7, 2025
@locker locker assigned nshy and unassigned locker Aug 7, 2025
@nshy nshy added the full-ci Enables all tests for a pull request label Aug 8, 2025
@nshy nshy assigned locker and unassigned nshy Aug 8, 2025
@locker locker merged commit 7195144 into tarantool:master Aug 11, 2025
58 of 59 checks passed
@locker
Copy link
Member

locker commented Aug 11, 2025

Cherry-picked to 3.2, 3.3, 3.4.

@nshy nshy deleted the gh-10361-misc-zero-len-mp-decoding branch August 11, 2025 08:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

full-ci Enables all tests for a pull request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SIGSEGV on parsing MessagePack buffer: decimal_unpack: Assertion `len > 0' failed

4 participants