Rockbox mail archiveSubject: Re: cvs: firmware id3.c,1.41,1.42
Re: cvs: firmware id3.c,1.41,1.42
From: Magnus Holmgren <lear_at_algonet.se>
Date: Mon, 16 Sep 2002 11:56:25 +0200 (CEST)
Daniel Stenberg wrote:
> Wouldn't be enough to just make sure that when we encounter an unown tag, we
> check that if readsize (after "readsize += headerlen") gets bigger than size,
> we bail out of the loop?
That'd work too, but why treat this case any differently? :) But with my suggestion, the bail-out is only delayed slightly (and the code could perhaps be generalized a bit?).
Received on 2002-09-16