]> granicus.if.org Git - taglib/commitdiff
Don't overrun the buffer if the first byte is 0xff
authorScott Wheeler <wheeler@kde.org>
Tue, 20 Jun 2006 20:41:30 +0000 (20:41 +0000)
committerScott Wheeler <wheeler@kde.org>
Tue, 20 Jun 2006 20:41:30 +0000 (20:41 +0000)
git-svn-id: svn://anonsvn.kde.org/home/kde/trunk/kdesupport/taglib@553365 283d02a7-25f6-0310-bc7c-ecb5cbfe19da

mpeg/mpegfile.cpp

index a825ad08050f72946e92f896cf80d747f6081bd7..46c6c605854042eb36fbf46c0a49985901efa0eb 100644 (file)
@@ -452,7 +452,7 @@ long MPEG::File::nextFrameOffset(long position)
     seek(position);
     buffer = readBlock(bufferSize());
 
-    for(uint i = 0; i < buffer.size(); i++) {
+    for(uint i = 0; i < buffer.size() - 1; i++) {
       if(uchar(buffer[i]) == 0xff && secondSynchByte(buffer[i + 1]))
        return position + i;
     }