Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20120627021640.42af7e4f@sibserver.ru>
Date: Wed, 27 Jun 2012 02:16:40 +0800
From: orc <orc@...server.ru>
To: musl@...ts.openwall.com
Subject: Re: openssh and sendmsg() problem

On Tue, 26 Jun 2012 13:32:05 -0400
Rich Felker <dalias@...ifal.cx> wrote:

> On Wed, Jun 27, 2012 at 01:24:41AM +0800, orc wrote:
> > On Tue, 26 Jun 2012 12:01:06 -0400
> > Rich Felker <dalias@...ifal.cx> wrote:
> > 
> > > On Tue, Jun 26, 2012 at 02:00:17PM +0200, Szabolcs Nagy wrote:
> > > > * orc <orc@...server.ru> [2012-06-26 19:01:46 +0800]:
> > > > > strace shows that sendmsg system call returns EINVAL:
> > > > > 
> > > > > sendmsg(5, {msg_name(0)=NULL, msg_iov(1)=[{"\0", 1}],
> > > > > msg_controllen=24, {cmsg_len=20, cmsg_level=SOL_SOCKET, cmsg_
> > > > > type=SCM_RIGHTS, {4, 385875968}}, msg_flags=0}, 0) = -1 EINVAL
> > > > > (Invalid argument)
> > > > > 
> > > > 
> > > > src/network/sendmsg.c has an ifdef for long>int case
> > > > 
> > > > i don't know what it's supposed to do
> > > 
> > > It's there because the kernel expects certain fields to be long
> > > but POSIX requires them to be int. Thus the syscall wrapper has
> > > to copy the userspace struct (with int and padding) to a
> > > kernel-format struct (with the padding filled with 0's so it's a
> > > valid long).
> > 
> > And how to deal with this? Should it be openssh workaround or fix in
> > musl itself?
> 
> There's nothing to be dealt with here; musl already works around the
> kernel bug. Something is wrong, possibly in this code or elsewhere,
> but I don't know what and I don't see anything obviously wrong with
> the syscall. Can you compare the strace to what happens on glibc?
> 
> Rich

strace from glibc system on this syscall:

sendmsg(5, {msg_name(0)=NULL, msg_iov(1)=[{"\0", 1}], msg_controllen=24, {cmsg_len=20, cmsg_level=SOL_SOCKET, cmsg_type=SCM_RIGHTS, {4}}, msg_flags=0}, 0) = 1
sendmsg(5, {msg_name(0)=NULL, msg_iov(1)=[{"\0", 1}], msg_controllen=24, {cmsg_len=20, cmsg_level=SOL_SOCKET, cmsg_type=SCM_RIGHTS, {6}}, msg_flags=0}, 0) = 1

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.