dirmngr: Print a WARNING status for DNS config problems.
[gnupg.git] / kbx / keybox-init.c
index cfee7b8..6a83f71 100644 (file)
@@ -1,4 +1,4 @@
-/* keybox-init.c - Initalization of the library
+/* keybox-init.c - Initialization of the library
  *     Copyright (C) 2001 Free Software Foundation, Inc.
  *
  * This file is part of GnuPG.
@@ -14,7 +14,7 @@
  * GNU General Public License for more details.
  *
  * You should have received a copy of the GNU General Public License
- * along with this program; if not, see <http://www.gnu.org/licenses/>.
+ * along with this program; if not, see <https://www.gnu.org/licenses/>.
  */
 
 #include <config.h>
@@ -266,10 +266,10 @@ _keybox_close_file (KEYBOX_HANDLE hd)
 gpg_error_t
 keybox_lock (KEYBOX_HANDLE hd, int yes)
 {
-  gpg_error_t err;
+  gpg_error_t err = 0;
   KB_NAME kb = hd->kb;
 
-  if (!keybox_is_writable ((void*)kb))
+  if (!keybox_is_writable (kb))
     return 0;
 
   /* Make sure the lock handle has been created.  */
@@ -278,38 +278,52 @@ keybox_lock (KEYBOX_HANDLE hd, int yes)
       kb->lockhd = dotlock_create (kb->fname, 0);
       if (!kb->lockhd)
         {
-          /* Unfortuntaley dotlock_create does not properly set ERRNO.  */
+          err = gpg_error_from_syserror ();
           log_info ("can't allocate lock for '%s'\n", kb->fname );
-          return gpg_error (GPG_ERR_GENERAL);
+          return err;
         }
     }
 
   if (yes) /* Take the lock.  */
     {
-      if (kb->is_locked)
-        ;
-      else if (!dotlock_take (kb->lockhd, -1))
-        kb->is_locked = 1;
-      else
+      if (!kb->is_locked)
         {
-          /* Unfortuntaley dotlock_take does not properly set ERRNO.  */
-          log_info ("can't lock '%s'\n", kb->fname );
-          err = gpg_error (GPG_ERR_GENERAL);
+#ifdef HAVE_W32_SYSTEM
+            /* Under Windows we need to close the file before we try
+             * to lock it.  This is because another process might have
+             * taken the lock and is using keybox_file_rename to
+             * rename the base file.  How if our dotlock_take below is
+             * waiting for the lock but we have the base file still
+             * open, keybox_file_rename will never succeed as we are
+             * in a deadlock.  */
+          if (hd->fp)
+            {
+              fclose (hd->fp);
+              hd->fp = NULL;
+            }
+#endif /*HAVE_W32_SYSTEM*/
+          if (dotlock_take (kb->lockhd, -1))
+            {
+              err = gpg_error_from_syserror ();
+              log_info ("can't lock '%s'\n", kb->fname );
+            }
+          else
+            kb->is_locked = 1;
         }
     }
   else /* Release the lock.  */
     {
-      if (!kb->is_locked)
-        ;
-      else if (!dotlock_release (kb->lockhd))
-        kb->is_locked = 0;
-      else
+      if (kb->is_locked)
         {
-          /* Unfortuntaley dotlock_release does not properly set ERRNO.  */
-          log_info ("can't unlock '%s'\n", kb->fname );
-          err = gpg_error (GPG_ERR_GENERAL);
+          if (dotlock_release (kb->lockhd))
+            {
+              err = gpg_error_from_syserror ();
+              log_info ("can't unlock '%s'\n", kb->fname );
+            }
+          else
+            kb->is_locked = 0;
         }
-    }
+   }
 
   return err;
 }