From d4c8a60358a87b319a9c348b0542ed501746b0f4 Mon Sep 17 00:00:00 2001
From: pweilbacher <pweilbacher@noemail.net>
Date: Thu, 14 Feb 2008 00:00:50 +0000
Subject: [PATCH] small correctness fix for os2CheckReservedLock() (CVS 4785)

FossilOrigin-Name: f364d9342345cb11a4fd1961ab4f6387fb6abacc
---
 manifest      | 12 ++++++------
 manifest.uuid |  2 +-
 src/os_os2.c  | 18 ++++++++++--------
 3 files changed, 17 insertions(+), 15 deletions(-)

diff --git a/manifest b/manifest
index 68f3ec81f4..8c7f0575cd 100644
--- a/manifest
+++ b/manifest
@@ -1,5 +1,5 @@
-C always\suse\srandom\saccess\smode\swhen\sopening\sfiles\s(like\son\sWindows)\s(CVS\s4784)
-D 2008-02-13T23:48:03
+C small\scorrectness\sfix\sfor\sos2CheckReservedLock()\s(CVS\s4785)
+D 2008-02-14T00:00:50
 F Makefile.arm-wince-mingw32ce-gcc ac5f7b2cef0cd850d6f755ba6ee4ab961b1fadf7
 F Makefile.in bc2b5df3e3d0d4b801b824b7ef6dec43812b049b
 F Makefile.linux-gcc d53183f4aa6a9192d249731c90dbdffbd2c68654
@@ -117,7 +117,7 @@ F src/mutex_w32.c 6e197765f283815496193e78e9548b5d0e53b68e
 F src/os.c 2f2753b8d33f79d169c43d6bb0b25b3c58fd33de
 F src/os.h d04706d54a072c7a30ab9e346ad916ef28c842d5
 F src/os_common.h 98862f120ca6bf7a48ce8b16f158b77d00bc9d2f
-F src/os_os2.c c87c6501bd90896cfca822a4e4d0ac5a79726833
+F src/os_os2.c 10b23539e0050bdfc9f136242086a5c18c70c6f8
 F src/os_os2.h c3f7d0af7e3453d1d7aa81b06c0a56f5a226530b
 F src/os_test.c 49833426101f99aee4bb5f6a44b7c4b2029fda1c
 F src/os_test.h 903c93554c23d88f34f667f1979e4a1cee792af3
@@ -617,7 +617,7 @@ F www/tclsqlite.tcl 8be95ee6dba05eabcd27a9d91331c803f2ce2130
 F www/vdbe.tcl 87a31ace769f20d3627a64fa1fade7fed47b90d0
 F www/version3.tcl 890248cf7b70e60c383b0e84d77d5132b3ead42b
 F www/whentouse.tcl fc46eae081251c3c181bd79c5faef8195d7991a5
-P 990237e27e417aff3dbf05784b716c21f3761a3a
-R 3788206f84973cfe13dcb12194a04a3f
+P 9f4da1013b558fcabadc3a3bec6bcdae9b06399e
+R 94f187d6e650525a58156f7c677755bf
 U pweilbacher
-Z e273b159944d84bb5a06e978a939110a
+Z 887a4d708e81eb8b0e1ad4f9e83b5340
diff --git a/manifest.uuid b/manifest.uuid
index 4eca3aa07e..6fe0200ef4 100644
--- a/manifest.uuid
+++ b/manifest.uuid
@@ -1 +1 @@
-9f4da1013b558fcabadc3a3bec6bcdae9b06399e
\ No newline at end of file
+f364d9342345cb11a4fd1961ab4f6387fb6abacc
\ No newline at end of file
diff --git a/src/os_os2.c b/src/os_os2.c
index 5a5f1fad62..0b4f7a71e0 100644
--- a/src/os_os2.c
+++ b/src/os_os2.c
@@ -422,15 +422,16 @@ int os2Lock( sqlite3_file *id, int locktype ){
 ** non-zero, otherwise zero.
 */
 int os2CheckReservedLock( sqlite3_file *id ){
-  APIRET rc = NO_ERROR;
+  int r = 0;
   os2File *pFile = (os2File*)id;
   assert( pFile!=0 );
   if( pFile->locktype>=RESERVED_LOCK ){
-    rc = 1;
-    OSTRACE3( "TEST WR-LOCK %d %d (local)\n", pFile->h, rc );
+    r = 1;
+    OSTRACE3( "TEST WR-LOCK %d %d (local)\n", pFile->h, r );
   }else{
     FILELOCK  LockArea,
               UnlockArea;
+    APIRET rc = NO_ERROR;
     memset(&LockArea, 0, sizeof(LockArea));
     memset(&UnlockArea, 0, sizeof(UnlockArea));
     LockArea.lOffset = RESERVED_BYTE;
@@ -440,17 +441,18 @@ int os2CheckReservedLock( sqlite3_file *id ){
     rc = DosSetFileLocks( pFile->h, &UnlockArea, &LockArea, 2000L, 1L );
     OSTRACE3( "TEST WR-LOCK %d lock reserved byte rc=%d\n", pFile->h, rc );
     if( rc == NO_ERROR ){
-      int r;
+      APIRET rcu = NO_ERROR; /* return code for unlocking */
       LockArea.lOffset = 0L;
       LockArea.lRange = 0L;
       UnlockArea.lOffset = RESERVED_BYTE;
       UnlockArea.lRange = 1L;
-      r = DosSetFileLocks( pFile->h, &UnlockArea, &LockArea, 2000L, 1L );
-      OSTRACE3( "TEST WR-LOCK %d unlock reserved byte r=%d\n", pFile->h, r );
+      rcu = DosSetFileLocks( pFile->h, &UnlockArea, &LockArea, 2000L, 1L );
+      OSTRACE3( "TEST WR-LOCK %d unlock reserved byte r=%d\n", pFile->h, rcu );
     }
-    OSTRACE3( "TEST WR-LOCK %d %d (remote)\n", pFile->h, rc );
+    r = !(rc == NO_ERROR);
+    OSTRACE3( "TEST WR-LOCK %d %d (remote)\n", pFile->h, r );
   }
-  return rc;
+  return r;
 }
 
 /*
-- 
2.47.3