Re: [PATCH] fix for a memleak in grub_ntfs_mount

[Robert Millan]

On Wed, Aug 20, 2008 at 11:01:26PM +0200, Felix Zielcke wrote:
> 2008-08-20  Felix Zielcke  <address@hidden>
> 
>       * fs/ntfs.c (grub_ntfs_mount): Fix a memory leak. 
> 
> Index: fs/ntfs.c
> ===================================================================
> --- fs/ntfs.c (Revision 1822)
> +++ fs/ntfs.c (Arbeitskopie)
> @@ -850,6 +850,7 @@ fail:
>      {
>        free_file (&data->mmft);
>        free_file (&data->cmft);
> +      grub_free (data);
>      }
>    return 0;
>  }

Looks fine to me.

-- 
Robert Millan

  The DRM opt-in fallacy: "Your data belongs to us. We will decide when (and
  how) you may access your data; but nobody's threatening your freedom: we
  still allow you to remove your data and not access it at all."