bugGNU nano - Bugs: bug #58443, when backup fails, ^C is...

 
 

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

bug #58443: when backup fails, ^C is understood as "Yes"

Submitter:  Benno Schulenberg <bens>
Submitted:  Mon 25 May 2020 09:47:42 AM UTC
   
 
Severity:  4 - Important Status:  Fixed
Assigned to:  bens Open/Closed:  Closed

Thu 28 May 2020 05:59:54 PM UTC, comment #3: 

Well, it is fixed, it's just that it writes the error message anyway.  In fact, it should print the error message first (an accurate one, see bug #58410) and then prompt what to do.  And after the prompt has been answered, be quiet.

Benno Schulenberg <bens>
Group administrator
Thu 28 May 2020 02:57:17 PM UTC, comment #2: 

No, it's not fixed, of course.  I've just changed ^C from Yes to No.  That doesn't help.

Benno Schulenberg <bens>
Group administrator
Thu 28 May 2020 01:03:21 PM UTC, comment #1: 

Fixed in git, commit 90772ad0.

Benno Schulenberg <bens>
Group administrator
Mon 25 May 2020 09:47:42 AM UTC, original submission:  

The same recipe as in bug #58442:

mkdir tub
touch tub/bar~ tub/bar
sudo chown root:root tub
sudo chmod go-w tub
echo aaa >tub/bar~
echo bbb >tub/bar

Now run 'src/nano -B tub/bar' and type: ccc ^S
At the "continue saving?" prompt, type: ^C
Result: "[ Wrote 1 line ]"  Oops!

Expected behavior: when the user presses ^C at the "continue saving?" prompt, nano should cancel the saving, both of the backup file and of the file itself, and return to the edit window, so that the user can open another terminal (or suspend nano) and fix the permissions or simply inspect what is going on.

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 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2020-07-30 bens Open/ClosedOpen Closed
    2020-05-28 bens StatusNone Fixed
    2020-05-28 bens StatusFixed None
    2020-05-28 bens StatusNone Fixed

    Back to the top

    Powered by Savane 3.13-758e.
    Corresponding source code