Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20110722043021.GE132@brightrain.aerifal.cx>
Date: Fri, 22 Jul 2011 00:30:21 -0400
From: Rich Felker <dalias@...ifal.cx>
To: musl@...ts.openwall.com
Subject: Re: some fixes to musl

On Thu, Jul 21, 2011 at 09:57:39PM -0400, Rich Felker wrote:
> > forkpty():
> > - It should be guaranteed that master fd is closed, tty is setup, slave
> >   fd is dup'ed to 1,2,3.  The latter can be broken by setting small
> >   rlimit.  setsid() is checked for company :)  I think the only way to
> >   handle the failure is _exit().  While it may be not the best choise,
> >   however, continuing the work with half dropped privileges is more
> >   dangerous.
> > 
> > openpty():
> > - close() shouldn't change errno updated by failed ioctl()/open().
> > - I suppose the last calls to tcsetattr() and ioctl() may fail too.
> 
> Going to try to find a good solution for these...

I believe I've fixed forkpty's issue with fd exhaustion. Please tell
me if anything seems wrong.

Rich

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.