Re: [PATCH] Don't crash on empty ftp paths



On Sun, 2006-03-19 at 18:24 +0100, Christian Neumair wrote:
> The attached patch is meant to fix bug 308231 [1] by replacing some
> occurences of
>  uri->text
> by
>  uri->text != NULL ? uri->text : "/"
> 
> It also fixes two hypothetical memleaks, which probably will never be an
> issue, since I've never seen a GnomeVFS FTP URI with uri->text strlen 0,
> i.e with a "" text, which is why I just checked for validity of the
> URI's text to discover empty paths.

Looks good to commit.

=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
 Alexander Larsson                                            Red Hat, Inc 
                   alexl redhat com    alla lysator liu se 
He's a jaded day-dreaming paranormal investigator plagued by the memory of his 
family's brutal murder. She's a pregnant gold-digging magician's assistant 
with the power to bend men's minds. They fight crime! 




[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]