FAQ Search Today's Posts Mark Forums Read
» Video Reviews

» Linux Archive

Linux-archive is a website aiming to archive linux email lists and to make them easily accessible for linux users/developers.


» Sponsor

» Partners

» Sponsor

Go Back   Linux Archive > Redhat > Fedora Development

 
 
LinkBack Thread Tools
 
Old 12-02-2010, 04:34 PM
Kyle McMartin
 
Default vfork() semantics changed: ERESTARTNOINTR

On Thu, Dec 02, 2010 at 09:27:34AM -0800, John Reiser wrote:
> vfork() can fail with ERESTARTNOINTR which is 513
> and somewhat young. 'make' did not know:
> https://bugzilla.redhat.com/show_bug.cgi?id=659382
>
> If your package has any shell-like feature
> then it might be good to check for vfork().
>

These are internal kernel errnos, and I'm fairly certain are not
supposed to leak to userspace, but be fixed up by the syscall return
code...

Roland?

--Kyle
--
devel mailing list
devel@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/devel
 
Old 12-02-2010, 10:25 PM
Roland McGrath
 
Default vfork() semantics changed: ERESTARTNOINTR

> On Thu, Dec 02, 2010 at 09:27:34AM -0800, John Reiser wrote:
> > vfork() can fail with ERESTARTNOINTR which is 513
> > and somewhat young. 'make' did not know:
> > https://bugzilla.redhat.com/show_bug.cgi?id=659382
> >
> > If your package has any shell-like feature
> > then it might be good to check for vfork().
> >
>
> These are internal kernel errnos, and I'm fairly certain are not
> supposed to leak to userspace, but be fixed up by the syscall return
> code...
>
> Roland?

That is correct. You should never see an ERESTART* code as the result of
any system call. The only time userland code should be able to see this is
via ptrace, which can sometimes see it (maybe only in syscall tracing?)
before the rollback of user register state it indicates has been done.
So seeing this in strace is not unexpected, but any user program seeing
it leak out is a kernel bug.


Thanks,
Roland
--
devel mailing list
devel@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/devel
 
Old 12-03-2010, 08:58 PM
Roland McGrath
 
Default vfork() semantics changed: ERESTARTNOINTR

> I am not sure ERESTARTNOINTR leaks to user-space. Probably reporter
> noticed ERESTARTNOINTR in strace.out and came to the wrong conclusion.
> Afaics, make reports -EINVAL.

But I don't think vfork is supposed to be able to fail with EINVAL.
So something is fishy.
--
devel mailing list
devel@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/devel
 

Thread Tools




All times are GMT. The time now is 04:45 PM.

VBulletin, Copyright ©2000 - 2014, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.
Copyright 2007 - 2008, www.linux-archive.org