Fixed bounds checking in PlaybackService::GetTrack.

clangen · Nov 26, 2016 · 64a187d · 64a187d
1 parent 5ac6f41
commit 64a187d
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/src/musikbox/app/service/PlaybackService.cpp b/src/musikbox/app/service/PlaybackService.cpp
@@ -437,7 +437,9 @@ double PlaybackService::GetDuration() {
 IRetainedTrack* PlaybackService::GetTrack(size_t index) {
     boost::recursive_mutex::scoped_lock lock(this->playlistMutex);
 
-    if (index >= 0 && index < this->playlist.Count()) {
+    const size_t count = this->playlist.Count();
+
+    if (count && index >= 0 && index < this->playlist.Count()) {
         return new RetainedTrack(this->playlist.Get(index));
     }