src/lib/CMXParser.cpp | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-)
New commits: commit e646d40f53bfb112abfd8153268665f73153b00c Author: Miklos Vajna <[email protected]> Date: Mon Nov 27 08:31:23 2017 +0100 CMXParser: remove never read length / maxLength Technically both were read and written at least once so the compiler did not warn on them; but in practice these are unused. Leave the readU32() call there, though -- as it has a side effect. Change-Id: I24240ef216b6ce18d792594ff77fad46e6688508 diff --git a/src/lib/CMXParser.cpp b/src/lib/CMXParser.cpp index 9cbfecc..57a6f5e 100644 --- a/src/lib/CMXParser.cpp +++ b/src/lib/CMXParser.cpp @@ -345,10 +345,7 @@ void libcdr::CMXParser::readIxmr(librevenge::RVNGInputStream *input) unsigned fourCC = readU32(input, m_bigEndian); if (CDR_FOURCC_ixmr != fourCC) return; - unsigned length = readU32(input, m_bigEndian); - const unsigned long maxLength = getRemainingLength(input); - if (length > maxLength) - length = maxLength; + readU32(input, m_bigEndian); // Length readU16(input, m_bigEndian); // Master ID readU16(input, m_bigEndian); // Size _______________________________________________ Libreoffice-commits mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits
