On 21 Mar 2016, at 00:29, Alexander Motin <m...@freebsd.org> wrote:
> Author: mav
> Date: Sun Mar 20 23:29:58 2016
> New Revision: 297089
> URL: https://svnweb.freebsd.org/changeset/base/297089
> 
> Log:
>  MFC r292653 (by bapt): Report an error if zdb cannot initialize zfs
> 
>  If the zfs module is not present and not loadable, report an error
>  to the user instead of crashing
> 
>  Differential Revision:  https://reviews.freebsd.org/D4691
> 
> Modified:
>  stable/10/cddl/contrib/opensolaris/cmd/zdb/zdb.c
> Directory Properties:
>  stable/10/   (props changed)
> 
> Modified: stable/10/cddl/contrib/opensolaris/cmd/zdb/zdb.c
> ==============================================================================
> --- stable/10/cddl/contrib/opensolaris/cmd/zdb/zdb.c  Sun Mar 20 23:26:52 
> 2016        (r297088)
> +++ stable/10/cddl/contrib/opensolaris/cmd/zdb/zdb.c  Sun Mar 20 23:29:58 
> 2016        (r297089)
> @@ -3658,7 +3658,8 @@ main(int argc, char **argv)
> 
>       kernel_init(FREAD);
>       g_zfs = libzfs_init();
> -     ASSERT(g_zfs != NULL);
> +     if (g_zfs == NULL)
> +             fatal("Fail to initialize zfs");

Minor nit: "Failed to" or "Unable to" is probably better here.

-Dimitry

Attachment: signature.asc
Description: Message signed with OpenPGP using GPGMail

Reply via email to