summaryrefslogtreecommitdiff
path: root/sys
diff options
context:
space:
mode:
authorchristos <christos@NetBSD.org>2013-01-11 19:01:36 +0000
committerchristos <christos@NetBSD.org>2013-01-11 19:01:36 +0000
commit891f436cdc2df396e5c5cbfe5d8af27437d690a3 (patch)
treec6acfe63553e327cc96134de60d868f223e00a3c /sys
parentc07ede2251b2ebdc6a8f39f549f28834909b74cd (diff)
Use copyin/copyout and linux-specific ifreq structures (they are the same
as the netbsd ones, but this disconnects them)
Diffstat (limited to 'sys')
-rw-r--r--sys/compat/linux/common/linux_socket.c20
-rw-r--r--sys/compat/linux/common/linux_sockio.h13
-rw-r--r--sys/compat/linux32/common/linux32_socket.c20
-rw-r--r--sys/compat/linux32/common/linux32_sockio.h13
4 files changed, 48 insertions, 18 deletions
diff --git a/sys/compat/linux/common/linux_socket.c b/sys/compat/linux/common/linux_socket.c
index c9dfb5ef30d..d7a1cd73e2e 100644
--- a/sys/compat/linux/common/linux_socket.c
+++ b/sys/compat/linux/common/linux_socket.c
@@ -1,4 +1,4 @@
-/* $NetBSD: linux_socket.c,v 1.114 2012/06/20 15:03:18 christos Exp $ */
+/* $NetBSD: linux_socket.c,v 1.115 2013/01/11 19:01:36 christos Exp $ */
/*-
* Copyright (c) 1995, 1998, 2008 The NetBSD Foundation, Inc.
@@ -35,7 +35,7 @@
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: linux_socket.c,v 1.114 2012/06/20 15:03:18 christos Exp $");
+__KERNEL_RCSID(0, "$NetBSD: linux_socket.c,v 1.115 2013/01/11 19:01:36 christos Exp $");
#if defined(_KERNEL_OPT)
#include "opt_inet.h"
@@ -1083,7 +1083,7 @@ int
linux_getifconf(struct lwp *l, register_t *retval, void *data)
{
struct linux_ifreq ifr, *ifrp;
- struct ifconf *ifc = data;
+ struct linux_ifconf ifc;
struct ifnet *ifp;
struct ifaddr *ifa;
struct sockaddr *sa;
@@ -1091,11 +1091,15 @@ linux_getifconf(struct lwp *l, register_t *retval, void *data)
int space, error = 0;
const int sz = (int)sizeof(ifr);
- ifrp = (struct linux_ifreq *)ifc->ifc_req;
+ error = copyin(data, &ifc, sizeof(ifc));
+ if (error)
+ return error;
+
+ ifrp = ifc.ifc_req;
if (ifrp == NULL)
space = 0;
else
- space = ifc->ifc_len;
+ space = ifc.ifc_len;
IFNET_FOREACH(ifp) {
(void)strncpy(ifr.ifr_name, ifp->if_xname,
@@ -1123,11 +1127,11 @@ linux_getifconf(struct lwp *l, register_t *retval, void *data)
}
if (ifrp != NULL)
- ifc->ifc_len -= space;
+ ifc.ifc_len -= space;
else
- ifc->ifc_len = -space;
+ ifc.ifc_len = -space;
- return 0;
+ return copyout(&ifc, data, sizeof(ifc));
}
int
diff --git a/sys/compat/linux/common/linux_sockio.h b/sys/compat/linux/common/linux_sockio.h
index cb51bbd3664..cdf63cb92e5 100644
--- a/sys/compat/linux/common/linux_sockio.h
+++ b/sys/compat/linux/common/linux_sockio.h
@@ -1,4 +1,4 @@
-/* $NetBSD: linux_sockio.h,v 1.18 2009/11/28 22:11:42 dsl Exp $ */
+/* $NetBSD: linux_sockio.h,v 1.19 2013/01/11 19:01:36 christos Exp $ */
/*-
* Copyright (c) 1995, 1998 The NetBSD Foundation, Inc.
@@ -76,4 +76,15 @@ struct linux_ifreq {
#define ifr_map ifr_ifru.ifru_map /* device map */
};
+struct linux_ifconf {
+ int ifc_len;
+ union {
+ char *ifcu_buf;
+ struct linux_ifreq *ifcu_req;
+ } ifc_ifcu;
+};
+
+#define ifc_buf ifc_ifcu.ifcu_buf
+#define ifc_req ifc_ifcu.ifcu_req
+
#endif /* !_LINUX_SOCKIO_H */
diff --git a/sys/compat/linux32/common/linux32_socket.c b/sys/compat/linux32/common/linux32_socket.c
index f0675bf4413..526b473a304 100644
--- a/sys/compat/linux32/common/linux32_socket.c
+++ b/sys/compat/linux32/common/linux32_socket.c
@@ -1,4 +1,4 @@
-/* $NetBSD: linux32_socket.c,v 1.16 2012/03/15 16:17:48 bouyer Exp $ */
+/* $NetBSD: linux32_socket.c,v 1.17 2013/01/11 19:01:36 christos Exp $ */
/*-
* Copyright (c) 2006 Emmanuel Dreyfus, all rights reserved.
@@ -33,7 +33,7 @@
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: linux32_socket.c,v 1.16 2012/03/15 16:17:48 bouyer Exp $");
+__KERNEL_RCSID(0, "$NetBSD: linux32_socket.c,v 1.17 2013/01/11 19:01:36 christos Exp $");
#include <sys/types.h>
#include <sys/param.h>
@@ -414,7 +414,7 @@ int
linux32_getifconf(struct lwp *l, register_t *retval, void *data)
{
struct linux32_ifreq ifr, *ifrp;
- struct netbsd32_ifconf *ifc = data;
+ struct linux32_ifconf ifc;
struct ifnet *ifp;
struct ifaddr *ifa;
struct sockaddr *sa;
@@ -422,11 +422,15 @@ linux32_getifconf(struct lwp *l, register_t *retval, void *data)
int space, error = 0;
const int sz = (int)sizeof(ifr);
- ifrp = (struct linux32_ifreq *)NETBSD32PTR64(ifc->ifc_req);
+ error = copyin(data, &ifc, sizeof(ifc));
+ if (error)
+ return error;
+
+ ifrp = NETBSD32PTR64(ifc.ifc_req);
if (ifrp == NULL)
space = 0;
else
- space = ifc->ifc_len;
+ space = ifc.ifc_len;
IFNET_FOREACH(ifp) {
(void)strncpy(ifr.ifr_name, ifp->if_xname,
@@ -454,11 +458,11 @@ linux32_getifconf(struct lwp *l, register_t *retval, void *data)
}
if (ifrp != NULL)
- ifc->ifc_len -= space;
+ ifc.ifc_len -= space;
else
- ifc->ifc_len = -space;
+ ifc.ifc_len = -space;
- return 0;
+ return copyout(&ifc, data, sizeof(ifc));
}
int
diff --git a/sys/compat/linux32/common/linux32_sockio.h b/sys/compat/linux32/common/linux32_sockio.h
index 90a0765bafb..2803e6d9b5b 100644
--- a/sys/compat/linux32/common/linux32_sockio.h
+++ b/sys/compat/linux32/common/linux32_sockio.h
@@ -1,4 +1,4 @@
-/* $NetBSD: linux32_sockio.h,v 1.3 2009/11/13 21:45:03 joerg Exp $ */
+/* $NetBSD: linux32_sockio.h,v 1.4 2013/01/11 19:01:36 christos Exp $ */
/*
* Copyright (c) 2008 Nicolas Joly
@@ -56,4 +56,15 @@ struct linux32_ifreq {
#define ifr_map ifr_ifru.ifru_map /* device map */
};
+struct linux32_ifconf {
+ int ifc_len;
+ union {
+ netbsd32_caddr_t ifcu_buf;
+ netbsd32_ifreq_tp_t ifcu_req;
+ } ifc_ifcu;
+};
+
+#define ifc_buf ifc_ifcu.ifcu_buf
+#define ifc_req ifc_ifcu.ifcu_req
+
#endif /* !_LINUX32_SOCKIO_H */