Source-Changes-HG archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
[src/trunk]: src/sys Use M_UNWRITABLE, no functional change.
details: https://anonhg.NetBSD.org/src/rev/79efc32bc653
branches: trunk
changeset: 318471:79efc32bc653
user: maxv <maxv%NetBSD.org@localhost>
date: Thu Apr 26 07:28:21 2018 +0000
description:
Use M_UNWRITABLE, no functional change.
diffstat:
sys/netinet/ip_icmp.c | 6 +++---
sys/netinet6/icmp6.c | 7 +++----
sys/netinet6/ip6_mroute.c | 14 +++++---------
3 files changed, 11 insertions(+), 16 deletions(-)
diffs (104 lines):
diff -r 99e5a3561c7c -r 79efc32bc653 sys/netinet/ip_icmp.c
--- a/sys/netinet/ip_icmp.c Thu Apr 26 07:01:38 2018 +0000
+++ b/sys/netinet/ip_icmp.c Thu Apr 26 07:28:21 2018 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: ip_icmp.c,v 1.168 2018/02/08 09:32:02 maxv Exp $ */
+/* $NetBSD: ip_icmp.c,v 1.169 2018/04/26 07:28:21 maxv Exp $ */
/*
* Copyright (c) 1998, 2000 The NetBSD Foundation, Inc.
@@ -94,7 +94,7 @@
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: ip_icmp.c,v 1.168 2018/02/08 09:32:02 maxv Exp $");
+__KERNEL_RCSID(0, "$NetBSD: ip_icmp.c,v 1.169 2018/04/26 07:28:21 maxv Exp $");
#ifdef _KERNEL_OPT
#include "opt_ipsec.h"
@@ -448,7 +448,7 @@
goto freeit;
}
i = hlen + min(icmplen, ICMP_ADVLENMIN);
- if ((m->m_len < i || M_READONLY(m)) && (m = m_pullup(m, i)) == NULL) {
+ if (M_UNWRITABLE(m, i) && (m = m_pullup(m, i)) == NULL) {
ICMP_STATINC(ICMP_STAT_TOOSHORT);
return;
}
diff -r 99e5a3561c7c -r 79efc32bc653 sys/netinet6/icmp6.c
--- a/sys/netinet6/icmp6.c Thu Apr 26 07:01:38 2018 +0000
+++ b/sys/netinet6/icmp6.c Thu Apr 26 07:28:21 2018 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: icmp6.c,v 1.230 2018/04/14 17:55:47 maxv Exp $ */
+/* $NetBSD: icmp6.c,v 1.231 2018/04/26 07:28:21 maxv Exp $ */
/* $KAME: icmp6.c,v 1.217 2001/06/20 15:03:29 jinmei Exp $ */
/*
@@ -62,7 +62,7 @@
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: icmp6.c,v 1.230 2018/04/14 17:55:47 maxv Exp $");
+__KERNEL_RCSID(0, "$NetBSD: icmp6.c,v 1.231 2018/04/26 07:28:21 maxv Exp $");
#ifdef _KERNEL_OPT
#include "opt_inet.h"
@@ -642,8 +642,7 @@
/* Give up local */
n = m;
m = NULL;
- } else if (M_READONLY(n) ||
- n->m_len < off + sizeof(struct icmp6_hdr)) {
+ } else if (M_UNWRITABLE(n, off + sizeof(struct icmp6_hdr))) {
struct mbuf *n0 = n;
/*
diff -r 99e5a3561c7c -r 79efc32bc653 sys/netinet6/ip6_mroute.c
--- a/sys/netinet6/ip6_mroute.c Thu Apr 26 07:01:38 2018 +0000
+++ b/sys/netinet6/ip6_mroute.c Thu Apr 26 07:28:21 2018 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: ip6_mroute.c,v 1.123 2018/03/21 14:23:54 roy Exp $ */
+/* $NetBSD: ip6_mroute.c,v 1.124 2018/04/26 07:28:21 maxv Exp $ */
/* $KAME: ip6_mroute.c,v 1.49 2001/07/25 09:21:18 jinmei Exp $ */
/*
@@ -117,7 +117,7 @@
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: ip6_mroute.c,v 1.123 2018/03/21 14:23:54 roy Exp $");
+__KERNEL_RCSID(0, "$NetBSD: ip6_mroute.c,v 1.124 2018/04/26 07:28:21 maxv Exp $");
#ifdef _KERNEL_OPT
#include "opt_inet.h"
@@ -1131,8 +1131,7 @@
* Pullup packet header if needed before storing it,
* as other references may modify it in the meantime.
*/
- if (mb0 &&
- (M_READONLY(mb0) || mb0->m_len < sizeof(struct ip6_hdr)))
+ if (mb0 && M_UNWRITABLE(mb0, sizeof(struct ip6_hdr)))
mb0 = m_pullup(mb0, sizeof(struct ip6_hdr));
if (mb0 == NULL) {
free(rte, M_MRTABLE);
@@ -1416,9 +1415,7 @@
struct omrt6msg *oim;
mm = m_copy(m, 0, sizeof(struct ip6_hdr));
- if (mm &&
- (M_READONLY(mm) ||
- mm->m_len < sizeof(struct ip6_hdr)))
+ if (mm && M_UNWRITABLE(mm, sizeof(struct ip6_hdr)))
mm = m_pullup(mm, sizeof(struct ip6_hdr));
if (mm == NULL)
return ENOBUFS;
@@ -1552,8 +1549,7 @@
* so that ip6_output() only scribbles on the copy.
*/
mb_copy = m_copy(m, 0, M_COPYALL);
- if (mb_copy &&
- (M_READONLY(mb_copy) || mb_copy->m_len < sizeof(struct ip6_hdr)))
+ if (mb_copy && M_UNWRITABLE(mb_copy, sizeof(struct ip6_hdr)))
mb_copy = m_pullup(mb_copy, sizeof(struct ip6_hdr));
if (mb_copy == NULL) {
splx(s);
Home |
Main Index |
Thread Index |
Old Index