Use NamedTemporaryFile(delete=False) instead of mkstemp

weboob/applications/webcontentedit/webcontentedit.py

@@ -66,8 +66,7 @@ class WebContentEdit(ReplApplication):
                 tmpdir = os.path.join(tempfile.gettempdir(), "weboob")
                 if not os.path.isdir(tmpdir):
                     os.makedirs(tmpdir)
-                fd, path = tempfile.mkstemp(prefix='%s_' % content.id.replace(os.path.sep, '_'), dir=tmpdir)
-                with os.fdopen(fd, 'w') as f:
+                with tempfile.NamedTemporaryFile(prefix='%s_' % content.id.replace(os.path.sep, '_'), dir=tmpdir, delete=False) as f:
                     data = content.content
                     if isinstance(data, unicode):
                         data = data.encode('utf-8')
@@ -75,7 +74,7 @@ class WebContentEdit(ReplApplication):
                         content.content = u''
                         data = ''
                     f.write(data)
-                paths[path.encode('utf-8')] = content
+                paths[f.name.encode('utf-8')] = content
 
             params = ''
             editor = os.environ.get('EDITOR', 'vim')

weboob/tools/config/yamlconfig.py

@@ -18,8 +18,6 @@
 # along with weboob. If not, see <http://www.gnu.org/licenses/>.
 
 
-
-
 import os
 import tempfile
 import logging
@@ -60,10 +58,9 @@ class YamlConfig(IConfig):
 
     def save(self):
         # write in a temporary file to avoid corruption problems
-        fd, path = tempfile.mkstemp(dir=os.path.dirname(self.path))
-        with os.fdopen(fd, 'w') as f:
+        with tempfile.NamedTemporaryFile(dir=os.path.dirname(self.path), delete=False) as f:
             yaml.dump(self.values, f, Dumper=Dumper)
-        os.rename(path, self.path)
+        os.rename(f.name, self.path)
 
     def get(self, *args, **kwargs):
         default = None