diff options
| author | Marcelo Ricardo Leitner <marcelo.leitner@gmail.com> | 2016-12-28 09:26:34 -0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2016-12-28 14:06:31 -0500 |
| commit | 66b91d2cd0344c417194596ef6e387e52be69e57 (patch) | |
| tree | d5fd1bee8cd7226b9b255a0ef2255c37bb72853d /net/sctp/sm_statefuns.c | |
| parent | 0630c56e40b0bcca299d3b4c20ffcfddbe6a0218 (diff) | |
| download | lwn-66b91d2cd0344c417194596ef6e387e52be69e57.tar.gz lwn-66b91d2cd0344c417194596ef6e387e52be69e57.zip | |
sctp: remove return value from sctp_packet_init/config
There is no reason to use this cascading. It doesn't add anything.
Let's remove it and simplify.
Signed-off-by: Marcelo Ricardo Leitner <marcelo.leitner@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/sctp/sm_statefuns.c')
| -rw-r--r-- | net/sctp/sm_statefuns.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/net/sctp/sm_statefuns.c b/net/sctp/sm_statefuns.c index a95915ef9dba..9a223d5b2314 100644 --- a/net/sctp/sm_statefuns.c +++ b/net/sctp/sm_statefuns.c @@ -6032,8 +6032,9 @@ static struct sctp_packet *sctp_ootb_pkt_new(struct net *net, sctp_transport_route(transport, (union sctp_addr *)&chunk->dest, sctp_sk(net->sctp.ctl_sock)); - packet = sctp_packet_init(&transport->packet, transport, sport, dport); - packet = sctp_packet_config(packet, vtag, 0); + packet = &transport->packet; + sctp_packet_init(packet, transport, sport, dport); + sctp_packet_config(packet, vtag, 0); return packet; |