View Issue Details

IDCategoryLast Update
0007661bugs2018-10-30 07:59
Reporterjohne53Assigned Tox42 
Reproducibilityalways 
Status resolvedResolutionfixed 
PlatformAllOSAllOS Version
Product Version5.X git (version in description) 
Fixed in Version 
Summary0007661: Unitialized variable in 'libs/ardour/ltc_file_reader.cc'
DescriptionDiscovered 11th Sept 2018

libardour contains the following at line 233 of 'libs/ardour/ltc_file_reader.cc':-

      double fp_sec = frame.off_start / (double) _info.samplerate;

which is near to the bottom of function 'LTCFileReader::read_ltc()'.

I might have missed something but AFAICT 'frame.off_start' never got initialized anywhere.
TagsNo tags attached.

Activities

x42

2018-09-11 14:04

administrator   ~0020381

Thanks for the heads up. Fixed in 6.0-pre0-1000-gd7dc224da

johne53

2018-09-13 11:28

reporter   ~0020382

Hi Robin - just a heads-op that your fix won't work yet in the Mixbus five branch. Basically, it uses Temporal::samplepos_t but I don't think libtemporal's been ported yet to the five branch.

Issue History

Date Modified Username Field Change
2018-09-11 09:27 johne53 New Issue
2018-09-11 14:04 x42 Note Added: 0020381
2018-09-11 14:04 x42 Status new => resolved
2018-09-11 14:04 x42 Resolution open => fixed
2018-09-11 14:04 x42 Assigned To => x42
2018-09-13 11:28 johne53 Note Added: 0020382