Hi,

please don't forget to mention what changed in a revised version

On Sat, May 25, 2013 at 03:16:39PM +0100, Filipe David Borba Manana wrote:
> @@ -228,6 +227,8 @@ out:
>       free(chunk_root);
>       free(dev_root);
>       free(csum_root);
> +     if (fs_info)
> +             free(fs_info->super_copy);

free accepts NULL pointers and does nothing, so it's fine to do it as in
the first version.

>       free(fs_info);
>       return NULL;

otherwise ok. Thanks.

david
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to