bugGNU Screen - Bugs: bug #29959, screen crashes/freezes when...

 
 

bug #29959: screen crashes/freezes when creating/renaming window inside group and windowlist active

Submitter:  Artur Skonecki <askonecki>
Submitted:  Tue 25 May 2010 06:25:50 PM UTC
Votes: 10
 
Category:  Crash/Freeze/Infloop Severity:  3 - Normal
Priority:  * 5 - Normal Status:  Need Info
Privacy:  Public Assigned to:  None
Open/Closed:  Open Release:  Cur Dev Sources
Fixed Release:  None Planned Release:  None
Work Required:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Mon 14 Feb 2011 09:19:25 PM UTC, comment #4: 

Removing D_user check from AddWindowFlags() prevents the crash. I hope it doesn't have any serious consequences with a different setup.

diff --git a/src/process.c b/src/process.c
index e494004..a18bc28 100644
--- a/src/process.c
+++ b/src/process.c
@@ -5634,11 +5634,7 @@ struct win *p;
 #endif
   if (p->w_layer.l_cvlist && p->w_layer.l_cvlist->c_lnext)
     *s++ = '&';
-  if (p->w_monitor == MON_DONE
-#ifdef MULTIUSER
-      && (ACLBYTE(p->w_mon_notify, D_user->u_id) & ACLBIT(D_user->u_id))
-#endif
-     )
+  if (p->w_monitor == MON_DONE)
     *s++ = '@';
   if (p->w_bell == BELL_DONE)
     *s++ = '!';


Artur Skonecki <askonecki>
Mon 14 Feb 2011 04:33:25 PM UTC, comment #3: 

Currently, the monitor feature and grouping of windows are unusable together.
To produce a crash it is enough to:
1. Create 2 windows inside a group.
2. Turn on monitoring in one of them and induce monitor activation(tail -f file_to_be_modified).
3. Try to open windowlist.

Artur Skonecki <askonecki>
Tue 25 May 2010 08:23:46 PM UTC, comment #2: 

I missed the trigger: defmonitor on
Now I tested vanilla 2f565ea with config:
defmonitor on
bind x screen -t G group
bind o windowlist  


Artur Skonecki <askonecki>
Tue 25 May 2010 06:46:09 PM UTC, comment #1: 

Hi. I can't reproduce this crash with revision 2f565ea. What version/revision of screen are you using?

Sadrul Habib Chowdhury <sadrul>
Group administrator
Tue 25 May 2010 06:25:50 PM UTC, original submission:  

Steps to reproduce:
1) create a group inside screen session: screen -t G group
2) enter this group, create some windows inside and open windowlist
3) attach to this screen session from another term: screen -x
4) go to group G and try to create new window or rename existing one

Then, there is either a dead screen session or crash:
Suddenly the dungeon collapses!! - You die...

5) the effect is the same if autoaka changes title

A punishment for all cheating in Nethack!

Artur Skonecki <askonecki>

 

(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 ssducf (Voted in favor of this item)
  • -email is unavailable- added by sadrul (Posted a comment)
  • -email is unavailable- added by askonecki (Submitted the item)
  •  

    There are 10 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 2 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2017-07-22 ssducf Carbon-Copy- Added ssducf
    2010-05-25 sadrul StatusNone Need Info

    Back to the top

    Powered by Savane 3.13-cf05.
    Corresponding source code