bugGNU nano - Bugs: bug #56394, resizing the window while reading...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #56394: resizing the window while reading data from stdin messes up the terminal

Submitter:  Benno Schulenberg <bens>
Submitted:  Sun 26 May 2019 03:38:21 PM UTC
   
 
Severity:  2 - Minor Status:  Fixed
Assigned to:  bens Open/Closed:  Closed

Mon 27 May 2019 06:06:59 PM UTC, comment #3: 

Now fixed (as far as I could test) with commit b53dffae.

Benno Schulenberg <bens>
Group administrator
Mon 27 May 2019 03:47:01 PM UTC, comment #2: 

That was the wrong fix, because now the terminal is messed up when data is piped in, like with:

  cat README | src/nano -

Benno Schulenberg <bens>
Group administrator
Sun 26 May 2019 05:18:42 PM UTC, comment #1: 

Fixed in git, commit b7e2da74.

Benno Schulenberg <bens>
Group administrator
Sun 26 May 2019 03:38:21 PM UTC, original submission:  

To reproduce, run:

  src/nano --ignore -

Then first resize the terminal a bit.
Then type: aaa <Enter> bbb ^D ^D ^X ^N

Now type <Enter> a few times, and see that the cursor doesn't move to a new line.

In nano-4.0 and earlier, a resizing of the window would also abort the reading from stdin (and the terminal would  be messed up upon exit).

In nano-2.4.1 things still worked okay (a SIGWINCH did not abort reading and the terminal state would be okay afterward).  In nano-2.4.3 things are bad.  The rewriting of the handling of SIGWINCH in commit 75d64e67 is the likely culprit.

Benno Schulenberg <bens>
Group administrator

 

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

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 bens (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.

     

    Follow 6 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2019-06-19 bens Open/ClosedOpen Closed
    2019-05-27 bens StatusIn Progress Fixed
    2019-05-27 bens StatusFixed In Progress
    2019-05-26 bens Severity3 - Normal 2 - Minor
        StatusNone Fixed
        Assigned toNone bens

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code