diff options
Diffstat (limited to 'drivers/md/persistent-data/dm-space-map-checker.c')
-rw-r--r-- | drivers/md/persistent-data/dm-space-map-checker.c | 24 |
1 files changed, 12 insertions, 12 deletions
diff --git a/drivers/md/persistent-data/dm-space-map-checker.c b/drivers/md/persistent-data/dm-space-map-checker.c index 50ed53bf4aa2..6d7c8329250f 100644 --- a/drivers/md/persistent-data/dm-space-map-checker.c +++ b/drivers/md/persistent-data/dm-space-map-checker.c | |||
@@ -343,25 +343,25 @@ struct dm_space_map *dm_sm_checker_create(struct dm_space_map *sm) | |||
343 | int r; | 343 | int r; |
344 | struct sm_checker *smc; | 344 | struct sm_checker *smc; |
345 | 345 | ||
346 | if (!sm) | 346 | if (IS_ERR_OR_NULL(sm)) |
347 | return NULL; | 347 | return ERR_PTR(-EINVAL); |
348 | 348 | ||
349 | smc = kmalloc(sizeof(*smc), GFP_KERNEL); | 349 | smc = kmalloc(sizeof(*smc), GFP_KERNEL); |
350 | if (!smc) | 350 | if (!smc) |
351 | return NULL; | 351 | return ERR_PTR(-ENOMEM); |
352 | 352 | ||
353 | memcpy(&smc->sm, &ops_, sizeof(smc->sm)); | 353 | memcpy(&smc->sm, &ops_, sizeof(smc->sm)); |
354 | r = ca_create(&smc->old_counts, sm); | 354 | r = ca_create(&smc->old_counts, sm); |
355 | if (r) { | 355 | if (r) { |
356 | kfree(smc); | 356 | kfree(smc); |
357 | return NULL; | 357 | return ERR_PTR(r); |
358 | } | 358 | } |
359 | 359 | ||
360 | r = ca_create(&smc->counts, sm); | 360 | r = ca_create(&smc->counts, sm); |
361 | if (r) { | 361 | if (r) { |
362 | ca_destroy(&smc->old_counts); | 362 | ca_destroy(&smc->old_counts); |
363 | kfree(smc); | 363 | kfree(smc); |
364 | return NULL; | 364 | return ERR_PTR(r); |
365 | } | 365 | } |
366 | 366 | ||
367 | smc->real_sm = sm; | 367 | smc->real_sm = sm; |
@@ -371,7 +371,7 @@ struct dm_space_map *dm_sm_checker_create(struct dm_space_map *sm) | |||
371 | ca_destroy(&smc->counts); | 371 | ca_destroy(&smc->counts); |
372 | ca_destroy(&smc->old_counts); | 372 | ca_destroy(&smc->old_counts); |
373 | kfree(smc); | 373 | kfree(smc); |
374 | return NULL; | 374 | return ERR_PTR(r); |
375 | } | 375 | } |
376 | 376 | ||
377 | r = ca_commit(&smc->old_counts, &smc->counts); | 377 | r = ca_commit(&smc->old_counts, &smc->counts); |
@@ -379,7 +379,7 @@ struct dm_space_map *dm_sm_checker_create(struct dm_space_map *sm) | |||
379 | ca_destroy(&smc->counts); | 379 | ca_destroy(&smc->counts); |
380 | ca_destroy(&smc->old_counts); | 380 | ca_destroy(&smc->old_counts); |
381 | kfree(smc); | 381 | kfree(smc); |
382 | return NULL; | 382 | return ERR_PTR(r); |
383 | } | 383 | } |
384 | 384 | ||
385 | return &smc->sm; | 385 | return &smc->sm; |
@@ -391,25 +391,25 @@ struct dm_space_map *dm_sm_checker_create_fresh(struct dm_space_map *sm) | |||
391 | int r; | 391 | int r; |
392 | struct sm_checker *smc; | 392 | struct sm_checker *smc; |
393 | 393 | ||
394 | if (!sm) | 394 | if (IS_ERR_OR_NULL(sm)) |
395 | return NULL; | 395 | return ERR_PTR(-EINVAL); |
396 | 396 | ||
397 | smc = kmalloc(sizeof(*smc), GFP_KERNEL); | 397 | smc = kmalloc(sizeof(*smc), GFP_KERNEL); |
398 | if (!smc) | 398 | if (!smc) |
399 | return NULL; | 399 | return ERR_PTR(-ENOMEM); |
400 | 400 | ||
401 | memcpy(&smc->sm, &ops_, sizeof(smc->sm)); | 401 | memcpy(&smc->sm, &ops_, sizeof(smc->sm)); |
402 | r = ca_create(&smc->old_counts, sm); | 402 | r = ca_create(&smc->old_counts, sm); |
403 | if (r) { | 403 | if (r) { |
404 | kfree(smc); | 404 | kfree(smc); |
405 | return NULL; | 405 | return ERR_PTR(r); |
406 | } | 406 | } |
407 | 407 | ||
408 | r = ca_create(&smc->counts, sm); | 408 | r = ca_create(&smc->counts, sm); |
409 | if (r) { | 409 | if (r) { |
410 | ca_destroy(&smc->old_counts); | 410 | ca_destroy(&smc->old_counts); |
411 | kfree(smc); | 411 | kfree(smc); |
412 | return NULL; | 412 | return ERR_PTR(r); |
413 | } | 413 | } |
414 | 414 | ||
415 | smc->real_sm = sm; | 415 | smc->real_sm = sm; |