diff options
| author |   Pavel Emelyanov <xemul@parallels.com> | 2011-12-15 02:43:27 +0000 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2011-12-16 13:48:27 -0500 |
| commit | aec8dc62f66199aef153d86e1f90d9c1d14696e3 (patch) | |
| tree | bdf927f62708cfa5b7a0561725ab76da8ac2ceb2 /net/core/sock_diag.c | |
| parent | sock_diag: Move the SOCK_DIAG_BY_FAMILY cmd declaration (diff) | |
| download | linux-dev-aec8dc62f66199aef153d86e1f90d9c1d14696e3.tar.xz linux-dev-aec8dc62f66199aef153d86e1f90d9c1d14696e3.zip | |
sock_diag: Fix module netlink aliases
I've made a mistake when fixing the sock_/inet_diag aliases :(
1. The sock_diag layer should request the family-based alias,
not just the IPPROTO_IP one;
2. The inet_diag layer should request for AF_INET+protocol alias,
not just the protocol one.
Thus fix this.
Signed-off-by: Pavel Emelyanov <xemul@parallels.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/core/sock_diag.c')
| -rw-r--r-- | net/core/sock_diag.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/core/sock_diag.c b/net/core/sock_diag.c index 9c27bcda59be..cee96f368108 100644 --- a/net/core/sock_diag.c +++ b/net/core/sock_diag.c @@ -64,7 +64,7 @@ static inline struct sock_diag_handler *sock_diag_lock_handler(int family) { if (sock_diag_handlers[family] == NULL) request_module("net-pf-%d-proto-%d-type-%d", PF_NETLINK, - NETLINK_SOCK_DIAG, IPPROTO_IP); + NETLINK_SOCK_DIAG, family); mutex_lock(&sock_diag_table_mutex); return sock_diag_handlers[family]; @@ -103,7 +103,7 @@ static int sock_diag_rcv_msg(struct sk_buff *skb, struct nlmsghdr *nlh) case DCCPDIAG_GETSOCK: if (inet_rcv_compat == NULL) request_module("net-pf-%d-proto-%d-type-%d", PF_NETLINK, - NETLINK_SOCK_DIAG, IPPROTO_IP); + NETLINK_SOCK_DIAG, AF_INET); mutex_lock(&sock_diag_table_mutex); if (inet_rcv_compat != NULL) |