MantisBT - ardour
View Issue Details
0007661ardourbugspublic2018-09-11 02:272018-10-30 00:59
johne53 
x42 
normalmajoralways
resolvedfixed 
AllAll
5.X git (version in description) 
 
0007661: Unitialized variable in 'libs/ardour/ltc_file_reader.cc'
Discovered 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.
No tags attached.
Issue History
2018-09-11 02:27johne53New Issue
2018-09-11 07:04x42Note Added: 0020381
2018-09-11 07:04x42Statusnew => resolved
2018-09-11 07:04x42Resolutionopen => fixed
2018-09-11 07:04x42Assigned To => x42
2018-09-13 04:28johne53Note Added: 0020382
2018-10-30 00:59user3518Note Added: 0020427
2018-10-31 10:47paulNote Deleted: 0020427

Notes
(0020381)
x42   
2018-09-11 07:04   
Thanks for the heads up. Fixed in 6.0-pre0-1000-gd7dc224da
(0020382)
johne53   
2018-09-13 04:28   
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.