bugThe nmh Mail Handling System - Bugs: bug #15213, mktemp is deprecated and we...

 
 

bug #15213: mktemp is deprecated and we shouldn't use it

Submitter:  Peter Maydell <pm215>
Submitted:  Thu 15 Dec 2005 01:42:07 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
   

Mon 27 Feb 2012 07:28:05 PM UTC, comment #5: 

We don't use mktemp anymore!  Although I can't take any credit for that; everyone else took care of that.

Ken Hornstein <kenh>
Group administrator
Thu 04 Feb 2010 02:02:06 AM UTC, comment #4: 

I did a commit into CVS with the first attempt at
removing use of mktemp().  Many files are affected,
so hopefully, others will check-out and test.

Earl Hood <ehood>
Group Member
Tue 02 Feb 2010 08:28:24 PM UTC, comment #3: 

I've taken an initial pass at the code base to replace
the usage of mktemp().  Diff of code has been posted
to nmh-workers list for review.  If there are no
major objects, I will commit changes, making it
easier for others to test things out.

Earl Hood <ehood>
Group Member
Sun 31 Dec 2006 03:44:55 AM UTC, comment #2: 

Here's a patch for mshcmds.c

(file #11649)

Craig Leres <leres>
Sun 31 Dec 2006 03:40:05 AM UTC, comment #1: 

This bug report addresses the mktemp() call in m_scratch.c:

    https://savannah.nongnu.org/bugs/index.php?18635

Craig Leres <leres>
Thu 15 Dec 2005 01:42:07 PM UTC, original submission:  

[This is debian bug #135972]

nmh uses mktemp() in a number of places. The man page says "never use mktemp()" and the linker bitches about it, because it's insecure. We need to investigate this and identify the right approach. (manpage link chasing suggests tmpfile() but since that returns a FILE* not a filename this may require reworking of the internals of nmh.)

Peter Maydell <pm215>
Group administrator

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #11649:  mshcmds.c.patch added by leres (1KiB - application/octet-stream)

 

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)
  • -email is unavailable- added by ehood (Posted a comment)
  • -email is unavailable- added by pm215 (Updated the item)
  • -email is unavailable- added by leres (Posted a comment)
  •  

    Follow 5 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2012-02-27 kenh StatusNone Fixed
        Open/ClosedOpen Closed
    2008-12-29 pm215 CategoryNone Bug
    2006-12-31 leres Attached File- Added mshcmds.c.patch, #11649
    2006-12-31 leres Carbon-Copy- Added -email is unavailable-

    Back to the top

    Powered by Savane 3.13-4b48.
    Corresponding source code