From: Stephen F. Booth <me@sbooth.org>
Date: Thu, 12 Apr 2012 14:53:32 +0000 (-0400)
Subject: Verify that an APE text item isn't empty before use
X-Git-Tag: v1.8beta~15
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=4f8a6fdfaf1d9335f7e97d578929f46070240582;p=taglib

Verify that an APE text item isn't empty before use
---

diff --git a/taglib/ape/apeitem.cpp b/taglib/ape/apeitem.cpp
index 1e60bc52..ac0e26ed 100644
--- a/taglib/ape/apeitem.cpp
+++ b/taglib/ape/apeitem.cpp
@@ -170,15 +170,15 @@ int APE::Item::size() const
   int result = 8 + d->key.size() /* d->key.data(String::UTF8).size() */ + 1;
   switch (d->type) {
     case Text:
-      {
+      if(d->text.size()) {
         StringList::ConstIterator it = d->text.begin();
 		  
         result += it->data(String::UTF8).size();
         it++;
         for(; it != d->text.end(); ++it)
           result += 1 + it->data(String::UTF8).size();
-        break;
       }
+      break;
 
     case Binary:
     case Locator: