diff options
| author | Vasily Averin <vvs@virtuozzo.com> | 2020-01-23 10:11:20 +0300 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2020-01-24 11:42:18 +0100 |
| commit | 8bf7092021f283944f0c5f4c364853201c45c611 (patch) | |
| tree | 05b13fe1a40bd2762b42c42ef5aeaec52d5cdc69 /net/atm | |
| parent | 66018a102f7756cf72db4d2704e1b93969d9d332 (diff) | |
| download | lwn-8bf7092021f283944f0c5f4c364853201c45c611.tar.gz lwn-8bf7092021f283944f0c5f4c364853201c45c611.zip | |
vcc_seq_next should increase position index
if seq_file .next fuction does not change position index,
read after some lseek can generate unexpected output.
https://bugzilla.kernel.org/show_bug.cgi?id=206283
Signed-off-by: Vasily Averin <vvs@virtuozzo.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/atm')
| -rw-r--r-- | net/atm/proc.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/net/atm/proc.c b/net/atm/proc.c index d79221fd4dae..c31896707313 100644 --- a/net/atm/proc.c +++ b/net/atm/proc.c @@ -134,8 +134,7 @@ static void vcc_seq_stop(struct seq_file *seq, void *v) static void *vcc_seq_next(struct seq_file *seq, void *v, loff_t *pos) { v = vcc_walk(seq, 1); - if (v) - (*pos)++; + (*pos)++; return v; } |