diff options
author | Sudip Mukherjee <sudipm.mukherjee@gmail.com> | 2014-11-07 17:48:34 +0530 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2014-11-07 09:22:46 -0800 |
commit | f9b64692e13b2f0d2a393e41972aa317f34a3e46 (patch) | |
tree | 2200326530e9e82f64eb54af3bde5d577e802c35 /drivers/staging/unisys | |
parent | staging: unisys: visorchannel: Add braces to else arm of if/else statement (diff) | |
download | linux-dev-f9b64692e13b2f0d2a393e41972aa317f34a3e46.tar.xz linux-dev-f9b64692e13b2f0d2a393e41972aa317f34a3e46.zip |
staging: unisys: unneeded NULL check
the NULL check for memregion is not required as it has already been
checked for NULL after kzalloc. so we can reach this part of the code
only if memregion is not NULL.
Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/unisys')
-rw-r--r-- | drivers/staging/unisys/visorutil/memregion_direct.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/staging/unisys/visorutil/memregion_direct.c b/drivers/staging/unisys/visorutil/memregion_direct.c index 26faad470705..33522cc8c22c 100644 --- a/drivers/staging/unisys/visorutil/memregion_direct.c +++ b/drivers/staging/unisys/visorutil/memregion_direct.c @@ -57,10 +57,8 @@ visor_memregion_create(HOSTADDRESS physaddr, ulong nbytes) rc = memregion; cleanup: if (rc == NULL) { - if (memregion != NULL) { - visor_memregion_destroy(memregion); - memregion = NULL; - } + visor_memregion_destroy(memregion); + memregion = NULL; } return rc; } |