]> granicus.if.org Git - taglib/commitdiff
Check if the header is TTA1 before parsing
authorStephen F. Booth <me@sbooth.org>
Sat, 4 Feb 2012 13:39:45 +0000 (08:39 -0500)
committerStephen F. Booth <me@sbooth.org>
Sat, 4 Feb 2012 13:39:45 +0000 (08:39 -0500)
taglib/trueaudio/trueaudioproperties.cpp

index bae386aa82755db603742e9c461736643fcd29fd..1e4e87d3da6e7436f5972faed0dfbe148a09a7d0 100644 (file)
@@ -125,19 +125,26 @@ void TrueAudio::Properties::read()
   int pos = 3;
 
   d->version = d->data[pos] - '0';
-  pos += 1 + 2;
+  pos += 1;
 
-  d->channels = d->data.mid(pos, 2).toShort(false);
-  pos += 2;
+  // According to http://en.true-audio.com/TTA_Lossless_Audio_Codec_-_Format_Description
+  // TTA2 headers are in development, and have a different format
+  if(1 == d->version) {
+    // Skip the audio format
+    pos += 2;
 
-  d->bitsPerSample = d->data.mid(pos, 2).toShort(false);
-  pos += 2;
+    d->channels = d->data.mid(pos, 2).toShort(false);
+    pos += 2;
 
-  d->sampleRate = d->data.mid(pos, 4).toUInt(false);
-  pos += 4;
+    d->bitsPerSample = d->data.mid(pos, 2).toShort(false);
+    pos += 2;
 
-  d->sampleFrames = d->data.mid(pos, 4).toUInt(false);
-  d->length = d->sampleFrames / d->sampleRate;
+    d->sampleRate = d->data.mid(pos, 4).toUInt(false);
+    pos += 4;
 
-  d->bitrate = d->length > 0 ? ((d->streamLength * 8L) / d->length) / 1000 : 0;
+    d->sampleFrames = d->data.mid(pos, 4).toUInt(false);
+    d->length = d->sampleFrames / d->sampleRate;
+
+    d->bitrate = d->length > 0 ? ((d->streamLength * 8L) / d->length) / 1000 : 0;
+  }
 }