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

 
 

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

Submitted by:  Peter Maydell <pm215>
Submitted on:  Thu 15 Dec 2005 01:42:07 PM UTC  
 
Category: BugSeverity: 3 - Normal
Priority: 5 - NormalStatus: Fixed
Assigned to: NoneOpen/Closed: Closed

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

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>
Project 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>
Project 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>
Project 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>
Project Administrator

 

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

Attach File(s):
   
   
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
  • -unavailable- added by kenh (Posted a comment)
  • -unavailable- added by ehood (Posted a comment)
  • -unavailable- added by pm215 (Updated the item)
  • -unavailable- added by leres (Posted a comment)
  •  

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 5 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Mon 27 Feb 2012 07:28:05 PM UTCkenhStatusNone=>Fixed
      Open/ClosedOpen=>Closed
    Mon 29 Dec 2008 10:41:32 PM UTCpm215CategoryNone=>Bug
    Sun 31 Dec 2006 03:44:55 AM UTCleresAttached File-=>Added mshcmds.c.patch, #11649
    Sun 31 Dec 2006 03:37:41 AM UTCleresCarbon-Copy-=>Added -unavailable-

    Back to the top


    Powered by Savane 3.1-cleanup1