Re: [matroska] r901 - trunk/DvdMenuXtractor/libdvdread/dvdread

Hey,

> @@ -393,6 +393,10 @@
>  	if( strlen( path_copy ) > 1 ) {
>  	    if( path_copy[ strlen( path_copy ) - 1 ] == '/' ) 
>  		path_copy[ strlen( path_copy ) - 1 ] = '\0';
> +#if defined(WIN32)
> +	    if( path_copy[ strlen( path_copy ) - 1 ] == '\\' ) 
> +		path_copy[ strlen( path_copy ) - 1 ] = '\0';
> +#endif
>  	}

<nitpickmode>
This could segfault if path_copy is '\/'. This might not be a valid path
in Windows, but it should be checked for nevertheless.
</nitpickmode>

Mosu

-- 
If Darl McBride was in charge, he'd probably make marriage
unconstitutional too, since clearly it de-emphasizes the commercial
nature of normal human interaction, and probably is a major impediment
to the commercial growth of prostitution. - Linus Torvalds