diff options
author | Mikulas Patocka <mpatocka@redhat.com> | 2015-11-23 19:11:32 -0500 |
---|---|---|
committer | Mike Snitzer <snitzer@redhat.com> | 2015-12-10 10:38:57 -0500 |
commit | f98c8f797021e78ba27143ecd39c51995c47aa8b (patch) | |
tree | 790bba6c2e7acae814300fe5cae09fa01e3d9080 /drivers/md | |
parent | 313c9b97361fca3a606fc74de86bee6dc7988bca (diff) | |
download | linux-f98c8f797021e78ba27143ecd39c51995c47aa8b.tar.bz2 |
dm bufio: return NULL to improve code clarity
A small code cleanup in new_read() - return NULL instead of b (although
b is NULL at this point). This function is not returning pointer to the
buffer, it is returning a pointer to the bufffer's data, thus it makes
no sense to return the variable b.
Signed-off-by: Mikulas Patocka <mpatocka@redhat.com>
Signed-off-by: Mike Snitzer <snitzer@redhat.com>
Diffstat (limited to 'drivers/md')
-rw-r--r-- | drivers/md/dm-bufio.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/md/dm-bufio.c b/drivers/md/dm-bufio.c index 2dd33085b331..7734298bff3c 100644 --- a/drivers/md/dm-bufio.c +++ b/drivers/md/dm-bufio.c @@ -1068,7 +1068,7 @@ static void *new_read(struct dm_bufio_client *c, sector_t block, __flush_write_list(&write_list); if (!b) - return b; + return NULL; if (need_submit) submit_io(b, READ, b->block, read_endio); |