use putenv instead of setenv for portability
[oweals/gnunet.git] / src / util / disk.c
index 6ba5db773212d0bc29bd89ee62e34bf2868d44ad..2bdcf3262ae0761ea2146ba6a3da58ed1a48ea41 100644 (file)
 #include "gnunet_crypto_lib.h"
 #include "disk.h"
 
-#define DEBUG_NPIPE GNUNET_NO
+#define LOG(kind,...) GNUNET_log_from (kind, "util", __VA_ARGS__)
 
-#define DEBUG_PIPE GNUNET_NO
+#define LOG_STRERROR(kind,syscall) GNUNET_log_from_strerror (kind, "util", syscall)
+
+#define LOG_STRERROR_FILE(kind,syscall,filename) GNUNET_log_from_strerror_file (kind, "util", syscall, filename)
+
+#define DEBUG_NPIPE GNUNET_EXTRA_LOGGING
+
+#define DEBUG_PIPE GNUNET_EXTRA_LOGGING
 
 /**
  * Block size for IO for copying files.
@@ -139,7 +145,7 @@ translate_unix_perms (enum GNUNET_DISK_AccessPermissions perm)
 
 
 /**
- * Iterate over all files in the given directory and 
+ * Iterate over all files in the given directory and
  * accumulate their size.
  *
  * @param cls closure of type "struct GetFileSizeData"
@@ -160,13 +166,13 @@ getSizeRec (void *cls, const char *fn)
 #ifdef HAVE_STAT64
   if (0 != STAT64 (fn, &buf))
   {
-    GNUNET_log_strerror_file (GNUNET_ERROR_TYPE_WARNING, "stat64", fn);
+    LOG_STRERROR_FILE (GNUNET_ERROR_TYPE_WARNING, "stat64", fn);
     return GNUNET_SYSERR;
   }
 #else
   if (0 != STAT (fn, &buf))
   {
-    GNUNET_log_strerror_file (GNUNET_ERROR_TYPE_WARNING, "stat", fn);
+    LOG_STRERROR_FILE (GNUNET_ERROR_TYPE_WARNING, "stat", fn);
     return GNUNET_SYSERR;
   }
 #endif
@@ -339,7 +345,7 @@ GNUNET_DISK_file_get_identifiers (const char *filename, uint64_t * dev,
  * an absolute path, the current 'TMPDIR' will be prepended.  In any case,
  * 6 random characters will be appended to the name to create a unique
  * filename.
- * 
+ *
  * @param t component to use for the name;
  *        does NOT contain "XXXXXX" or "/tmp/".
  * @return NULL on error, otherwise name of fresh
@@ -382,12 +388,12 @@ GNUNET_DISK_mktemp (const char *t)
   fd = mkstemp (fn);
   if (fd == -1)
   {
-    GNUNET_log_strerror_file (GNUNET_ERROR_TYPE_ERROR, "mkstemp", fn);
+    LOG_STRERROR_FILE (GNUNET_ERROR_TYPE_ERROR, "mkstemp", fn);
     GNUNET_free (fn);
     return NULL;
   }
   if (0 != CLOSE (fd))
-    GNUNET_log_strerror_file (GNUNET_ERROR_TYPE_WARNING, "close", fn);
+    LOG_STRERROR_FILE (GNUNET_ERROR_TYPE_WARNING, "close", fn);
   return fn;
 }
 
@@ -407,7 +413,7 @@ GNUNET_DISK_get_blocks_available (const char *part)
 
   if (0 != statvfs (part, &buf))
   {
-    GNUNET_log_strerror_file (GNUNET_ERROR_TYPE_WARNING, "statfs", part);
+    LOG_STRERROR_FILE (GNUNET_ERROR_TYPE_WARNING, "statfs", part);
     return -1;
   }
   return buf.f_bavail;
@@ -425,9 +431,8 @@ GNUNET_DISK_get_blocks_available (const char *part)
   szDrive[3] = 0;
   if (!GetDiskFreeSpace (szDrive, &dwDummy, &dwDummy, &dwBlocks, &dwDummy))
   {
-    GNUNET_log (GNUNET_ERROR_TYPE_WARNING,
-                _("`%s' failed for drive `%s': %u\n"), "GetDiskFreeSpace",
-                szDrive, GetLastError ());
+    LOG (GNUNET_ERROR_TYPE_WARNING, _("`%s' failed for drive `%s': %u\n"),
+         "GetDiskFreeSpace", szDrive, GetLastError ());
 
     return -1;
   }
@@ -437,7 +442,7 @@ GNUNET_DISK_get_blocks_available (const char *part)
 
   if (0 != statfs (part, &s))
   {
-    GNUNET_log_strerror_file (GNUNET_ERROR_TYPE_WARNING, "statfs", part);
+    LOG_STRERROR_FILE (GNUNET_ERROR_TYPE_WARNING, "statfs", part);
     return -1;
   }
   return s.f_bavail;
@@ -466,7 +471,7 @@ GNUNET_DISK_directory_test (const char *fil)
   {
     if (errno != ENOENT)
     {
-      GNUNET_log_strerror_file (GNUNET_ERROR_TYPE_WARNING, "stat", fil);
+      LOG_STRERROR_FILE (GNUNET_ERROR_TYPE_WARNING, "stat", fil);
       return GNUNET_SYSERR;
     }
     return GNUNET_NO;
@@ -475,7 +480,7 @@ GNUNET_DISK_directory_test (const char *fil)
     return GNUNET_NO;
   if (ACCESS (fil, R_OK | X_OK) < 0)
   {
-    GNUNET_log_strerror_file (GNUNET_ERROR_TYPE_WARNING, "access", fil);
+    LOG_STRERROR_FILE (GNUNET_ERROR_TYPE_WARNING, "access", fil);
     return GNUNET_SYSERR;
   }
   return GNUNET_YES;
@@ -505,7 +510,7 @@ GNUNET_DISK_file_test (const char *fil)
   {
     if (errno != ENOENT)
     {
-      GNUNET_log_strerror_file (GNUNET_ERROR_TYPE_WARNING, "stat", rdir);
+      LOG_STRERROR_FILE (GNUNET_ERROR_TYPE_WARNING, "stat", rdir);
       GNUNET_free (rdir);
       return GNUNET_SYSERR;
     }
@@ -519,7 +524,7 @@ GNUNET_DISK_file_test (const char *fil)
   }
   if (ACCESS (rdir, R_OK) < 0)
   {
-    GNUNET_log_strerror_file (GNUNET_ERROR_TYPE_WARNING, "access", rdir);
+    LOG_STRERROR_FILE (GNUNET_ERROR_TYPE_WARNING, "access", rdir);
     GNUNET_free (rdir);
     return GNUNET_SYSERR;
   }
@@ -588,7 +593,7 @@ GNUNET_DISK_directory_create (const char *dir)
 #endif
         if ((ret != 0) && (errno != EEXIST))
         {
-          GNUNET_log_strerror_file (GNUNET_ERROR_TYPE_ERROR, "mkdir", rdir);
+          LOG_STRERROR_FILE (GNUNET_ERROR_TYPE_ERROR, "mkdir", rdir);
           GNUNET_free (rdir);
           return GNUNET_SYSERR;
         }
@@ -736,7 +741,7 @@ GNUNET_DISK_file_write (const struct GNUNET_DISK_FileHandle * h,
   else
   {
 #if DEBUG_PIPE
-    GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "It is a pipe trying to write\n");
+    LOG (GNUNET_ERROR_TYPE_DEBUG, "It is a pipe trying to write\n");
 #endif
     if (!WriteFile (h->h, buffer, n, NULL, h->oOverlapWrite))
     {
@@ -744,13 +749,13 @@ GNUNET_DISK_file_write (const struct GNUNET_DISK_FileHandle * h,
       {
         SetErrnoFromWinError (GetLastError ());
 #if DEBUG_PIPE
-        GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Error writing to pipe\n");
+        LOG (GNUNET_ERROR_TYPE_DEBUG, "Error writing to pipe\n");
 #endif
         return GNUNET_SYSERR;
       }
     }
 #if DEBUG_PIPE
-    GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Will get overlapped result\n");
+    LOG (GNUNET_ERROR_TYPE_DEBUG, "Will get overlapped result\n");
 #endif
     GetOverlappedResult (h->h, h->oOverlapWrite, &bytesWritten, TRUE);
   }
@@ -767,7 +772,7 @@ GNUNET_DISK_file_write (const struct GNUNET_DISK_FileHandle * h,
  * @param fn file name
  * @param buffer the data to write
  * @param n number of bytes to write
- * @param mode file permissions 
+ * @param mode file permissions
  * @return number of bytes written on success, GNUNET_SYSERR on error
  */
 ssize_t
@@ -787,8 +792,9 @@ GNUNET_DISK_fn_write (const char *fn, const void *buffer, size_t n,
   return ret;
 }
 
+
 /**
- * Scan a directory for files. 
+ * Scan a directory for files.
  *
  * @param dirName the name of the directory
  * @param callback the method to call for each file,
@@ -819,14 +825,14 @@ GNUNET_DISK_directory_scan (const char *dirName,
     dname[strlen (dname) - 1] = '\0';
   if (0 != STAT (dname, &istat))
   {
-    GNUNET_log_strerror_file (GNUNET_ERROR_TYPE_WARNING, "stat", dname);
+    LOG_STRERROR_FILE (GNUNET_ERROR_TYPE_WARNING, "stat", dname);
     GNUNET_free (dname);
     return GNUNET_SYSERR;
   }
   if (!S_ISDIR (istat.st_mode))
   {
-    GNUNET_log (GNUNET_ERROR_TYPE_WARNING,
-                _("Expected `%s' to be a directory!\n"), dirName);
+    LOG (GNUNET_ERROR_TYPE_WARNING, _("Expected `%s' to be a directory!\n"),
+         dirName);
     GNUNET_free (dname);
     return GNUNET_SYSERR;
   }
@@ -834,7 +840,7 @@ GNUNET_DISK_directory_scan (const char *dirName,
   dinfo = OPENDIR (dname);
   if ((errno == EACCES) || (dinfo == NULL))
   {
-    GNUNET_log_strerror_file (GNUNET_ERROR_TYPE_WARNING, "opendir", dname);
+    LOG_STRERROR_FILE (GNUNET_ERROR_TYPE_WARNING, "opendir", dname);
     if (dinfo != NULL)
       closedir (dinfo);
     GNUNET_free (dname);
@@ -1057,7 +1063,7 @@ GNUNET_DISK_directory_remove (const char *fileName)
        * So we also explicitly check "isDirectory" */
       (GNUNET_YES != GNUNET_DISK_directory_test (fileName)))
   {
-    GNUNET_log_strerror_file (GNUNET_ERROR_TYPE_WARNING, "rmdir", fileName);
+    LOG_STRERROR_FILE (GNUNET_ERROR_TYPE_WARNING, "rmdir", fileName);
     return GNUNET_SYSERR;
   }
   if (GNUNET_SYSERR ==
@@ -1065,7 +1071,7 @@ GNUNET_DISK_directory_remove (const char *fileName)
     return GNUNET_SYSERR;
   if (0 != RMDIR (fileName))
   {
-    GNUNET_log_strerror_file (GNUNET_ERROR_TYPE_WARNING, "rmdir", fileName);
+    LOG_STRERROR_FILE (GNUNET_ERROR_TYPE_WARNING, "rmdir", fileName);
     return GNUNET_SYSERR;
   }
   return GNUNET_OK;
@@ -1176,13 +1182,13 @@ GNUNET_DISK_file_change_owner (const char *filename, const char *user)
   pws = getpwnam (user);
   if (pws == NULL)
   {
-    GNUNET_log (GNUNET_ERROR_TYPE_ERROR,
-                _("Cannot obtain information about user `%s': %s\n"), user,
-                STRERROR (errno));
+    LOG (GNUNET_ERROR_TYPE_ERROR,
+         _("Cannot obtain information about user `%s': %s\n"), user,
+         STRERROR (errno));
     return GNUNET_SYSERR;
   }
   if (0 != chown (filename, pws->pw_uid, pws->pw_gid))
-    GNUNET_log_strerror_file (GNUNET_ERROR_TYPE_WARNING, "chown", filename);
+    LOG_STRERROR_FILE (GNUNET_ERROR_TYPE_WARNING, "chown", filename);
 #endif
   return GNUNET_OK;
 }
@@ -1342,9 +1348,9 @@ GNUNET_DISK_file_open (const char *fn, enum GNUNET_DISK_OpenFlags flags,
   if (fd == -1)
   {
     if (0 == (flags & GNUNET_DISK_OPEN_FAILIFEXISTS))
-      GNUNET_log_strerror_file (GNUNET_ERROR_TYPE_WARNING, "open", expfn);
+      LOG_STRERROR_FILE (GNUNET_ERROR_TYPE_WARNING, "open", expfn);
     else
-      GNUNET_log_strerror_file (GNUNET_ERROR_TYPE_DEBUG, "open", expfn);
+      LOG_STRERROR_FILE (GNUNET_ERROR_TYPE_DEBUG, "open", expfn);
     GNUNET_free (expfn);
     return NULL;
   }
@@ -1387,7 +1393,7 @@ GNUNET_DISK_file_open (const char *fn, enum GNUNET_DISK_OpenFlags flags,
   if (h == INVALID_HANDLE_VALUE)
   {
     SetErrnoFromWinError (GetLastError ());
-    GNUNET_log_strerror_file (GNUNET_ERROR_TYPE_WARNING, "open", expfn);
+    LOG_STRERROR_FILE (GNUNET_ERROR_TYPE_WARNING, "open", expfn);
     GNUNET_free (expfn);
     return NULL;
   }
@@ -1396,8 +1402,7 @@ GNUNET_DISK_file_open (const char *fn, enum GNUNET_DISK_OpenFlags flags,
     if (SetFilePointer (h, 0, 0, FILE_END) == INVALID_SET_FILE_POINTER)
     {
       SetErrnoFromWinError (GetLastError ());
-      GNUNET_log_strerror_file (GNUNET_ERROR_TYPE_WARNING, "SetFilePointer",
-                                expfn);
+      LOG_STRERROR_FILE (GNUNET_ERROR_TYPE_WARNING, "SetFilePointer", expfn);
       CloseHandle (h);
       GNUNET_free (expfn);
       return NULL;
@@ -1434,7 +1439,7 @@ GNUNET_DISK_file_close (struct GNUNET_DISK_FileHandle *h)
   if (!CloseHandle (h->h))
   {
     SetErrnoFromWinError (GetLastError ());
-    GNUNET_log_strerror (GNUNET_ERROR_TYPE_WARNING, "close");
+    LOG_STRERROR (GNUNET_ERROR_TYPE_WARNING, "close");
     GNUNET_free (h->oOverlapRead);
     GNUNET_free (h->oOverlapWrite);
     GNUNET_free (h);
@@ -1443,7 +1448,7 @@ GNUNET_DISK_file_close (struct GNUNET_DISK_FileHandle *h)
 #else
   if (close (h->fd) != 0)
   {
-    GNUNET_log_strerror (GNUNET_ERROR_TYPE_WARNING, "close");
+    LOG_STRERROR (GNUNET_ERROR_TYPE_WARNING, "close");
     GNUNET_free (h);
     return GNUNET_SYSERR;
   }
@@ -1482,9 +1487,9 @@ GNUNET_DISK_get_home_filename (const struct GNUNET_CONFIGURATION_Handle *cfg,
     return NULL;
   if (pfx == NULL)
   {
-    GNUNET_log (GNUNET_ERROR_TYPE_WARNING,
-                _("No `%s' specified for service `%s' in configuration.\n"),
-                "HOME", serviceName);
+    LOG (GNUNET_ERROR_TYPE_WARNING,
+         _("No `%s' specified for service `%s' in configuration.\n"), "HOME",
+         serviceName);
     return NULL;
   }
   needed = strlen (pfx) + 2;
@@ -1735,8 +1740,8 @@ create_selectable_pipe (PHANDLE read_pipe_ptr, PHANDLE write_pipe_ptr,
     snprintf (pipename, sizeof pipename, "\\\\.\\pipe\\gnunet-%d-%ld",
               getpid (), InterlockedIncrement ((LONG *) & pipe_unique_id));
 #if DEBUG_PIPE
-    GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-                "CreateNamedPipe: name = %s, size = %lu\n", pipename, psize);
+    LOG (GNUNET_ERROR_TYPE_DEBUG, "CreateNamedPipe: name = %s, size = %lu\n",
+         pipename, psize);
 #endif
     /* Use CreateNamedPipe instead of CreatePipe, because the latter
      * returns a write handle that does not permit FILE_READ_ATTRIBUTES
@@ -1755,8 +1760,7 @@ create_selectable_pipe (PHANDLE read_pipe_ptr, PHANDLE write_pipe_ptr,
     if (read_pipe != INVALID_HANDLE_VALUE)
     {
 #if DEBUG_PIPE
-      GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "pipe read handle = %p\n",
-                  read_pipe);
+      LOG (GNUNET_ERROR_TYPE_DEBUG, "pipe read handle = %p\n", read_pipe);
 #endif
       break;
     }
@@ -1769,45 +1773,45 @@ create_selectable_pipe (PHANDLE read_pipe_ptr, PHANDLE write_pipe_ptr,
       /* The pipe is already open with compatible parameters.
        * Pick a new name and retry.  */
 #if DEBUG_PIPE
-      GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "pipe busy, retrying\n");
+      LOG (GNUNET_ERROR_TYPE_DEBUG, "pipe busy, retrying\n");
 #endif
       continue;
     case ERROR_ACCESS_DENIED:
       /* The pipe is already open with incompatible parameters.
        * Pick a new name and retry.  */
 #if DEBUG_PIPE
-      GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "pipe access denied, retrying\n");
+      LOG (GNUNET_ERROR_TYPE_DEBUG, "pipe access denied, retrying\n");
 #endif
       continue;
     case ERROR_CALL_NOT_IMPLEMENTED:
       /* We are on an older Win9x platform without named pipes.
        * Return an anonymous pipe as the best approximation.  */
 #if DEBUG_PIPE
-      GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-                  "CreateNamedPipe not implemented, resorting to "
-                  "CreatePipe: size = %lu\n", psize);
+      LOG (GNUNET_ERROR_TYPE_DEBUG,
+           "CreateNamedPipe not implemented, resorting to "
+           "CreatePipe: size = %lu\n", psize);
 #endif
       if (CreatePipe (read_pipe_ptr, write_pipe_ptr, sa_ptr, psize))
       {
 #if DEBUG_PIPE
-        GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "pipe read handle = %p\n",
-                    *read_pipe_ptr);
-        GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "pipe write handle = %p\n",
-                    *write_pipe_ptr);
+        LOG (GNUNET_ERROR_TYPE_DEBUG, "pipe read handle = %p\n",
+             *read_pipe_ptr);
+        LOG (GNUNET_ERROR_TYPE_DEBUG, "pipe write handle = %p\n",
+             *write_pipe_ptr);
 #endif
         return GNUNET_OK;
       }
       err = GetLastError ();
-      GNUNET_log (GNUNET_ERROR_TYPE_ERROR, "CreatePipe failed: %d\n", err);
+      LOG (GNUNET_ERROR_TYPE_ERROR, "CreatePipe failed: %d\n", err);
       return err;
     default:
-      GNUNET_log (GNUNET_ERROR_TYPE_ERROR, "CreateNamedPipe failed: %d\n", err);
+      LOG (GNUNET_ERROR_TYPE_ERROR, "CreateNamedPipe failed: %d\n", err);
       return err;
     }
     /* NOTREACHED */
   }
 #if DEBUG_PIPE
-  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "CreateFile: name = %s\n", pipename);
+  LOG (GNUNET_ERROR_TYPE_DEBUG, "CreateFile: name = %s\n", pipename);
 #endif
 
   /* Open the named pipe for writing.
@@ -1822,13 +1826,13 @@ create_selectable_pipe (PHANDLE read_pipe_ptr, PHANDLE write_pipe_ptr,
     DWORD err = GetLastError ();
 
 #if DEBUG_PIPE
-    GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "CreateFile failed: %d\n", err);
+    LOG (GNUNET_ERROR_TYPE_DEBUG, "CreateFile failed: %d\n", err);
 #endif
     CloseHandle (read_pipe);
     return err;
   }
 #if DEBUG_PIPE
-  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "pipe write handle = %p\n", write_pipe);
+  LOG (GNUNET_ERROR_TYPE_DEBUG, "pipe write handle = %p\n", write_pipe);
 #endif
   /* Success. */
   *read_pipe_ptr = read_pipe;
@@ -1867,7 +1871,7 @@ GNUNET_DISK_pipe (int blocking, int inherit_read, int inherit_write)
   if (ret == -1)
   {
     eno = errno;
-    GNUNET_log_strerror (GNUNET_ERROR_TYPE_ERROR, "pipe");
+    LOG_STRERROR (GNUNET_ERROR_TYPE_ERROR, "pipe");
     GNUNET_free (p);
     errno = eno;
     return NULL;
@@ -1897,7 +1901,7 @@ GNUNET_DISK_pipe (int blocking, int inherit_read, int inherit_write)
   if (ret == -1)
   {
     eno = errno;
-    GNUNET_log_strerror (GNUNET_ERROR_TYPE_ERROR, "fcntl");
+    LOG_STRERROR (GNUNET_ERROR_TYPE_ERROR, "fcntl");
     GNUNET_break (0 == close (p->fd[0]->fd));
     GNUNET_break (0 == close (p->fd[1]->fd));
     GNUNET_free (p);
@@ -2116,8 +2120,8 @@ GNUNET_DISK_npipe_create (char **fn, enum GNUNET_DISK_OpenFlags flags,
     {
       GNUNET_asprintf (&name, "\\\\.\\pipe\\%.246s", fn);
 #if DEBUG_NPIPE
-      GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-                  "Trying to create an instance of named pipe `%s'\n", name);
+      LOG (GNUNET_ERROR_TYPE_DEBUG,
+           "Trying to create an instance of named pipe `%s'\n", name);
 #endif
       h = CreateNamedPipe (name, openMode | FILE_FLAG_OVERLAPPED,
                            PIPE_TYPE_BYTE | PIPE_READMODE_BYTE, 2, 1, 1, 0,
@@ -2129,8 +2133,8 @@ GNUNET_DISK_npipe_create (char **fn, enum GNUNET_DISK_OpenFlags flags,
                        GNUNET_CRYPTO_random_u64 (GNUNET_CRYPTO_QUALITY_WEAK,
                                                  UINT64_MAX));
 #if DEBUG_NPIPE
-      GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-                  "Trying to create unique named pipe `%s'\n", *fn);
+      LOG (GNUNET_ERROR_TYPE_DEBUG, "Trying to create unique named pipe `%s'\n",
+           *fn);
 #endif
       h = CreateNamedPipe (*fn,
                            openMode | FILE_FLAG_OVERLAPPED |
@@ -2146,15 +2150,15 @@ GNUNET_DISK_npipe_create (char **fn, enum GNUNET_DISK_OpenFlags flags,
     {
       SetErrnoFromWinError (error_code);
 #if DEBUG_NPIPE
-      GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-                  "Pipe creation have failed because of %d, errno is %d\n",
-                  error_code, errno);
+      LOG (GNUNET_ERROR_TYPE_DEBUG,
+           "Pipe creation have failed because of %d, errno is %d\n", error_code,
+           errno);
 #endif
       if (name == NULL)
       {
 #if DEBUG_NPIPE
-        GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-                    "Pipe was to be unique, considering re-creation\n");
+        LOG (GNUNET_ERROR_TYPE_DEBUG,
+             "Pipe was to be unique, considering re-creation\n");
 #endif
         GNUNET_free (*fn);
         *fn = NULL;
@@ -2163,8 +2167,8 @@ GNUNET_DISK_npipe_create (char **fn, enum GNUNET_DISK_OpenFlags flags,
           return NULL;
         }
 #if DEBUG_NPIPE
-        GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-                    "Pipe name was not unique, trying again\n");
+        LOG (GNUNET_ERROR_TYPE_DEBUG,
+             "Pipe name was not unique, trying again\n");
 #endif
         h = NULL;
       }
@@ -2192,7 +2196,7 @@ GNUNET_DISK_npipe_create (char **fn, enum GNUNET_DISK_OpenFlags flags,
 
     if (mkdtemp (dir) == NULL)
     {
-      GNUNET_log_strerror (GNUNET_ERROR_TYPE_ERROR, "mkdtemp");
+      LOG_STRERROR (GNUNET_ERROR_TYPE_ERROR, "mkdtemp");
       return NULL;
     }
     GNUNET_asprintf (fn, "%s/child-control", dir);