← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/atomicfile.py

  • Committer: John Arbash Meinel
  • Date: 2009-08-26 16:03:59 UTC
  • mto: (4634.6.7 2.0)
  • mto: This revision was merged to the branch mainline in revision 4660.
  • Revision ID: john@arbash-meinel.com-20090826160359-ge4mai928bi3a5g2
https://launchpad.net/bugs/419241
Fix bug #419241. If a graph had a mainline ghost
we could get a segfault during KnownGraph.merge_sort().

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/atomicfile.py
1
 
# Copyright (C) 2004, 2005 by Canonical Ltd
 
1
# Copyright (C) 2004, 2005 Canonical Ltd
2
2
#
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
12
12
#
13
13
# You should have received a copy of the GNU General Public License
14
14
# along with this program; if not, write to the Free Software
15
 
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
16
 
 
17
 
 
18
 
import codecs
19
 
import errno
 
15
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
 
16
 
 
17
 
20
18
import os
 
19
import sys
 
20
 
 
21
from bzrlib.lazy_import import lazy_import
 
22
lazy_import(globals(), """
21
23
import stat
22
24
import socket
23
 
import sys
24
25
import warnings
25
26
 
26
27
from bzrlib import (
28
29
    osutils,
29
30
    symbol_versioning,
30
31
    )
31
 
from bzrlib.osutils import rename
 
32
""")
32
33
 
33
34
# not forksafe - but we dont fork.
34
35
_pid = os.getpid()
35
 
_hostname = socket.gethostname()
 
36
_hostname = None
36
37
 
37
38
 
38
39
class AtomicFile(object):
47
48
    __slots__ = ['tmpfilename', 'realfilename', '_fd']
48
49
 
49
50
    def __init__(self, filename, mode='wb', new_mode=None):
 
51
        global _hostname
 
52
 
50
53
        self._fd = None
51
 
        assert mode in ('wb', 'wt'), \
52
 
            "invalid AtomicFile mode %r" % mode
53
 
 
54
 
        self.tmpfilename = '%s.%d.%s.tmp' % (filename, _pid, _hostname)
 
54
 
 
55
        if _hostname is None:
 
56
            _hostname = osutils.get_host_name()
 
57
 
 
58
        self.tmpfilename = '%s.%d.%s.%s.tmp' % (filename, _pid, _hostname,
 
59
                                                osutils.rand_chars(10))
55
60
 
56
61
        self.realfilename = filename
57
 
        
 
62
 
58
63
        flags = os.O_EXCL | os.O_CREAT | os.O_WRONLY
59
64
        if mode == 'wb':
60
65
            flags |= osutils.O_BINARY
 
66
        elif mode != 'wt':
 
67
            raise ValueError("invalid AtomicFile mode %r" % mode)
61
68
 
62
69
        if new_mode is not None:
63
70
            local_mode = new_mode
64
71
        else:
65
72
            local_mode = 0666
66
 
        
 
73
 
67
74
        # Use a low level fd operation to avoid chmodding later.
68
75
        # This may not succeed, but it should help most of the time
69
76
        self._fd = os.open(self.tmpfilename, flags, local_mode)
102
109
    def commit(self):
103
110
        """Close the file and move to final name."""
104
111
        self._close_tmpfile('commit')
105
 
        rename(self.tmpfilename, self.realfilename)
 
112
        osutils.rename(self.tmpfilename, self.realfilename)
106
113
 
107
114
    def abort(self):
108
115
        """Discard temporary file without committing changes."""