summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorstefan301 <32997632+stefan301@users.noreply.github.com>2017-10-23 18:15:18 +0200
committerWouter van Oortmerssen <aardappel@gmail.com>2017-10-23 09:15:18 -0700
commit6bb0a728d3d6039e26528f5a3c15309a8dfc13d2 (patch)
treeedf2f8866e31f7e993ce9cc55baa2812b709aee5
parent97face1527989f59479ab61d47bb13765257bf8a (diff)
downloadflatbuffers-6bb0a728d3d6039e26528f5a3c15309a8dfc13d2.tar.gz
flatbuffers-6bb0a728d3d6039e26528f5a3c15309a8dfc13d2.tar.bz2
flatbuffers-6bb0a728d3d6039e26528f5a3c15309a8dfc13d2.zip
Added missing EndTable() call to VerifyObject() (#4468)
VerifyObject called VerifyTableStart() but not EndTable(). This made Verifier::VerifyComplexity() increase depth_ with each table, not with the depth of tables. https://groups.google.com/forum/#!topic/flatbuffers/OpxtW5UFAdg
-rw-r--r--src/reflection.cpp3
1 files changed, 3 insertions, 0 deletions
diff --git a/src/reflection.cpp b/src/reflection.cpp
index 2f05a34c..6736ec7b 100644
--- a/src/reflection.cpp
+++ b/src/reflection.cpp
@@ -700,6 +700,9 @@ bool VerifyObject(flatbuffers::Verifier &v,
}
}
+ if (!v.EndTable())
+ return false;
+
return true;
}