aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSergey Bugaev <bugaevc@gmail.com>2023-02-12 14:10:33 +0300
committerSamuel Thibault <samuel.thibault@ens-lyon.org>2023-02-12 15:52:07 +0100
commitf4315054b46d5e58b44a709a51943fb73f846afb (patch)
tree08dd446b613f6549260c17b3060453fa040eb4d8
parent8a86e7b6a67f21294ada5bf67f3484ae3e134848 (diff)
downloadglibc-f4315054b46d5e58b44a709a51943fb73f846afb.tar
glibc-f4315054b46d5e58b44a709a51943fb73f846afb.tar.gz
glibc-f4315054b46d5e58b44a709a51943fb73f846afb.tar.bz2
glibc-f4315054b46d5e58b44a709a51943fb73f846afb.zip
hurd: Use mach_msg_type_number_t where appropriate
It has been decided that on x86_64, mach_msg_type_number_t stays 32-bit. Therefore, it's not possible to use mach_msg_type_number_t interchangeably with size_t, in particular this breaks when a pointer to a variable is passed to a MIG routine. Signed-off-by: Sergey Bugaev <bugaevc@gmail.com> Message-Id: <20230212111044.610942-3-bugaevc@gmail.com>
-rw-r--r--hurd/hurdioctl.c2
-rw-r--r--hurd/hurdprio.c2
-rw-r--r--hurd/lookup-retry.c2
-rw-r--r--hurd/xattr.c4
-rw-r--r--sysdeps/mach/hurd/getcwd.c2
-rw-r--r--sysdeps/mach/hurd/readlinkat.c6
-rw-r--r--sysdeps/mach/hurd/sendfile64.c2
7 files changed, 11 insertions, 9 deletions
diff --git a/hurd/hurdioctl.c b/hurd/hurdioctl.c
index e9a6a7ea45..59e6a5c1f6 100644
--- a/hurd/hurdioctl.c
+++ b/hurd/hurdioctl.c
@@ -311,7 +311,7 @@ static int
siocgifconf (int fd, int request, struct ifconf *ifc)
{
error_t err;
- size_t data_len = ifc->ifc_len;
+ mach_msg_type_number_t data_len = ifc->ifc_len;
char *data = ifc->ifc_buf;
if (data_len <= 0)
diff --git a/hurd/hurdprio.c b/hurd/hurdprio.c
index dbeb272bdc..954d3987f1 100644
--- a/hurd/hurdprio.c
+++ b/hurd/hurdprio.c
@@ -58,7 +58,7 @@ _hurd_priority_which_map (enum __priority_which which, int who,
int *oldpi = pi;
mach_msg_type_number_t oldpisize = pisize;
char *tw = 0;
- size_t twsz = 0;
+ mach_msg_type_number_t twsz = 0;
err = __USEPORT (PROC, __proc_getprocinfo (port, pids[i],
&pi_flags,
&pi, &pisize,
diff --git a/hurd/lookup-retry.c b/hurd/lookup-retry.c
index 0d344026e4..8850c4fd27 100644
--- a/hurd/lookup-retry.c
+++ b/hurd/lookup-retry.c
@@ -162,7 +162,7 @@ __hurd_file_name_lookup_retry (error_t (*use_init_port)
{
char buf[1024];
char *trans = buf;
- size_t translen = sizeof buf;
+ mach_msg_type_number_t translen = sizeof buf;
err = __file_get_translator (*result,
&trans, &translen);
if (!err
diff --git a/hurd/xattr.c b/hurd/xattr.c
index f98e548c19..48914bcf26 100644
--- a/hurd/xattr.c
+++ b/hurd/xattr.c
@@ -60,7 +60,7 @@ _hurd_xattr_get (io_t port, const char *name, void *value, size_t *size)
if (!strcmp (name, "translator"))
{
char *buf = value;
- size_t bufsz = value ? *size : 0;
+ mach_msg_type_number_t bufsz = value ? *size : 0;
error_t err = __file_get_translator (port, &buf, &bufsz);
if (err)
return err;
@@ -144,7 +144,7 @@ _hurd_xattr_set (io_t port, const char *name, const void *value, size_t size,
{
/* Must make sure it's already there. */
char *buf = NULL;
- size_t bufsz = 0;
+ mach_msg_type_number_t bufsz = 0;
error_t err = __file_get_translator (port, &buf, &bufsz);
if (err)
return err;
diff --git a/sysdeps/mach/hurd/getcwd.c b/sysdeps/mach/hurd/getcwd.c
index fc5e78e7d3..f24b35b380 100644
--- a/sysdeps/mach/hurd/getcwd.c
+++ b/sysdeps/mach/hurd/getcwd.c
@@ -117,7 +117,7 @@ __hurd_canonicalize_directory_name_internal (file_t thisdir,
int mount_point;
file_t newp;
char *dirdata;
- size_t dirdatasize;
+ mach_msg_type_number_t dirdatasize;
int direntry, nentries;
diff --git a/sysdeps/mach/hurd/readlinkat.c b/sysdeps/mach/hurd/readlinkat.c
index dabdbb3713..5bb8b75371 100644
--- a/sysdeps/mach/hurd/readlinkat.c
+++ b/sysdeps/mach/hurd/readlinkat.c
@@ -35,6 +35,7 @@ __readlinkat (int fd, const char *file_name, char *buf, size_t len)
char retryname[1024];
file_t file;
char *rbuf = buf;
+ mach_msg_type_number_t nread = len;
file_stat = __file_name_lookup_at (fd, 0, file_name, O_NOLINK, 0);
if (file_stat == MACH_PORT_NULL)
@@ -59,15 +60,16 @@ __readlinkat (int fd, const char *file_name, char *buf, size_t len)
goto out;
}
- err = __io_read (file, &rbuf, &len, 0, len);
+ err = __io_read (file, &rbuf, &nread, 0, len);
__mach_port_deallocate (__mach_task_self (), file);
if (err)
goto out;
+ len = nread;
if (rbuf != buf)
{
memcpy (buf, rbuf, len);
- __vm_deallocate (__mach_task_self (), (vm_address_t) rbuf, len);
+ __vm_deallocate (__mach_task_self (), (vm_address_t) rbuf, nread);
}
diff --git a/sysdeps/mach/hurd/sendfile64.c b/sysdeps/mach/hurd/sendfile64.c
index 658b02826d..e36b979078 100644
--- a/sysdeps/mach/hurd/sendfile64.c
+++ b/sysdeps/mach/hurd/sendfile64.c
@@ -35,7 +35,7 @@ __sendfile64 (int out_fd, int in_fd, off64_t *offset, size_t count)
which might blow if it's huge or address space is real tight. */
char *data = 0;
- size_t datalen = 0;
+ mach_msg_type_number_t datalen = 0;
error_t err = HURD_DPORT_USE (in_fd,
__io_read (port, &data, &datalen,
offset ? *offset : (off_t) -1,