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

 
 

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

Submitted by:  Micah Cowan <micahcowan>
Submitted on:  Tue 24 Jun 2008 05:36:13 AM UTC  
 
Category: Program LogicSeverity: 3 - Normal
Priority: 5 - NormalStatus: Fixed
Privacy: PublicAssigned to: None
Open/Closed: OpenRelease: 5.0.0
Fixed Release: NonePlanned Release: Later
Work Required: 0 - Hours

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

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>
Project 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 File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

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

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 3 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Sun 09 Jul 2017 10:07:01 PM UTCamadeStatusConfirmed=>Fixed
      ReleaseNone=>5.0.0
    Wed 28 Jan 2009 06:51:48 PM UTCmicahcowanPlanned Release4.1.0=>Later

    Back to the top


    Powered by Savane 3.1-cleanup1