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

fix using null string in vector (#7872)

Browse Source 
Use 0 offset as special value. 0 offset is not a valid relative offset, so it's safe to use 0 offset to indicate value is null.

https://github.com/google/flatbuffers/issues/7846
This commit is contained in:
phenixxy
2023-03-24 11:15:34 +08:00
committed by GitHub
parent 50cdf92e1e
commit 1cb1c4baee
2 changed files with 7 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
net/FlatBuffers/FlatBufferBuilder.cs
View File

@@ -438,7 +438,8 @@ namespace Google.FlatBuffers
if (off > Offset) if (off > Offset)
throw new ArgumentException(); throw new ArgumentException();
off = Offset - off + sizeof(int); if (off != 0)
off = Offset - off + sizeof(int);
PutInt(off); PutInt(off);
} }

6
net/FlatBuffers/Table.cs
View File

@@ -65,7 +65,11 @@ namespace Google.FlatBuffers
// Create a .NET String from UTF-8 data stored inside the flatbuffer. // Create a .NET String from UTF-8 data stored inside the flatbuffer.
public string __string(int offset) public string __string(int offset)
{ {
offset += bb.GetInt(offset); int stringOffset = bb.GetInt(offset);
if (stringOffset == 0)
return null;
offset += stringOffset;
var len = bb.GetInt(offset); var len = bb.GetInt(offset);
var startPos = offset + sizeof(int); var startPos = offset + sizeof(int);
return bb.GetStringUTF8(startPos, len); return bb.GetStringUTF8(startPos, len);