We have never used this "err = PTR_ERR(p);" assignment and it annoys static checkers. Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> diff --git a/fs/seq_file.c b/fs/seq_file.c index 13e8c092d4d2..d6f82ce288f4 100644 --- a/fs/seq_file.c +++ b/fs/seq_file.c @@ -261,10 +261,8 @@ ssize_t seq_read(struct file *file, char __user *buf, size_t size, loff_t *ppos) size_t offs = m->count; loff_t next = pos; p = m->op->next(m, p, &next); - if (!p || IS_ERR(p)) { - err = PTR_ERR(p); + if (!p || IS_ERR(p)) break; - } err = m->op->show(m, p); if (seq_has_overflowed(m) || err) { m->count = offs;