diff options
author | Wei Yongjun <yjwei@cn.fujitsu.com> | 2008-05-09 15:11:53 -0700 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-05-12 03:11:48 -0700 |
commit | c4492586a618d18e8a5343a04bad0ec606064846 (patch) | |
tree | cf9f5a38e3e9330edf61699f5c4cb8ae8876b879 /net | |
parent | 6e40a915de82e00d18f75941e531b40c4e0d94c4 (diff) | |
download | op-kernel-dev-c4492586a618d18e8a5343a04bad0ec606064846.zip op-kernel-dev-c4492586a618d18e8a5343a04bad0ec606064846.tar.gz |
sctp: Add address type check while process paramaters of ASCONF chunk
If socket is create by AF_INET type, add IPv6 address to asoc will cause
kernel panic while packet is transmitted on that transport.
This patch add address type check before process paramaters of ASCONF
chunk. If peer is not support this address type, return with error
invald parameter.
Signed-off-by: Wei Yongjun <yjwei@cn.fujitsu.com>
Signed-off-by: Vlad Yasevich <vladislav.yasevich@hp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net')
-rw-r--r-- | net/sctp/sm_make_chunk.c | 13 |
1 files changed, 13 insertions, 0 deletions
diff --git a/net/sctp/sm_make_chunk.c b/net/sctp/sm_make_chunk.c index 69a464f..6eeee53 100644 --- a/net/sctp/sm_make_chunk.c +++ b/net/sctp/sm_make_chunk.c @@ -2827,6 +2827,19 @@ static __be16 sctp_process_asconf_param(struct sctp_association *asoc, union sctp_addr addr; union sctp_addr_param *addr_param; + switch (addr_param->v4.param_hdr.type) { + case SCTP_PARAM_IPV6_ADDRESS: + if (!asoc->peer.ipv6_address) + return SCTP_ERROR_INV_PARAM; + break; + case SCTP_PARAM_IPV4_ADDRESS: + if (!asoc->peer.ipv4_address) + return SCTP_ERROR_INV_PARAM; + break; + default: + return SCTP_ERROR_INV_PARAM; + } + addr_param = (union sctp_addr_param *) ((void *)asconf_param + sizeof(sctp_addip_param_t)); |