A two code blocks above, the code makes sure that lv_name isn't NULL. No need to make this check here. Signed-off-by: Mikulas Patocka alloc, vg))) + if (!(lv = lv_create_empty(lv_name, NULL, status, lp->alloc, vg))) return_0; if (lp->read_ahead) {