diff options
| author |   Cong Wang <xiyou.wangcong@gmail.com> | 2020-06-12 00:16:55 -0700 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2020-06-12 14:05:08 -0700 |
| commit | b65ce380b754e77fbfdcfc83fd6e29c8ceedf431 (patch) | |
| tree | 8a3a04539dbdafe4336223edb3b5da5684efdf77 /net/atm | |
| parent | Merge branch 'net-ipa-endpoint-configuration-fixes' (diff) | |
| download | linux-dev-b65ce380b754e77fbfdcfc83fd6e29c8ceedf431.tar.xz linux-dev-b65ce380b754e77fbfdcfc83fd6e29c8ceedf431.zip | |
genetlink: clean up family attributes allocations
genl_family_rcv_msg_attrs_parse() and genl_family_rcv_msg_attrs_free()
take a boolean parameter to determine whether allocate/free the family
attrs. This is unnecessary as we can just check family->parallel_ops.
More importantly, callers would not need to worry about pairing these
parameters correctly after this patch.
And this fixes a memory leak, as after commit c36f05559104
("genetlink: fix memory leaks in genl_family_rcv_msg_dumpit()")
we call genl_family_rcv_msg_attrs_parse() for both parallel and
non-parallel cases.
Fixes: c36f05559104 ("genetlink: fix memory leaks in genl_family_rcv_msg_dumpit()")
Reported-by: Ido Schimmel <idosch@idosch.org>
Signed-off-by: Cong Wang <xiyou.wangcong@gmail.com>
Reviewed-by: Ido Schimmel <idosch@mellanox.com>
Tested-by: Ido Schimmel <idosch@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/atm')
0 files changed, 0 insertions, 0 deletions