diff options
| author |   Michał Mirosław <mirq-linux@rere.qmqm.pl> | 2009-05-21 10:34:05 +0000 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2009-05-21 16:50:24 -0700 |
| commit | 7ae740df3a9c68622156476dca29991de664fae4 (patch) | |
| tree | c2c63d862b9dba337c1eeafc848189e849850346 /net/netlabel/netlabel_mgmt.c | |
| parent | ipvs: Use genl_register_family_with_ops() (diff) | |
| download | linux-dev-7ae740df3a9c68622156476dca29991de664fae4.tar.xz linux-dev-7ae740df3a9c68622156476dca29991de664fae4.zip | |
netlabel: Use genl_register_family_with_ops()
Use genl_register_family_with_ops() instead of a copy. This fixes genetlink
family leak on error path.
Signed-off-by: Michał Mirosław <mirq-linux@rere.qmqm.pl>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/netlabel/netlabel_mgmt.c')
| -rw-r--r-- | net/netlabel/netlabel_mgmt.c | 16 |
1 files changed, 2 insertions, 14 deletions
diff --git a/net/netlabel/netlabel_mgmt.c b/net/netlabel/netlabel_mgmt.c index 1821c5d50fb8..8203623e65ad 100644 --- a/net/netlabel/netlabel_mgmt.c +++ b/net/netlabel/netlabel_mgmt.c @@ -779,18 +779,6 @@ static struct genl_ops netlbl_mgmt_genl_ops[] = { */ int __init netlbl_mgmt_genl_init(void) { - int ret_val, i; - - ret_val = genl_register_family(&netlbl_mgmt_gnl_family); - if (ret_val != 0) - return ret_val; - - for (i = 0; i < ARRAY_SIZE(netlbl_mgmt_genl_ops); i++) { - ret_val = genl_register_ops(&netlbl_mgmt_gnl_family, - &netlbl_mgmt_genl_ops[i]); - if (ret_val != 0) - return ret_val; - } - - return 0; + return genl_register_family_with_ops(&netlbl_mgmt_gnl_family, + netlbl_mgmt_genl_ops, ARRAY_SIZE(netlbl_mgmt_genl_ops)); } |