* [PATCH bluetooth-next] 6lowpan: iphc: fix handling of link-local compression
@ 2016-03-16 12:52 Alexander Aring
2016-03-16 13:39 ` Jukka Rissanen
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Alexander Aring @ 2016-03-16 12:52 UTC (permalink / raw)
To: linux-wpan; +Cc: linux-bluetooth, kernel, jukka.rissanen, Alexander Aring
This patch fixes handling in case of link-local address compression. A
IPv6 link-local address is defined as fe80::/10 prefix which is also
what ipv6_addr_type checks for link-local addresses.
But IPHC compression for link-local addresses are for fe80::/64 types
only. This patch adds additional checks for zero padded bits in case of
link-local address compression to match on a fe80::/64 address only.
Signed-off-by: Alexander Aring <aar@pengutronix.de>
---
net/6lowpan/iphc.c | 11 +++++++++--
1 file changed, 9 insertions(+), 2 deletions(-)
diff --git a/net/6lowpan/iphc.c b/net/6lowpan/iphc.c
index 99bb22a..68c80f3 100644
--- a/net/6lowpan/iphc.c
+++ b/net/6lowpan/iphc.c
@@ -148,6 +148,11 @@
(((a)->s6_addr16[6]) == 0) && \
(((a)->s6_addr[14]) == 0))
+#define lowpan_is_linklocal_zero_padded(a) \
+ (!(hdr->saddr.s6_addr[1] & 0x3f) && \
+ !hdr->saddr.s6_addr16[1] && \
+ !hdr->saddr.s6_addr32[1])
+
#define LOWPAN_IPHC_CID_DCI(cid) (cid & 0x0f)
#define LOWPAN_IPHC_CID_SCI(cid) ((cid & 0xf0) >> 4)
@@ -1101,7 +1106,8 @@ int lowpan_header_compress(struct sk_buff *skb, const struct net_device *dev,
true);
iphc1 |= LOWPAN_IPHC_SAC;
} else {
- if (ipv6_saddr_type & IPV6_ADDR_LINKLOCAL) {
+ if (ipv6_saddr_type & IPV6_ADDR_LINKLOCAL &&
+ lowpan_is_linklocal_zero_padded(hdr->saddr)) {
iphc1 |= lowpan_compress_addr_64(&hc_ptr,
&hdr->saddr,
saddr, true);
@@ -1135,7 +1141,8 @@ int lowpan_header_compress(struct sk_buff *skb, const struct net_device *dev,
false);
iphc1 |= LOWPAN_IPHC_DAC;
} else {
- if (ipv6_daddr_type & IPV6_ADDR_LINKLOCAL) {
+ if (ipv6_daddr_type & IPV6_ADDR_LINKLOCAL &&
+ lowpan_is_linklocal_zero_padded(hdr->daddr)) {
iphc1 |= lowpan_compress_addr_64(&hc_ptr,
&hdr->daddr,
daddr, false);
--
2.7.3
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH bluetooth-next] 6lowpan: iphc: fix handling of link-local compression
2016-03-16 12:52 [PATCH bluetooth-next] 6lowpan: iphc: fix handling of link-local compression Alexander Aring
@ 2016-03-16 13:39 ` Jukka Rissanen
2016-03-16 15:44 ` Stefan Schmidt
2016-04-08 17:32 ` Marcel Holtmann
2 siblings, 0 replies; 4+ messages in thread
From: Jukka Rissanen @ 2016-03-16 13:39 UTC (permalink / raw)
To: Alexander Aring, linux-wpan; +Cc: linux-bluetooth, kernel
Hi Alex,
On Wed, 2016-03-16 at 13:52 +0100, Alexander Aring wrote:
> This patch fixes handling in case of link-local address compression.
> A
> IPv6 link-local address is defined as fe80::/10 prefix which is also
> what ipv6_addr_type checks for link-local addresses.
>
> But IPHC compression for link-local addresses are for fe80::/64 types
> only. This patch adds additional checks for zero padded bits in case
> of
> link-local address compression to match on a fe80::/64 address only.
Makes sense although this is probably very unlikely thing to happen.
Acked-by: Jukka Rissanen <jukka.rissanen@linux.intel.com>
> Signed-off-by: Alexander Aring <aar@pengutronix.de>
> ---
> net/6lowpan/iphc.c | 11 +++++++++--
> 1 file changed, 9 insertions(+), 2 deletions(-)
>
> diff --git a/net/6lowpan/iphc.c b/net/6lowpan/iphc.c
> index 99bb22a..68c80f3 100644
> --- a/net/6lowpan/iphc.c
> +++ b/net/6lowpan/iphc.c
> @@ -148,6 +148,11 @@
> (((a)->s6_addr16[6]) == 0) && \
> (((a)->s6_addr[14]) == 0))
>
> +#define lowpan_is_linklocal_zero_padded(a) \
> + (!(hdr->saddr.s6_addr[1] & 0x3f) && \
> + !hdr->saddr.s6_addr16[1] && \
> + !hdr->saddr.s6_addr32[1])
> +
> #define LOWPAN_IPHC_CID_DCI(cid) (cid & 0x0f)
> #define LOWPAN_IPHC_CID_SCI(cid) ((cid & 0xf0) >> 4)
>
> @@ -1101,7 +1106,8 @@ int lowpan_header_compress(struct sk_buff *skb,
> const struct net_device *dev,
> true);
> iphc1 |= LOWPAN_IPHC_SAC;
> } else {
> - if (ipv6_saddr_type & IPV6_ADDR_LINKLOCAL) {
> + if (ipv6_saddr_type & IPV6_ADDR_LINKLOCAL &&
> + lowpan_is_linklocal_zero_padded(hdr-
> >saddr)) {
> iphc1 |=
> lowpan_compress_addr_64(&hc_ptr,
> &hd
> r->saddr,
> sad
> dr, true);
> @@ -1135,7 +1141,8 @@ int lowpan_header_compress(struct sk_buff *skb,
> const struct net_device *dev,
> false);
> iphc1 |= LOWPAN_IPHC_DAC;
> } else {
> - if (ipv6_daddr_type & IPV6_ADDR_LINKLOCAL) {
> + if (ipv6_daddr_type & IPV6_ADDR_LINKLOCAL &&
> + lowpan_is_linklocal_zero_padded(hdr-
> >daddr)) {
> iphc1 |=
> lowpan_compress_addr_64(&hc_ptr,
> &hd
> r->daddr,
> dad
> dr, false);
Cheers,
Jukka
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH bluetooth-next] 6lowpan: iphc: fix handling of link-local compression
2016-03-16 12:52 [PATCH bluetooth-next] 6lowpan: iphc: fix handling of link-local compression Alexander Aring
2016-03-16 13:39 ` Jukka Rissanen
@ 2016-03-16 15:44 ` Stefan Schmidt
2016-04-08 17:32 ` Marcel Holtmann
2 siblings, 0 replies; 4+ messages in thread
From: Stefan Schmidt @ 2016-03-16 15:44 UTC (permalink / raw)
To: Alexander Aring, linux-wpan; +Cc: linux-bluetooth, kernel, jukka.rissanen
Hello.
On 16/03/16 13:52, Alexander Aring wrote:
> This patch fixes handling in case of link-local address compression. A
> IPv6 link-local address is defined as fe80::/10 prefix which is also
> what ipv6_addr_type checks for link-local addresses.
>
> But IPHC compression for link-local addresses are for fe80::/64 types
> only. This patch adds additional checks for zero padded bits in case of
> link-local address compression to match on a fe80::/64 address only.
>
> Signed-off-by: Alexander Aring <aar@pengutronix.de>
> ---
> net/6lowpan/iphc.c | 11 +++++++++--
> 1 file changed, 9 insertions(+), 2 deletions(-)
>
> diff --git a/net/6lowpan/iphc.c b/net/6lowpan/iphc.c
> index 99bb22a..68c80f3 100644
> --- a/net/6lowpan/iphc.c
> +++ b/net/6lowpan/iphc.c
> @@ -148,6 +148,11 @@
> (((a)->s6_addr16[6]) == 0) && \
> (((a)->s6_addr[14]) == 0))
>
> +#define lowpan_is_linklocal_zero_padded(a) \
> + (!(hdr->saddr.s6_addr[1] & 0x3f) && \
> + !hdr->saddr.s6_addr16[1] && \
> + !hdr->saddr.s6_addr32[1])
> +
> #define LOWPAN_IPHC_CID_DCI(cid) (cid & 0x0f)
> #define LOWPAN_IPHC_CID_SCI(cid) ((cid & 0xf0) >> 4)
>
> @@ -1101,7 +1106,8 @@ int lowpan_header_compress(struct sk_buff *skb, const struct net_device *dev,
> true);
> iphc1 |= LOWPAN_IPHC_SAC;
> } else {
> - if (ipv6_saddr_type & IPV6_ADDR_LINKLOCAL) {
> + if (ipv6_saddr_type & IPV6_ADDR_LINKLOCAL &&
> + lowpan_is_linklocal_zero_padded(hdr->saddr)) {
> iphc1 |= lowpan_compress_addr_64(&hc_ptr,
> &hdr->saddr,
> saddr, true);
> @@ -1135,7 +1141,8 @@ int lowpan_header_compress(struct sk_buff *skb, const struct net_device *dev,
> false);
> iphc1 |= LOWPAN_IPHC_DAC;
> } else {
> - if (ipv6_daddr_type & IPV6_ADDR_LINKLOCAL) {
> + if (ipv6_daddr_type & IPV6_ADDR_LINKLOCAL &&
> + lowpan_is_linklocal_zero_padded(hdr->daddr)) {
> iphc1 |= lowpan_compress_addr_64(&hc_ptr,
> &hdr->daddr,
> daddr, false);
Nicely spotted.
Reviewed-by: Stefan Schmidt <stefan@osg.samsung.com>
regards
Stefan Schmidt
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH bluetooth-next] 6lowpan: iphc: fix handling of link-local compression
2016-03-16 12:52 [PATCH bluetooth-next] 6lowpan: iphc: fix handling of link-local compression Alexander Aring
2016-03-16 13:39 ` Jukka Rissanen
2016-03-16 15:44 ` Stefan Schmidt
@ 2016-04-08 17:32 ` Marcel Holtmann
2 siblings, 0 replies; 4+ messages in thread
From: Marcel Holtmann @ 2016-04-08 17:32 UTC (permalink / raw)
To: Alexander Aring; +Cc: linux-wpan, linux-bluetooth, kernel, jukka.rissanen
Hi Alex,
> This patch fixes handling in case of link-local address compression. A
> IPv6 link-local address is defined as fe80::/10 prefix which is also
> what ipv6_addr_type checks for link-local addresses.
>
> But IPHC compression for link-local addresses are for fe80::/64 types
> only. This patch adds additional checks for zero padded bits in case of
> link-local address compression to match on a fe80::/64 address only.
>
> Signed-off-by: Alexander Aring <aar@pengutronix.de>
> ---
> net/6lowpan/iphc.c | 11 +++++++++--
> 1 file changed, 9 insertions(+), 2 deletions(-)
patch has been applied to bluetooth-next tree.
Regards
Marcel
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2016-04-08 17:32 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-03-16 12:52 [PATCH bluetooth-next] 6lowpan: iphc: fix handling of link-local compression Alexander Aring
2016-03-16 13:39 ` Jukka Rissanen
2016-03-16 15:44 ` Stefan Schmidt
2016-04-08 17:32 ` Marcel Holtmann
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.