]> git.cworth.org Git - apitrace/commitdiff
Fix size comparisons in traceloader.cpp.
authorJosé Fonseca <jose.r.fonseca@gmail.com>
Thu, 22 Mar 2012 16:26:47 +0000 (16:26 +0000)
committerJosé Fonseca <jose.r.fonseca@gmail.com>
Thu, 22 Mar 2012 18:13:41 +0000 (18:13 +0000)
This would never result in buffer in overflows, but this is more correct.

gui/traceloader.cpp

index 7cb07f00aa1331e814c6d7025deffe76498a36fc..0e6ed47351260a8d35e1998cc9728423cff7102f 100644 (file)
@@ -104,7 +104,7 @@ int TraceLoader::numberOfFrames() const
 
 int TraceLoader::numberOfCallsInFrame(int frameIdx) const
 {
-    if (frameIdx > m_frameBookmarks.size()) {
+    if (frameIdx >= m_frameBookmarks.size()) {
         return 0;
     }
     FrameBookmarks::const_iterator itr =
@@ -386,7 +386,7 @@ int TraceLoader::callInFrame(int callIdx) const
 {
     unsigned numCalls = 0;
 
-    for (int frameIdx = 0; frameIdx <= m_frameBookmarks.size(); ++frameIdx) {
+    for (int frameIdx = 0; frameIdx < m_frameBookmarks.size(); ++frameIdx) {
         const FrameBookmark &frameBookmark = m_frameBookmarks[frameIdx];
         unsigned firstCall = numCalls;
         unsigned endCall = numCalls + frameBookmark.numberOfCalls;