Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <alpine.LNX.2.20.13.2103160104210.17743@monopod.intra.ispras.ru>
Date: Tue, 16 Mar 2021 01:09:16 +0300 (MSK)
From: Alexander Monakov <amonakov@...ras.ru>
To: musl@...ts.openwall.com
cc: Dominic Chen <d.c.ddcc@...il.com>
Subject: Re: Issue with fread() and unaligned readv()

On Mon, 15 Mar 2021, Rich Felker wrote:

> On Mon, Mar 15, 2021 at 05:39:43PM -0400, Dominic Chen wrote:
> > Not sure this counts as a problem in musl or the application, but
> > I've been debugging a return error of EINVAL from `fread(&buf, 8,
> > 16, f)`, where `f = fopen("/proc/self/pagemap", "r")`. Internally,
> > musl converts this into a call to `readv(f->fd, iov, 2)`, where `iov
> > = {{iov_base = buf, iov_len = 127}, {iov_base = f->buf, iov_len =
> > 1024}}`. However, it turns out that the kernel VFS read
> > implementation inside `pagemap_read` checks that both the file
> > position and count are divisible by PM_ENTRY_BYTES (8 on x86_64),
> > otherwise it rejects the read with EINVAL. In comparison, glibc's
> > `_IO_file_xsgetn` does appear to try to maintain read alignment,
> > although I haven't looked at it in detail.
> 
> You can't use stdio to read or write special files/devices that depend
> on the reads or writes happening in particular units, because the
> relationship between stdio operations and the underlying
> buffer-fill/flush operations on the underlying fd is unspecified. It's
> really unfortunate that the kernel lies that procfs files are regular
> files but doesn't give them regular-file semantics, but you really
> need to use direct operations on the fd in the units the interface
> requires, rather than stdio, to work with these files.

Where does iov_len = 127 for the first iov tuple come from, though?
>From fread arguments I'd expect 8 * 16 = 128.

If musl always does such off-by-one, it is an efficiency issue (forces
a copy with mismatching source/dest alignment).

Alexander

Powered by blists - more mailing lists

Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.