diff options
| author | Jens Axboe <axboe@kernel.dk> | 2021-07-09 08:20:28 -0600 |
|---|---|---|
| committer | Jens Axboe <axboe@kernel.dk> | 2021-07-09 08:20:28 -0600 |
| commit | 9ce85ef2cb5c738754837a6937e120694cde33c9 (patch) | |
| tree | 8b42add96f803e2ce934a01b4adcac21a296c571 | |
| parent | 8f487ef2cbb2d4f6ca8c113d70da63baaf68c91a (diff) | |
| download | linux-amlogic-9ce85ef2cb5c738754837a6937e120694cde33c9.tar.gz | |
io_uring: remove dead non-zero 'poll' check
Colin reports that Coverity complains about checking for poll being
non-zero after having dereferenced it multiple times. This is a valid
complaint, and actually a leftover from back when this code was based
on the aio poll code.
Kill the redundant check.
Link: https://lore.kernel.org/io-uring/fe70c532-e2a7-3722-58a1-0fa4e5c5ff2c@canonical.com/
Reported-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
| -rw-r--r-- | fs/io_uring.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/io_uring.c b/fs/io_uring.c index 7167c61c6d1b82..d94fb5835a20e8 100644 --- a/fs/io_uring.c +++ b/fs/io_uring.c @@ -4956,7 +4956,7 @@ static int io_poll_double_wake(struct wait_queue_entry *wait, unsigned mode, list_del_init(&wait->entry); - if (poll && poll->head) { + if (poll->head) { bool done; spin_lock(&poll->head->lock); |