Re: [Libcdio-devel] Ready for merging: libcdio branch trackno-higher-one

[Edd Barrett]

On Sat, Dec 29, 2018 at 01:52:44PM +0100, Thomas Schmitt wrote:
> _init_bincue() sets .first_track to 1 and calls parse_cuefile() with a
> non-NULL argument "cd". It too sets cd->gen.i_first_track to 1 before
> examining the TRACK lines:
> 
>         /* TRACK N <mode> */
>       } else if (0 == strcmp("TRACK", psz_keyword)) {
>         ...
>           if (1!=sscanf(psz_field, "%d", &i_track)) {
> 
> The variable i_track is not used further. The only purpose for the scanf()
> is to check whether some arbitrary number is written there.
> 
> Remedy would be to record the track number of the first TRACK line
> as p_env->gen.i_first_track and to demand that the subsequent TRACK lines
> bear nicely incrementing numbers.
> 
> Then we probably get errors when using track numbers ...
> I will try what happens.

Nice find. Let us know the outcome!

-- 
Best Regards
Edd Barrett

http://www.theunixzoo.co.uk