[MPlayer-cvslog] r18963 - in trunk: libmpcodecs/ad_hwmpa.c libmpdemux/aviheader.c libmpdemux/demux_demuxers.c libmpdemux/demux_gif.c libmpdemux/demux_lmlm4.c libmpdemux/demux_mov.c libmpdemux/demux_mpg.c libmpdemux/demux_nuv.c libmpdemux/demux_ogg.c libmpdemux/demux_realaud.c libmpdemux/demux_ty.c libmpdemux/demux_ty_osd.c libmpdemux/http.c libmpdemux/muxer_avi.c libmpdemux/muxer_mpeg.c libmpdemux/tvi_v4l2.c m_option.c

[Diego Biurrun]

On Sun, Jul 09, 2006 at 01:38:29AM +0200, Roberto Togni wrote:
> On Sat,  8 Jul 2006 19:28:37 +0200 (CEST)
> diego <subversion at mplayerhq.hu> wrote:
> 
> [...]
> 
> > --- trunk/libmpdemux/http.c	(original)
> > +++ trunk/libmpdemux/http.c	Sat Jul  8 19:28:36 2006
> 
> [...]
> 
> > @@ -856,7 +855,6 @@
> >  static int fixup_open(stream_t *stream,int seekable) {
> >  	HTTP_header_t *http_hdr = stream->streaming_ctrl->data;
> >  	int is_icy = http_hdr && http_get_field(http_hdr, "Icy-MetaInt");
> > -	char *content_type = http_get_field( http_hdr, "Content-Type" );
> 
> Is this safe? http_get_field() modifies some parts of http_hdr struct. I
> have no idea if something depends on those changes.

Dunno, you tell me ...

Diego