Message ID | 20200724121753.16721-1-colin.king@canonical.com |
---|---|
State | Superseded |
Delegated to: | David Miller |
Headers | show |
Series | sctp: remove redundant initialization of variable status | expand |
On Fri, Jul 24, 2020 at 01:17:53PM +0100, Colin King wrote: > From: Colin Ian King <colin.king@canonical.com> > > The variable status is being initialized with a value that is never read > and it is being updated later with a new value. The initialization is > redundant and can be removed. > > Addresses-Coverity: ("Unused value") > Signed-off-by: Colin Ian King <colin.king@canonical.com> Acked-by: Marcelo Ricardo Leitner <marcelo.leitner@gmail.com> Are you willing to send another to patch to fix the var ordering in reverse christmass tree in there? > --- > net/sctp/protocol.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/net/sctp/protocol.c b/net/sctp/protocol.c > index 7ecaf7d575c0..a0448f7c64b9 100644 > --- a/net/sctp/protocol.c > +++ b/net/sctp/protocol.c > @@ -1368,7 +1368,7 @@ static struct pernet_operations sctp_ctrlsock_ops = { > static __init int sctp_init(void) > { > int i; > - int status = -EINVAL; > + int status; > unsigned long goal; > unsigned long limit; > unsigned long nr_pages = totalram_pages(); > -- > 2.27.0 >
On 24/07/2020 13:55, Marcelo Ricardo Leitner wrote: > On Fri, Jul 24, 2020 at 01:17:53PM +0100, Colin King wrote: >> From: Colin Ian King <colin.king@canonical.com> >> >> The variable status is being initialized with a value that is never read >> and it is being updated later with a new value. The initialization is >> redundant and can be removed. >> >> Addresses-Coverity: ("Unused value") >> Signed-off-by: Colin Ian King <colin.king@canonical.com> > > Acked-by: Marcelo Ricardo Leitner <marcelo.leitner@gmail.com> > > Are you willing to send another to patch to fix the var ordering in > reverse christmass tree in there? Sure, I'll send a V2 to re-order the vars. > >> --- >> net/sctp/protocol.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/net/sctp/protocol.c b/net/sctp/protocol.c >> index 7ecaf7d575c0..a0448f7c64b9 100644 >> --- a/net/sctp/protocol.c >> +++ b/net/sctp/protocol.c >> @@ -1368,7 +1368,7 @@ static struct pernet_operations sctp_ctrlsock_ops = { >> static __init int sctp_init(void) >> { >> int i; >> - int status = -EINVAL; >> + int status; >> unsigned long goal; >> unsigned long limit; >> unsigned long nr_pages = totalram_pages(); >> -- >> 2.27.0 >>
diff --git a/net/sctp/protocol.c b/net/sctp/protocol.c index 7ecaf7d575c0..a0448f7c64b9 100644 --- a/net/sctp/protocol.c +++ b/net/sctp/protocol.c @@ -1368,7 +1368,7 @@ static struct pernet_operations sctp_ctrlsock_ops = { static __init int sctp_init(void) { int i; - int status = -EINVAL; + int status; unsigned long goal; unsigned long limit; unsigned long nr_pages = totalram_pages();