Re: patch: stream leak fix

[Alexander Larsson <alexl redhat com>]

On 5 Dec 2002, Frederic Crozat wrote:

> Le jeu 05/12/2002 à 10:49, Alexander Larsson a écrit :
> > On 4 Dec 2002, Frederic Crozat wrote:
> > 
> > > Hi,
> > > 
> > > while trying to fix a monitoring problem for gnome-vfs mime user dir, I
> > > found what I think is a stream leak in both write_back_keys_user_file
> > > and write_back_mime_user_file..
> > > 
> > > Can I commit the attached patch (with changelog entry, of course..) ?
> > 
> > I think a better solution is to make ensure_user_directory_exist() return 
> > a gboolean and closedir() itself, since no users actually use the DIR *.
> 
> Agreed.. But my first though was to not modify internal calls :))
> 
> Better patch attached..

Looks good to me.

-- 
=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
 Alexander Larsson                                            Red Hat, Inc 
                   alexl redhat com    alla lysator liu se 
He's an uncontrollable amnesiac gentleman spy for the 21st century. She's a 
plucky cat-loving femme fatale from the wrong side of the tracks. They fight 
crime!