diff options
| author | Daniel Borkmann <dborkman@redhat.com> | 2013-04-15 03:27:17 +0000 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2013-04-15 14:11:37 -0400 |
| commit | ff2266cddd69f5e0c9d5121ed9218d2f694406cc (patch) | |
| tree | f8d3a419ff1413613a1f05d790cac7dffc75dbc4 /include | |
| parent | 1e8edc2ab35da30b08b008c26822ec956052bf4b (diff) | |
| download | linux-ff2266cddd69f5e0c9d5121ed9218d2f694406cc.tar.bz2 | |
net: sctp: remove sctp_ep_common struct member 'malloced'
There is actually no need to keep this member in the structure, because
after init it's always 1 anyway, thus always kfree called. This seems to
be an ancient leftover from the very initial implementation from 2.5
times. Only in case the initialization of an association fails, we leave
base.malloced as 0, but we nevertheless kfree it in the error path in
sctp_association_new().
Signed-off-by: Daniel Borkmann <dborkman@redhat.com>
Acked-by: Vlad Yasevich <vyasevich@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'include')
| -rw-r--r-- | include/net/sctp/structs.h | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/include/net/sctp/structs.h b/include/net/sctp/structs.h index 0e0f9d2322e3..3e80eedab17d 100644 --- a/include/net/sctp/structs.h +++ b/include/net/sctp/structs.h @@ -1174,11 +1174,9 @@ struct sctp_ep_common { /* Some fields to help us manage this object. * refcnt - Reference count access to this object. * dead - Do not attempt to use this object. - * malloced - Do we need to kfree this object? */ atomic_t refcnt; char dead; - char malloced; /* What socket does this endpoint belong to? */ struct sock *sk; |