Bug 139938

Summary: cscope does not handle SIGWINCH
Product: [Fedora] Fedora Reporter: Mike Gahagan <mgahagan>
Component: cscopeAssignee: Neil Horman <nhorman>
Status: CLOSED NEXTRELEASE QA Contact:
Severity: medium Docs Contact:
Priority: medium    
Version: 3CC: nhorman
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2004-11-23 16:16:49 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
patch to allow cscope to be resized without exiting
none
new version of patch that doesnt break ia64 and other platforms.
none
new version of the patch that moves resize work out of signal handler none

Description Mike Gahagan 2004-11-18 20:57:05 UTC 
Description of problem:

cscope does not handle SIGWINCH which causes it to terminate if run
out of a terminal window in X and you resize the window.

Version-Release number of selected component (if applicable):


How reproducible:

always

Steps to Reproduce:
1. open a terminal window from X (GNOME, KDE etc shouldn't matter)
2. go to a directory w/ source code and run 'cscope -R'
3. Attempt to resize the terminal window and watch cscope exit.
  
Actual results:

Window is resized but Cscope terminates

Expected results:

Resizing the term window cscope is running in shouldn't kill it.

Additional info:
Comment 1 Frank Ch. Eigler 2004-11-18 21:17:12 UTC 
cscope needs a SIGWINCH handler.  See web pages like:
http://sunse.jinr.ru/local/ncurses/ncurses-intro.html
http://www.apmaths.uwo.ca/~xli/ncurses.html

The gist of it is that a new SIGWINCH handler function should contain:

   int nh, nw;
   getmaxyx(stdscr, nh, nw);
   resizeterm (nh, nw);
Comment 3 Neil Horman 2004-11-22 20:35:19 UTC 
Created attachment 107237 [details]
patch to allow cscope to be resized without exiting

I've submitted this patch upstream, and will be integrating it into all of our
packages barring any serious objections from the maintainer.
Comment 4 Neil Horman 2004-11-23 01:01:34 UTC 
Created attachment 107270 [details]
new version of patch that doesnt break ia64 and other platforms.

probably shouldn't set sa_restorer as its not defined on all platforms, and is
depricated.  New patch attached.
Comment 5 Neil Horman 2004-11-27 17:52:36 UTC 
Created attachment 107500 [details]
new version of the patch that moves resize work out of signal handler
Comment 6 Ronald Wahl 2007-02-27 00:16:35 UTC 
In line oriented mode (-l) it still crashes. This is used by cscope integration in
vim. Everytime the window is resized the cscope background process crashes. Maybe
the handler is enabled only when "GUI" is enabled?