File ginac-replace-mktemp-with-mkstemp.patch of Package ginac

Overview Repositories Revisions Requests Users Attributes Meta

File ginac-replace-mktemp-with-mkstemp.patch of Package ginac (Revision e6f1dfceb5a00369758b191fb7943c64)

Currently displaying revision e6f1dfceb5a00369758b191fb7943c64 , Show latest

From 8305ec388e5c205b7392566d32582f9d08adb88d Mon Sep 17 00:00:00 2001
From: Till Hofmann <thofmann@fedoraproject.org>
Date: Sat, 8 Oct 2016 20:34:40 +0200
Subject: [PATCH] Replace mktemp with mkstemp.

As the manpage says, mktemp is deprecated ("Never use mktemp") and gcc
warns about the usage of mktemp. Replace mktemp with its safe variant
mkstemp.
---
 ginac/excompiler.cpp | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/ginac/excompiler.cpp b/ginac/excompiler.cpp
index 68e4a30..6b1ae4c 100644
--- a/ginac/excompiler.cpp
+++ b/ginac/excompiler.cpp
@@ -38,6 +38,8 @@
 #ifdef HAVE_LIBDL
 #include <dlfcn.h>
 #endif // def HAVE_LIBDL
+#include <unistd.h>
+#include <stdlib.h>
 #include <fstream>
 #include <ios>
 #include <sstream>
@@ -48,7 +50,7 @@
 namespace GiNaC {
 
 #ifdef HAVE_LIBDL
-	
+
 /**
  * Small class that manages modules opened by libdl. It is used by compile_ex
  * and link_ex in order to have a clean-up of opened modules and their
@@ -112,12 +114,14 @@ public:
 			const char* filename_pattern = "./GiNaCXXXXXX";
 			char* new_filename = new char[strlen(filename_pattern)+1];
 			strcpy(new_filename, filename_pattern);
-			if (!mktemp(new_filename)) {
+			int fd = mkstemp(new_filename);
+			if (fd == -1) {
 				delete[] new_filename;
-				throw std::runtime_error("mktemp failed");
+				throw std::runtime_error("mkstemp failed");
 			}
 			filename = std::string(new_filename);
 			ofs.open(new_filename, std::ios::out);
+			close(fd);
 			delete[] new_filename;
 		} else {
 			// use parameter as filename
@@ -155,7 +159,7 @@ public:
 	{
 		void* module = nullptr;
 		module = dlopen(filename.c_str(), RTLD_NOW);
-		if (module == nullptr)	{
+		if (module == nullptr) {
 			throw std::runtime_error("excompiler::link_so_file: could not open compiled module!");
 		}
 
-- 
2.1.4

Places

File ginac-replace-mktemp-with-mkstemp.patch of Package ginac (Revision e6f1dfceb5a00369758b191fb7943c64)

Places