diff options
| author |   Xin Long <lucien.xin@gmail.com> | 2018-08-27 18:40:18 +0800 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2018-08-27 15:13:17 -0700 |
| commit | 834539e69a5fe2aab33cc777ccfd4a4fcc5b9770 (patch) | |
| tree | 73f9c5e068b36bd6d430dc320c7e3879c8d42bb4 /net/sctp | |
| parent | sctp: hold transport before accessing its asoc in sctp_transport_get_next (diff) | |
| download | linux-dev-834539e69a5fe2aab33cc777ccfd4a4fcc5b9770.tar.xz linux-dev-834539e69a5fe2aab33cc777ccfd4a4fcc5b9770.zip | |
sctp: remove useless start_fail from sctp_ht_iter in proc
After changing rhashtable_walk_start to return void, start_fail would
never be set other value than 0, and the checking for start_fail is
pointless, so remove it.
Fixes: 97a6ec4ac021 ("rhashtable: Change rhashtable_walk_start to return void")
Signed-off-by: Xin Long <lucien.xin@gmail.com>
Acked-by: Neil Horman <nhorman@tuxdriver.com>
Acked-by: Marcelo Ricardo Leitner <marcelo.leitner@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/sctp')
| -rw-r--r-- | net/sctp/proc.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/net/sctp/proc.c b/net/sctp/proc.c index 4d6f1c8d6659..a644292f9faf 100644 --- a/net/sctp/proc.c +++ b/net/sctp/proc.c @@ -215,7 +215,6 @@ static const struct seq_operations sctp_eps_ops = { struct sctp_ht_iter { struct seq_net_private p; struct rhashtable_iter hti; - int start_fail; }; static void *sctp_transport_seq_start(struct seq_file *seq, loff_t *pos) @@ -224,7 +223,6 @@ static void *sctp_transport_seq_start(struct seq_file *seq, loff_t *pos) sctp_transport_walk_start(&iter->hti); - iter->start_fail = 0; return sctp_transport_get_idx(seq_file_net(seq), &iter->hti, *pos); } @@ -232,8 +230,6 @@ static void sctp_transport_seq_stop(struct seq_file *seq, void *v) { struct sctp_ht_iter *iter = seq->private; - if (iter->start_fail) - return; sctp_transport_walk_stop(&iter->hti); } |