dev builds
themes manual
device status forums
mailing lists
IRC bugs
dev guide

Rockbox mail archive

Subject: Re: cvs: firmware id3.c,1.41,1.42

Re: cvs: firmware id3.c,1.41,1.42

From: Magnus Holmgren <>
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

Page template was last modified "Tue Sep 7 00:00:02 2021" The Rockbox Crew -- Privacy Policy