Re: patch: stream leak fix

[Frederic Crozat <fcrozat mandrakesoft com>]

Le jeu 05/12/2002 à 18:19, Alexander Larsson a écrit :
> 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.

I've committed a fixed version of the patch (a return had to be changed
in the function..)

-- 
Frederic Crozat <fcrozat mandrakesoft com>
MandrakeSoft