bugThe nmh Mail Handling System - Bugs: bug #55998, inc does not use POP password...

 
 

bug #55998: inc does not use POP password entered interactively

Submitter:  None
Submitted:  Mon 25 Mar 2019 01:29:46 PM UTC
   
 
Category:  Bug Severity:  3 - Normal
Priority:  * 5 - Normal Status:  Fixed
Assigned to:  None Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Wed 27 Mar 2019 02:04:48 AM UTC, comment #1: 

Thank you for your bug report; this was already fixed in commit 896f07bf3, but that was after 1.7.1 was released.

Ken Hornstein <kenh>
Group administrator
Mon 25 Mar 2019 01:29:46 PM UTC, original submission:  

ruserpass calls nmh_getpass() to prompt the user for interactive password entry, but incorrectly copies the password to the *aname location before returning.

When using inc -snoop, the password sent it "(null)".

The following trivial patch corrects the problem

* nmh-1.7.1/sbr/ruserpass.c.orig      Tue Mar  6 11:05:55 2018
--- nmh-1.7.1/sbr/ruserpass.c   Sun Mar 24 12:35:55 2019
*************
* 167,176 **
        mypass = nmh_getpass(prompt);

        if (*mypass == '\0') {
!           mypass = *aname;
        }

!         *aname = mh_xstrdup(mypass);
      }

  }
--- 167,176 ----
        mypass = nmh_getpass(prompt);

        if (*mypass == '\0') {
!           mypass = *apass;
        }

!         *apass = mh_xstrdup(mypass);
      }

  }


Anonymous

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #46636:  nmh-ruserpass.patch added by None (496B - text/x-patch - bug + patch)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by kenh (Posted a comment)
  •  

    Follow 3 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2019-03-27 kenh StatusNone Fixed
        Open/ClosedOpen Closed
    2019-03-25 None Attached File- Added nmh-ruserpass.patch, #46636

    Back to the top

    Powered by Savane 3.13-4b48.
    Corresponding source code