[MPlayer-cvslog] r32731 - in trunk: path.c path.h

Clément Bœsch ubitux at gmail.com
Tue Dec 28 22:19:38 CET 2010


On Wed, Dec 29, 2010 at 12:57:37AM +0900, KO Myung-Hun wrote:
[...]
> Yes, but you should consider the case that people append a relative path
> to a current path.
> 
> \something and c: generate a incorrect path.
> 
> I agree absolutely with you that these codes should be reviewed
> thoroughly for a security.
> 
> And it would be better to provide a function to composite a absolute
> path from a relative path.
> 
> This can reduce the errors on OS using DOSish path.
> 

What do you think of this patch? Also, I'm not sure it solve the
"security" issue mentionned by Reimar since I didn't figure out a real
case example.

I'll commit this in the next days if no one object, so I can update my
subdirs patch and maybe have a review of it (diff reduced!) :)

-- 
Clément B.
Not sent from a jesusPhone.


More information about the MPlayer-cvslog mailing list