BigfootDev/flatbuffers
1
0
Fork 1
mirror of https://github.com/google/flatbuffers.git synced 2026-07-04 14:21:12 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix Issue #8653 - Python vtables not considering object size (#8683)

Browse Source 
* have vtables consider size

* simplification from comment

---------

Co-authored-by: Wouter van Oortmerssen <aardappel@gmail.com>
This commit is contained in:
Justin Davis
2025-11-17 16:43:28 -05:00
committed by GitHub
parent 2a8f4568e0
commit e3ee24830e
1 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
python/flatbuffers/builder.py
View File

@@ -245,7 +245,10 @@ class Builder(object):
vtKey.append(elem) vtKey.append(elem)
objectSize = UOffsetTFlags.py_type(objectOffset - self.objectEnd)
vtKey.append(objectSize)
vtKey = tuple(vtKey) vtKey = tuple(vtKey)
# calculate the size of the object
vt2Offset = self.vtables.get(vtKey) vt2Offset = self.vtables.get(vtKey)
if vt2Offset is None: if vt2Offset is None:
# Did not find a vtable, so write this one to the buffer. # Did not find a vtable, so write this one to the buffer.
@@ -275,7 +278,6 @@ class Builder(object):
# The two metadata fields are written last. # The two metadata fields are written last.
# First, store the object bytesize: # First, store the object bytesize:
objectSize = UOffsetTFlags.py_type(objectOffset - self.objectEnd)
self.PrependVOffsetT(VOffsetTFlags.py_type(objectSize)) self.PrependVOffsetT(VOffsetTFlags.py_type(objectSize))
# Second, store the vtable bytesize: # Second, store the vtable bytesize: