[FFmpeg-devel] [PATCH] 2/6 Fix broken APE tag key handling

Matti Hamalainen mhamalai
Mon Aug 10 20:49:52 CEST 2009


On Mon, 10 Aug 2009, Kostya wrote:

> >      size = get_le32(pb);  /* field size */
> > -    url_fskip(pb, 4);     /* skip field flags */
> > -
> > -    for (i=0; pb->buf_ptr[i]!='0' && pb->buf_ptr[i]>=0x20 && pb->buf_ptr[i]<=0x7E; i++);
> 
> Ahem, is this buf_ptr[i]!='0' an error by original author or you just
> forgot to check for it?

Probably meant to be '\0' by original author. The allowed characters in 
APE tag field keys are 0x20 through 0x7e (see the Hydrogenaudio docs).
Thus I think checking for < 0x20 && > 0x7e is sufficient.

-- 
] ccr/TNSP^DKD^pWp  :: ccr tnsp org ::  http://ccr.tnsp.org/
] PGP key: 0466 95ED 96DF 3701 C71D D62D 10A6 28A6 1374 C112



More information about the ffmpeg-devel mailing list