Skip to content

Check if data is not empty in dotnet integer decoding functions#1030

Merged
PeterMatula merged 3 commits intoavast:masterfrom
HoundThe:oob_dotnet
Oct 1, 2021
Merged

Check if data is not empty in dotnet integer decoding functions#1030
PeterMatula merged 3 commits intoavast:masterfrom
HoundThe:oob_dotnet

Conversation

@HoundThe
Copy link
Copy Markdown
Member

Fix potential OOB read if passed data, to Signed/Unsigned dotnet decode function, is empty.

@ladislav-zezula
Copy link
Copy Markdown
Contributor

ladislav-zezula commented Sep 27, 2021

All asserts are gone. Just a little question, why has the data type of result in decodeSigned() changed from std::int64_t to std::uint64_t?

@HoundThe
Copy link
Copy Markdown
Member Author

Oh thank you for that, that was a mistake.

@PeterMatula PeterMatula merged commit c6605e7 into avast:master Oct 1, 2021
PeterMatula added a commit that referenced this pull request Oct 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants