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_cipso_v4.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_cipso_v4.c')
-rw-r--r-- | net/netlabel/netlabel_cipso_v4.c | 16 |
1 files changed, 2 insertions, 14 deletions
diff --git a/net/netlabel/netlabel_cipso_v4.c b/net/netlabel/netlabel_cipso_v4.c index bf1ab1a6790d..e639298bc9c8 100644 --- a/net/netlabel/netlabel_cipso_v4.c +++ b/net/netlabel/netlabel_cipso_v4.c | |||
@@ -785,18 +785,6 @@ static struct genl_ops netlbl_cipsov4_ops[] = { | |||
785 | */ | 785 | */ |
786 | int __init netlbl_cipsov4_genl_init(void) | 786 | int __init netlbl_cipsov4_genl_init(void) |
787 | { | 787 | { |
788 | int ret_val, i; | 788 | return genl_register_family_with_ops(&netlbl_cipsov4_gnl_family, |
789 | 789 | netlbl_cipsov4_ops, ARRAY_SIZE(netlbl_cipsov4_ops)); | |
790 | ret_val = genl_register_family(&netlbl_cipsov4_gnl_family); | ||
791 | if (ret_val != 0) | ||
792 | return ret_val; | ||
793 | |||
794 | for (i = 0; i < ARRAY_SIZE(netlbl_cipsov4_ops); i++) { | ||
795 | ret_val = genl_register_ops(&netlbl_cipsov4_gnl_family, | ||
796 | &netlbl_cipsov4_ops[i]); | ||
797 | if (ret_val != 0) | ||
798 | return ret_val; | ||
799 | } | ||
800 | |||
801 | return 0; | ||
802 | } | 790 | } |