diff options
author | Alexander Aring <aar@pengutronix.de> | 2016-02-26 09:06:07 +0100 |
---|---|---|
committer | Marcel Holtmann <marcel@holtmann.org> | 2016-02-26 09:08:15 +0100 |
commit | 2306f65637079c8922aec15b4ced75ea457e757b (patch) | |
tree | 059f638c4e1e1a3b38a5748e137cd0e47ce4d900 /net | |
parent | Bluetooth: hci_uart: Add Intel/AG6xx support (diff) | |
download | linux-2306f65637079c8922aec15b4ced75ea457e757b.tar.xz linux-2306f65637079c8922aec15b4ced75ea457e757b.zip |
6lowpan: iphc: fix invalid case handling
This patch fixes the return value in a case which should never occur.
Instead returning "-EINVAL" we return LOWPAN_IPHC_DAM_00 which is
invalid on context based addresses. Also change the WARN_ON_ONCE to
WARN_ONCE which was suggested by Dan Carpenter.
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Alexander Aring <aar@pengutronix.de>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
Diffstat (limited to 'net')
-rw-r--r-- | net/6lowpan/iphc.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/6lowpan/iphc.c b/net/6lowpan/iphc.c index 06287ac03211..72172514fea0 100644 --- a/net/6lowpan/iphc.c +++ b/net/6lowpan/iphc.c @@ -811,8 +811,8 @@ static u8 lowpan_compress_ctx_addr(u8 **hc_ptr, const struct in6_addr *ipaddr, goto out; } - WARN_ON_ONCE("context found but no address mode matched\n"); - return -EINVAL; + WARN_ONCE(1, "context found but no address mode matched\n"); + return LOWPAN_IPHC_DAM_00; out: if (sam) |