summaryrefslogtreecommitdiffstats
path: root/net
diff options
context:
space:
mode:
authorAlexander Aring <aar@pengutronix.de>2016-02-26 09:06:07 +0100
committerMarcel Holtmann <marcel@holtmann.org>2016-02-26 09:08:15 +0100
commit2306f65637079c8922aec15b4ced75ea457e757b (patch)
tree059f638c4e1e1a3b38a5748e137cd0e47ce4d900 /net
parent395174bb07c1dce58fbf2baa3a01bb69f5103c59 (diff)
downloadlinux-2306f65637079c8922aec15b4ced75ea457e757b.tar.gz
linux-2306f65637079c8922aec15b4ced75ea457e757b.tar.bz2
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.c4
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)