Opened 12 years ago

Closed 12 years ago

#200 closed enhancement (fixed)

use mkstemp

Reported by: Heinrich Stamerjohanns Owned by: Nikolche Kolev
Priority: minor Milestone: Future
Component: conversion Version: 8.3
Keywords: Cc:
Complexity: Medium

Description

replace calls to tmpnam by mkstemp.

Change History (4)

comment:1 by Heinrich Stamerjohanns, 12 years ago

Owner: changed from Peter Baumann to Heinrich Stamerjohanns
Status: new → accepted

comment:2 by Peter Baumann, 12 years ago

Milestone: → Future
Priority: major → minor

comment:3 by Dimitar Misev, 12 years ago

Owner: changed from Heinrich Stamerjohanns to Nikolche Kolev
Status: accepted → assigned

@Nikolce

Find all the places where the tmpnam function is used in the code, and replace it with mkstemp.

Make sure that the systemtest gives same result before and after your changes. To run the systemtest:

cd systemtest
make check

Right now the output for me is

Detail test report is in testcases_open/test_select/log
testcases_fixed/test_memleak/test.sh ... OK
	see detail in testcases_fixed/test_memleak/log
testcases_fixed/test_memleak/test.sh ... OK
	see detail in testcases_fixed/test_memleak/log
testcases_mandatory/test_conversion/test.sh ... FAIL
	see detail in testcases_mandatory/test_conversion/log
testcases_mandatory/test_manipulation/test.sh ... OK
	see detail in testcases_mandatory/test_manipulation/log
testcases_mandatory/test_select/test.sh ... OK
	see detail in testcases_mandatory/test_select/log
testcases_mandatory/test_user/test.sh ... OK
	see detail in testcases_mandatory/test_user/log
testcases_open/test_select/test.sh ... OK
	see detail in testcases_open/test_select/log

comment:4 by Nikolche Kolev, 12 years ago

Complexity: → Medium
Resolution: → fixed
Status: assigned → closed
Note: See TracTickets for help on using tickets.