Commit 89c6cc2c authored Aug 20, 2019 by Revanth Rajashekar Committed by Jens Axboe Aug 20, 2019

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>

parent 5cc23ed7

block/sed-opal.c

+0 −4

Original line number	Diff line number	Diff line
		@@ -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;