[kdenlive] Review Request 125707: fix: Timeline incorrectly checks track index bounds crash

[The DiveO]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125707/
-----------------------------------------------------------

(Updated Oct. 19, 2015, 4:47 p.m.)


Review request for Kdenlive and Vincent Pinon.


Repository: kdenlive


Description
-------

Class Timeline incorrectly checks the bounds for track indices: the upper bound is checked against greater than track.count(), where in fact the indices run from 0 to count-1.

However, this masks the underlying problem with converted Kdenlive projects: when adding new tracks, the index gets out of bounds, so this needs to be fixed too. This patch just fixes the crash in itself but not the underlying problem of handing in an out-of-bounds index to these functions to be fixed.


Diffs (updated)
-----

  src/timeline/timeline.cpp fb5bf90 

Diff: https://git.reviewboard.kde.org/r/125707/diff/


Testing
-------

Tested against recent Kdenlive branch Applications/15.08, on Kubuntu 15.04 64bit system. Tested with a converted 15.04 Kdenlive project that exhibits this bug when trying to add new tracks to the timeline.


Thanks,

The DiveO

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdenlive/attachments/20151019/f3b4ee63/attachment.html>