[screen-devel] [bug #45713] [PATCH] stack overflow due to too deep recursion

[Kuang-che Wu]

URL:
  <http://savannah.gnu.org/bugs/?45713>

                 Summary: [PATCH] stack overflow due to too deep recursion
                 Project: GNU Screen
            Submitted by: kcwu
            Submitted on: Fri 07 Aug 2015 12:54:02 PM GMT
                Category: Crash/Freeze/Infloop
                Severity: 3 - Normal
                Priority: 5 - Normal
                  Status: None
                 Privacy: Public
             Assigned to: None
             Open/Closed: Open
         Discussion Lock: Any
                 Release: None
           Fixed Release: None
         Planned Release: None
           Work Required: None

    _______________________________________________________

Details:

How to reproduce:
Run this command inside screen
$ printf '\x1b[10000000T'

screen will recursively call MScrollV to depth n/256. This is time consuming
and will overflow stack if n is huge.

There is already code to handle huge value below the recursion.
if (ye - ys + 1 < n)
IIUC, we can just remove the recursion and don't change its behavior.




    _______________________________________________________

File Attachments:


-------------------------------------------------------
Date: Fri 07 Aug 2015 12:54:02 PM GMT  Name: diff-screen-ansi.txt  Size: 687B 
 By: kcwu

<http://savannah.gnu.org/bugs/download.php?file_id=34590>

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?45713>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/