Re: [MPlayer-dev-eng] [PATCH] add optional device with dvd://
- Date: Thu, 3 Jul 2008 14:35:56 +0200
- From: Nico Sabbi <Nicola.Sabbi@xxxxxxxx>
- Subject: Re: [MPlayer-dev-eng] [PATCH] add optional device with dvd://
On Thursday 03 July 2008 12:47:49 Mathieu SCHROETER wrote:
> Mathieu SCHROETER a écrit :
> >+ if(!p->device)
> >+ dvd_device_current = strdup(dvd_device ? dvd_device :
>
> DEFAULT_DVD_DEVICE);
>
> Sorry previous patch is wrong because a leak exists on
> dvd_device_current. Now strdup() is only used on dvd_device
> (if necessary)...
>
>
> Regards,
> Mathieu SCHROETER
thanks, I'll review it tonight
_______________________________________________
MPlayer-dev-eng mailing list
MPlayer-dev-eng@xxxxxxxxxxxx
https://lists.mplayerhq.hu/mailman/listinfo/mplayer-dev-eng