Skip to content
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

Jump to bottom

[.NET] Fix VerifyUnion using wrong offset for typeId #8360

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

[.NET] Fix VerifyUnion using wrong offset for typeId #8360

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
341b479
[.NET] Fix VerifyUnion using wrong offset for typeId
0x4261756D Jul 12, 2024
d9f3628
Merge branch 'google:master' into master
0x4261756D Sep 10, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions net/FlatBuffers/FlatBufferVerify.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -684,10 +684,10 @@ public bool VerifyUnion(uint tablePos, short typeIdVOffset, short valueVOffset,
{
return false;
}
// Check union data
offset = GetVOffset(tablePos, valueVOffset);
// Take type id
var typeId = verifier_buffer.Get(Convert.ToInt32(offset));
// Check union data
offset = GetVOffset(tablePos, valueVOffset);
if (offset == 0)
{
// When value data is not present, allow union verification function to deal with illegal offset
Expand Down
Loading