From 604379f16d657b55c695cca02d668bfe91d8e4ae Mon Sep 17 00:00:00 2001
From: ben <ben@cda61777-01e9-0310-a592-d414129be87e>
Date: Sat, 18 Jan 2003 12:03:28 +0000
Subject: [PATCH] Simplify the random-seed-saving code:  There's no need to
 take great care to preserve the old file in case the update is interrupted
 and replace it atomically, so we just overwrite in place.  Much simpler.

git-svn-id: svn://svn.tartarus.org/sgt/putty@2635 cda61777-01e9-0310-a592-d414129be87e
---
 mac/macstore.c | 42 +++++++++++++-----------------------------
 1 file changed, 13 insertions(+), 29 deletions(-)

diff --git a/mac/macstore.c b/mac/macstore.c
index 9d3f25c6..910b6c7e 100644
--- a/mac/macstore.c
+++ b/mac/macstore.c
@@ -1,4 +1,4 @@
-/* $Id: macstore.c,v 1.8 2003/01/14 19:09:24 ben Exp $ */
+/* $Id: macstore.c,v 1.9 2003/01/18 12:03:28 ben Exp $ */
 
 /*
  * macstore.c: Macintosh-specific impementation of the interface
@@ -390,12 +390,16 @@ void read_random_seed(noise_consumer_t consumer)
     FSClose(refnum);
 }
 
+/*
+ * We don't bother with the usual FSpExchangeFiles dance here because
+ * it doesn't really matter if the old random seed gets lost.
+ */
 void write_random_seed(void *data, int len)
 {
-    short puttyVRefNum, tmpVRefNum;
-    long puttyDirID, tmpDirID;
+    short puttyVRefNum;
+    long puttyDirID;
     OSErr error;
-    FSSpec dstfile, tmpfile;
+    FSSpec dstfile;
     short refnum;
     long count = len;
 
@@ -404,36 +408,16 @@ void write_random_seed(void *data, int len)
 
     error = FSMakeFSSpec(puttyVRefNum, puttyDirID, "\pPuTTY Random Seed",
 			 &dstfile);
-    if (error != noErr && error != fnfErr) return;
-
-    /* Create a temporary file to save to first. */
-    error = FindFolder(puttyVRefNum, kTemporaryFolderType, kCreateFolder,
-		       &tmpVRefNum, &tmpDirID);
+    if (error == fnfErr)
+	error = FSpCreate(&dstfile, PUTTY_CREATOR, SEED_TYPE, smRoman);
     if (error != noErr) return;
-    error = FSMakeFSSpec(tmpVRefNum, tmpDirID, "\pPuTTY Random Seed",
-			 &tmpfile);
-    if (error != noErr && error != fnfErr) return;
-    if (error == noErr) {
-	error = FSpDelete(&tmpfile);
-	if (error != noErr) return;
-    }
-    error = FSpCreate(&tmpfile, PUTTY_CREATOR, SEED_TYPE, smRoman);
-    if (error != noErr) return;
-
-    if (FSpOpenDF(&tmpfile, fsWrPerm, &refnum) != noErr) goto fail;
 
-    if (FSWrite(refnum, &count, data) != noErr) goto fail2;
-    if (FSClose(refnum) != noErr) goto fail;
+    if (FSpOpenDF(&dstfile, fsWrPerm, &refnum) != noErr) return;
 
-    if (FSpExchangeFiles(&tmpfile, &dstfile) != noErr) goto fail;
-    if (FSpDelete(&tmpfile) != noErr) return;
+    FSWrite(refnum, &count, data);
 
-    return;
-    
-  fail2:
     FSClose(refnum);
-  fail:
-    FSpDelete(&tmpfile);
+    return;
 }
 
 /*
-- 
2.11.0