diff options
author | Michał Mirosław <mirq-linux@rere.qmqm.pl> | 2009-05-21 06:34:05 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2009-05-21 19:50:24 -0400 |
commit | 7ae740df3a9c68622156476dca29991de664fae4 (patch) | |
tree | c2c63d862b9dba337c1eeafc848189e849850346 /net/netlabel/netlabel_unlabeled.c | |
parent | 8f698d54532172de3591af1e9394dc669f29e6ca (diff) |
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_unlabeled.c')
-rw-r--r-- | net/netlabel/netlabel_unlabeled.c | 16 |
1 files changed, 2 insertions, 14 deletions
diff --git a/net/netlabel/netlabel_unlabeled.c b/net/netlabel/netlabel_unlabeled.c index f3c5c68c6848..fb357f010189 100644 --- a/net/netlabel/netlabel_unlabeled.c +++ b/net/netlabel/netlabel_unlabeled.c | |||
@@ -1478,20 +1478,8 @@ static struct genl_ops netlbl_unlabel_genl_ops[] = { | |||
1478 | */ | 1478 | */ |
1479 | int __init netlbl_unlabel_genl_init(void) | 1479 | int __init netlbl_unlabel_genl_init(void) |
1480 | { | 1480 | { |
1481 | int ret_val, i; | 1481 | return genl_register_family_with_ops(&netlbl_unlabel_gnl_family, |
1482 | 1482 | netlbl_unlabel_genl_ops, ARRAY_SIZE(netlbl_unlabel_genl_ops)); | |
1483 | ret_val = genl_register_family(&netlbl_unlabel_gnl_family); | ||
1484 | if (ret_val != 0) | ||
1485 | return ret_val; | ||
1486 | |||
1487 | for (i = 0; i < ARRAY_SIZE(netlbl_unlabel_genl_ops); i++) { | ||
1488 | ret_val = genl_register_ops(&netlbl_unlabel_gnl_family, | ||
1489 | &netlbl_unlabel_genl_ops[i]); | ||
1490 | if (ret_val != 0) | ||
1491 | return ret_val; | ||
1492 | } | ||
1493 | |||
1494 | return 0; | ||
1495 | } | 1483 | } |
1496 | 1484 | ||
1497 | /* | 1485 | /* |