From 921e643b93c88c2a0b560bbc21989918cde47c60 Mon Sep 17 00:00:00 2001
From: Zooko O'Whielacronx <zooko@zooko.com>
Date: Tue, 22 May 2007 14:06:00 -0700
Subject: [PATCH] don't check for existence of portnum file and then try to
 open it -- instead try to open it and catch exception This avoids a race
 condition, also known as time-of-check-to-time-of-use.

---
 src/allmydata/node.py | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/src/allmydata/node.py b/src/allmydata/node.py
index 2611fddf..79dc3186 100644
--- a/src/allmydata/node.py
+++ b/src/allmydata/node.py
@@ -47,11 +47,12 @@ class Node(service.MultiService):
         f.write(idlib.b2a(self.nodeid) + "\n")
         f.close()
         self.short_nodeid = self.tub.tubID[:4] # ready for printing
-        portnum = 0
         assert self.PORTNUMFILE, "Your node.Node subclass must provide PORTNUMFILE"
         self._portnumfile = os.path.join(self.basedir, self.PORTNUMFILE)
-        if os.path.exists(self._portnumfile):
-            portnum = int(open(self._portnumfile, "r").read())
+        try:
+            portnum = int(open(self._portnumfile, "rU").read())
+        except EnvironmentError, ValueError:
+            portnum = 0
         self.tub.listenOn("tcp:%d" % portnum)
         # we must wait until our service has started before we can find out
         # our IP address and thus do tub.setLocation, and we can't register
-- 
2.45.2