diff options
| author |   Revanth Rajashekar <revanth.rajashekar@intel.com> | 2019-08-20 09:30:50 -0600 |
|---|---|---|
| committer |   Jens Axboe <axboe@kernel.dk> | 2019-08-20 09:33:21 -0600 |
| commit | 89c6cc2cab7e5090dc85ce0162ce92903b3aac5d (patch) | |
| tree | 68ecd4ab58448009f2aea4678e3e07e5d99d5e21 /block | |
| parent | block: sed-opal: Add/remove spaces (diff) | |
| download | linux-dev-89c6cc2cab7e5090dc85ce0162ce92903b3aac5d.tar.xz linux-dev-89c6cc2cab7e5090dc85ce0162ce92903b3aac5d.zip | |
block: sed-opal: Remove always false conditional statement
In the function 'response_parse', num_entries will never be 0 as
slen is checked for 0. Hence, the condition 'if (num_entries == 0)'
can never be true.
Signed-off-by: Revanth Rajashekar <revanth.rajashekar@intel.com>
Reviewed-by: Scott Bauer <sbauer@plzdonthack.me>
Reviewed-by: Jon Derrick <jonathan.derrick@intel.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'block')
| -rw-r--r-- | block/sed-opal.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/block/sed-opal.c b/block/sed-opal.c index d442f29e84f1..4e95a9792162 100644 --- a/block/sed-opal.c +++ b/block/sed-opal.c @@ -905,10 +905,6 @@ static int response_parse(const u8 *buf, size_t length, num_entries++; } - if (num_entries == 0) { - pr_debug("Couldn't parse response.\n"); - return -EINVAL; - } resp->num = num_entries; return 0; |