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