bugGNU Screen - Bugs: bug #23691, Unnecessary upper limit on how...

 
 

bug #23691: Unnecessary upper limit on how many pty's may be opened

Submitter:  Micah Cowan <micahcowan>
Submitted:  Tue 24 Jun 2008 05:36:13 AM UTC
   
 
Category:  Program Logic Severity:  3 - Normal
Priority:  * 5 - Normal Status:  Fixed
Privacy:  Public Assigned to:  None
Open/Closed:  Open Release:  5.0.0
Fixed Release:  None Planned Release:  Later
Work Required:  0 - Hours
* Mandatory Fields

Add a New Comment Rich Markup
   

Sun 09 Jul 2017 10:07:01 PM UTC, comment #2: 

As far as I can tell there should be no limit of ptys anymore on master branch. And windows are now allocated dynamically.

http://git.savannah.gnu.org/cgit/screen.git/commit/?id=6724ef2529c8bb80d0b103f528af371a1b3e77d8

Amadeusz Sławiński <amade>
Group administrator
Sun 25 Jan 2009 06:43:13 PM UTC, comment #1: 

Correction: patch isn't at all reasonable, as it removes the BSD-style pty code, replacing it with SCO's conventions.

exg points us at openssh's pty.c, which uses a method that has the advantage that it doesn't really care if it's running under BSD or SCO (it tries both). It has the disadvantage of checking twice as many tty names.

Micah Cowan <micahcowan>
Tue 24 Jun 2008 05:36:13 AM UTC, original submission:  

At least on SCO.

http://www.linuxquestions.org/questions/linux-software-2/screen-error-no-more-ptys-299380/

The patch seems mostly reasonable, though it includes a patch against config.h, which is inappropriate (as is the addition of PTYMAX to that location). I don't understand why MAXWIN was modified there, either.

Micah Cowan <micahcowan>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by amade (Posted a comment)
  • -email is unavailable- added by micahcowan (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 3 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2017-07-09 amade StatusConfirmed Fixed
        ReleaseNone 5.0.0
    2009-01-28 micahcowan Planned Release4.1.0 Later

    Back to the top

    Powered by Savane 3.13-758e.
    Corresponding source code