Commit f881ca19 authored by Markus Armbruster's avatar Markus Armbruster
Browse files

vl: Assert drive_new() does not fail in default_drive() 



If creating (empty) default drives fails, it's a bug.  Therefore,
assert() is more appropriate than exit(1).

Cc: Kevin Wolf <kwolf@redhat.com>
Cc: Max Reitz <mreitz@redhat.com>
Signed-off-by: default avatarMarkus Armbruster <armbru@redhat.com>
Reviewed-by: default avatarEric Blake <eblake@redhat.com>
Reviewed-by: default avatarMax Reitz <mreitz@redhat.com>
Message-Id: <20181017082702.5581-33-armbru@redhat.com>
parent b836723d

vl.c

+1 −3
Original line number Diff line number Diff line
@@ -1156,9 +1156,7 @@ static void default_drive(int enable, int snapshot, BlockInterfaceType type, 
    }



    dinfo = drive_new(opts, type);

    if (!dinfo) {

        exit(1);

    }

    assert(dinfo);

    dinfo->is_default = true;



}