Silly second declaration of the variable here that triggers an infinite

loop in some cases (I'm surprised that it doesn't happen in more...this
code has been around for a while.).

BUG:113437

git-svn-id: svn://anonsvn.kde.org/home/kde/trunk/kdesupport/taglib@464483 283d02a7-25f6-0310-bc7c-ecb5cbfe19da

diff --git a/mpeg/mpegfile.cpp b/mpeg/mpegfile.cpp
index cf126ce8b724f76b1d56a741ff7b686fd826cae7..a825ad08050f72946e92f896cf80d747f6081bd7 100644 (file)
--- a/mpeg/mpegfile.cpp
+++ b/mpeg/mpegfile.cpp
@@ -450,7 +450,7 @@ long MPEG::File::nextFrameOffset(long position)
 
   while(buffer.size() > 0) {
     seek(position);
-    ByteVector buffer = readBlock(bufferSize());
+    buffer = readBlock(bufferSize());
 
     for(uint i = 0; i < buffer.size(); i++) {
       if(uchar(buffer[i]) == 0xff && secondSynchByte(buffer[i + 1]))