After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 679790 - Add support for allocating BSD PTYs through openpty(3)
Add support for allocating BSD PTYs through openpty(3)
Status: RESOLVED FIXED
Product: gvfs
Classification: Core
Component: daemon
1.12.x
Other OpenBSD
: Normal normal
: ---
Assigned To: gvfs-maint
gvfs-maint
Depends on:
Blocks:
 
 
Reported: 2012-07-12 11:05 UTC by Martin Pieuchot
Modified: 2012-07-13 14:52 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Diff implementing the openpty support (5.53 KB, patch)
2012-07-12 11:05 UTC, Martin Pieuchot
committed Details | Review

Description Martin Pieuchot 2012-07-12 11:05:27 UTC
Created attachment 218613 [details] [review]
Diff implementing the openpty support

The diff attached allow systems like OpenBSD, that does not implement the
Unix98 PTY family of functions, to allocate the pseudo-tty required for the ssh authentication using the openpty(3) BSD function.

It is similar to the VTE diff [0] I submitted some months ago and has been committed to the OpenBSD ports tree.

[0] https://bugzilla.gnome.org/show_bug.cgi?id=670758
Comment 1 Antoine Jacoutot 2012-07-12 11:34:49 UTC
I can confirm this diff works beautifully on several OpenBSD+GNOME installations I use here. It's also enabled by default in our gvfs package.

I would love to be able to push this.
Comment 2 Tomas Bzatek 2012-07-13 14:51:04 UTC
Review of attachment 218613 [details] [review]:

Thanks for the patch, it looks good to me, most of the functionality is ifdef'ed anyway. The code is basically shared with other components and duplicated, if it works in vte, I assume it'll work fine in gvfs.
Comment 3 Tomas Bzatek 2012-07-13 14:52:35 UTC
Committed in master:

commit 2fec402d6523e4cf416d754f69d012dc67a58a2e
Author: Martin Pieuchot <mpi@openbsd.org>
Date:   Fri Jul 13 16:51:39 2012 +0200

    daemon: Support allocating PTYs through openpty on BSD
    
    Allows systems supporting the BSD openpty(3) utily function but not
    the Unix98 PTY function family (grantpt(3), unlockpt(3), ...) to
    allocate a pseudo-tty required for the ssh authentication.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=679790
    
    Signed-off-by: Tomas Bzatek <tbzatek@redhat.com>