implement extended status information for arm
[oweals/gnunet.git] / src / arm / test_exponential_backoff.c
index 0561c3ea83b20909a8eced836f537096b447de1a..b13ab1cb899ffa9d42e813b208cfcb246ed997e5 100644 (file)
@@ -1,42 +1,41 @@
 /*
      This file is part of GNUnet.
-     (C) 2009 Christian Grothoff (and other contributing authors)
+     Copyright (C) 2009, 2016 GNUnet e.V.
 
-     GNUnet is free software; you can redistribute it and/or modify
-     it under the terms of the GNU General Public License as published
-     by the Free Software Foundation; either version 3, or (at your
-     option) any later version.
+     GNUnet is free software: you can redistribute it and/or modify it
+     under the terms of the GNU Affero General Public License as published
+     by the Free Software Foundation, either version 3 of the License,
+     or (at your option) any later version.
 
      GNUnet is distributed in the hope that it will be useful, but
      WITHOUT ANY WARRANTY; without even the implied warranty of
      MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
-     General Public License for more details.
+     Affero General Public License for more details.
 
-     You should have received a copy of the GNU General Public License
-     along with GNUnet; see the file COPYING.  If not, write to the
-     Free Software Foundation, Inc., 59 Temple Place - Suite 330,
-     Boston, MA 02111-1307, USA.
-*/
+     You should have received a copy of the GNU Affero General Public License
+     along with this program.  If not, see <http://www.gnu.org/licenses/>.
+
+     SPDX-License-Identifier: AGPL3.0-or-later
+ */
 /**
  * @file arm/test_exponential_backoff.c
  * @brief testcase for gnunet-service-arm.c
+ * @author Christian Grothoff
  */
 #include "platform.h"
 #include "gnunet_arm_service.h"
 #include "gnunet_util_lib.h"
 #include "gnunet_protocols.h"
 
-#define LOG(...) GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, __VA_ARGS__)
-
-#define START_ARM GNUNET_YES
+#define LOG(...) GNUNET_log(GNUNET_ERROR_TYPE_DEBUG, __VA_ARGS__)
 
 #define LOG_BACKOFF GNUNET_NO
 
-#define TIMEOUT GNUNET_TIME_relative_multiply (GNUNET_TIME_UNIT_SECONDS, 10)
+#define TIMEOUT GNUNET_TIME_relative_multiply(GNUNET_TIME_UNIT_SECONDS, 10)
 
 #define SERVICE_TEST_TIMEOUT GNUNET_TIME_UNIT_FOREVER_REL
 
-#define FIVE_MILLISECONDS GNUNET_TIME_relative_multiply (GNUNET_TIME_UNIT_MILLISECONDS, 5)
+#define FIVE_MILLISECONDS GNUNET_TIME_relative_multiply(GNUNET_TIME_UNIT_MILLISECONDS, 5)
 
 #define SERVICE "do-nothing"
 
@@ -51,6 +50,8 @@ static struct GNUNET_ARM_Handle *arm;
 
 static struct GNUNET_ARM_MonitorHandle *mon;
 
+static struct GNUNET_SCHEDULER_Task *kt;
+
 static int ok = 1;
 
 static int phase = 0;
@@ -70,331 +71,237 @@ static char *killLogFileName;
 #endif
 
 
-typedef void (*GNUNET_CLIENT_ShutdownTask) (void *cls, int reason);
-
 /**
  * Context for handling the shutdown of a service.
  */
-struct ShutdownContext
-{
+struct ShutdownContext {
   /**
    * Connection to the service that is being shutdown.
    */
-  struct GNUNET_CLIENT_Connection *sock;
-
-  /**
-   * Time allowed for shutdown to happen.
-   */
-  struct GNUNET_TIME_Absolute timeout;
+  struct GNUNET_MQ_Handle *mq;
 
   /**
    * Task set up to cancel the shutdown request on timeout.
    */
-  GNUNET_SCHEDULER_TaskIdentifier cancel_task;
-
-  /**
-   * Task to call once shutdown complete
-   */
-  GNUNET_CLIENT_ShutdownTask cont;
-
-  /**
-   * Closure for shutdown continuation
-   */
-  void *cont_cls;
-
-  /**
-   * We received a confirmation that the service will shut down.
-   */
-  int confirmed;
-
+  struct GNUNET_SCHEDULER_Task *cancel_task;
 };
 
-/**
- * Handler receiving response to service shutdown requests.
- * We expect it to be called with NULL, since the service that
- * we are shutting down will just die without replying.
- *
- * @param cls closure
- * @param msg NULL, indicating socket closure.
- */
-static void
-service_shutdown_handler (void *cls, const struct GNUNET_MessageHeader *msg)
-{
-  struct ShutdownContext *shutdown_ctx = cls;
 
-  if (NULL == msg)
-  {
-    GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Service shutdown complete.\n");
-    if (shutdown_ctx->cont != NULL)
-      shutdown_ctx->cont (shutdown_ctx->cont_cls, GNUNET_NO);
-
-    GNUNET_SCHEDULER_cancel (shutdown_ctx->cancel_task);
-    GNUNET_CLIENT_disconnect (shutdown_ctx->sock);
-    GNUNET_free (shutdown_ctx);
-    return;
-  }
-  GNUNET_assert (0);
-}
+static void
+kill_task(void *cbData);
 
 
 /**
  * Shutting down took too long, cancel receive and return error.
  *
  * @param cls closure
- * @param tc context information (why was this task triggered now)
  */
-void
-service_shutdown_cancel (void *cls,
-                        const struct GNUNET_SCHEDULER_TaskContext *tc)
+static void
+service_shutdown_timeout(void *cls)
 {
-  struct ShutdownContext *shutdown_ctx = cls;
-
-  GNUNET_log (GNUNET_ERROR_TYPE_WARNING, "service_shutdown_cancel called!\n");
-  shutdown_ctx->cont (shutdown_ctx->cont_cls, GNUNET_SYSERR);
-  GNUNET_CLIENT_disconnect (shutdown_ctx->sock);
-  GNUNET_free (shutdown_ctx);
+  GNUNET_assert(0);
 }
 
 
 /**
- * If possible, write a shutdown message to the target
- * buffer and destroy the client connection.
+ * Generic error handler, called with the appropriate error code and
+ * the same closure specified at the creation of the message queue.
+ * Not every message queue implementation supports an error handler.
  *
- * @param cls the "struct GNUNET_CLIENT_Connection" to destroy
- * @param size number of bytes available in buf
- * @param buf NULL on error, otherwise target buffer
- * @return number of bytes written to buf
+ * @param cls closure with the `struct ShutdownContext *`
+ * @param error error code
  */
-static size_t
-write_shutdown (void *cls, size_t size, void *buf)
+static void
+mq_error_handler(void *cls,
+                 enum GNUNET_MQ_Error error)
 {
-  struct GNUNET_MessageHeader *msg;
   struct ShutdownContext *shutdown_ctx = cls;
 
-  if (size < sizeof (struct GNUNET_MessageHeader))
-  {
-    LOG ("Failed to send a shutdown request\n");
-    shutdown_ctx->cont (shutdown_ctx->cont_cls, GNUNET_SYSERR);
-    GNUNET_CLIENT_disconnect (shutdown_ctx->sock);
-    GNUNET_free (shutdown_ctx);
-    return 0;                  /* client disconnected */
-  }
-
-  GNUNET_CLIENT_receive (shutdown_ctx->sock, &service_shutdown_handler,
-                        shutdown_ctx, GNUNET_TIME_UNIT_FOREVER_REL);
-  shutdown_ctx->cancel_task = GNUNET_SCHEDULER_add_delayed (
-      GNUNET_TIME_absolute_get_remaining (shutdown_ctx->timeout),
-      &service_shutdown_cancel, shutdown_ctx);
-  msg = (struct GNUNET_MessageHeader *) buf;
-  msg->type = htons (GNUNET_MESSAGE_TYPE_ARM_STOP);
-  msg->size = htons (sizeof (struct GNUNET_MessageHeader));
-  strcpy ((char *) &msg[1], SERVICE);
-  LOG ("Sent a shutdown request\n");
-  return sizeof (struct GNUNET_MessageHeader) + strlen (SERVICE) + 1;
-}
-
-
-/**
- * Request that the service should shutdown.
- * Afterwards, the connection will automatically be
- * disconnected.  Hence the "sock" should not
- * be used by the caller after this call
- * (calling this function frees "sock" after a while).
- *
- * @param sock the socket connected to the service
- * @param timeout how long to wait before giving up on transmission
- * @param cont continuation to call once the service is really down
- * @param cont_cls closure for continuation
- *
- */
-static void
-do_nothing_service_shutdown (struct GNUNET_CLIENT_Connection *sock,
-                     struct GNUNET_TIME_Relative timeout,
-                     GNUNET_CLIENT_ShutdownTask cont, void *cont_cls)
-{
-  struct ShutdownContext *shutdown_ctx;
-
-  shutdown_ctx = GNUNET_malloc (sizeof (struct ShutdownContext));
-  shutdown_ctx->cont = cont;
-  shutdown_ctx->cont_cls = cont_cls;
-  shutdown_ctx->sock = sock;
-  shutdown_ctx->timeout = GNUNET_TIME_relative_to_absolute (timeout);
-  GNUNET_CLIENT_notify_transmit_ready (sock,
-                                      sizeof (struct GNUNET_MessageHeader) + strlen (SERVICE) + 1,
-                                      timeout, GNUNET_NO, &write_shutdown,
-                                      shutdown_ctx);
+  GNUNET_log(GNUNET_ERROR_TYPE_DEBUG,
+             "Service shutdown complete (MQ error).\n");
+  GNUNET_SCHEDULER_cancel(shutdown_ctx->cancel_task);
+  GNUNET_MQ_destroy(shutdown_ctx->mq);
+  GNUNET_free(shutdown_ctx);
 }
 
 
 static void
-kill_task (void *cbData, const struct GNUNET_SCHEDULER_TaskContext *tc);
-
-
-static void
-shutdown_cont (void *cls, int reason)
+kill_task(void *cbData)
 {
-  if (GNUNET_NO != reason)
-  {
-    /* Re-try shutdown */
-    LOG ("do-nothing didn't die, trying again\n");
-    GNUNET_SCHEDULER_add_now (kill_task, NULL);
-    return;
-  }
-  startedWaitingAt = GNUNET_TIME_absolute_get ();
-  LOG ("do-nothing is dead, starting the countdown\n");
-}
-
+  struct ShutdownContext *shutdown_ctx
+    = GNUNET_new(struct ShutdownContext);
+  struct GNUNET_MQ_Envelope *env;
+  struct GNUNET_MessageHeader *msg;
+  struct GNUNET_MQ_MessageHandler handlers[] = {
+    GNUNET_MQ_handler_end()
+  };
 
-static void
-kill_task (void *cbData, const struct GNUNET_SCHEDULER_TaskContext *tc)
-{
-  static struct GNUNET_CLIENT_Connection *doNothingConnection = NULL;
-
-  if (NULL != cbData)
-  {
-    waitedFor = GNUNET_TIME_absolute_get_duration (startedWaitingAt);
-    LOG ("Waited for: %s\n",
-        GNUNET_STRINGS_relative_time_to_string (waitedFor, GNUNET_YES));
-  }
-  else
-  {
-    waitedFor.rel_value_us = 0;
-  }
-  /* Connect to the doNothing task */
-  doNothingConnection = GNUNET_CLIENT_connect (SERVICE, cfg);
-  GNUNET_assert (doNothingConnection != NULL);
-  if (trialCount == 12)
-    waitedFor_prev = waitedFor;
-  else if (trialCount == 13)
-  {
-    GNUNET_CLIENT_disconnect (doNothingConnection);
-    GNUNET_ARM_request_service_stop (arm, SERVICE, TIMEOUT, NULL, NULL);
-    if (waitedFor_prev.rel_value_us >= waitedFor.rel_value_us)
-      ok = 9;
-    else
+  kt = NULL;
+  if (trialCount == 13)
+    {
+      LOG("Saw enough kills, asking ARM to stop mock service for good\n");
+      GNUNET_ARM_request_service_stop(arm,
+                                      SERVICE,
+                                      NULL,
+                                      NULL);
       ok = 0;
-    trialCount += 1;
-    return;
-  }
-  trialCount += 1;
-  /* Use the created connection to kill the doNothingTask */
-  do_nothing_service_shutdown (doNothingConnection,
-      TIMEOUT, &shutdown_cont, NULL);
+      trialCount++;
+      GNUNET_free(shutdown_ctx);
+      return;
+    }
+  shutdown_ctx->mq = GNUNET_CLIENT_connect(cfg,
+                                           SERVICE,
+                                           handlers,
+                                           &mq_error_handler,
+                                           shutdown_ctx);
+  GNUNET_assert(NULL != shutdown_ctx->mq);
+  trialCount++;
+  LOG("Sending a shutdown request to the mock service\n");
+  env = GNUNET_MQ_msg(msg,
+                      GNUNET_MESSAGE_TYPE_ARM_STOP);  /* FIXME: abuse of message type */
+  GNUNET_MQ_send(shutdown_ctx->mq,
+                 env);
+  shutdown_ctx->cancel_task
+    = GNUNET_SCHEDULER_add_delayed(TIMEOUT,
+                                   &service_shutdown_timeout,
+                                   shutdown_ctx);
 }
 
 
 static void
-trigger_disconnect (void *cls, const struct GNUNET_SCHEDULER_TaskContext *tc)
+trigger_disconnect(void *cls)
 {
-  GNUNET_ARM_disconnect_and_free (arm);
-  GNUNET_ARM_monitor_disconnect_and_free (mon);
+  GNUNET_ARM_disconnect(arm);
+  GNUNET_ARM_monitor_stop(mon);
+  if (NULL != kt)
+    {
+      GNUNET_SCHEDULER_cancel(kt);
+      kt = NULL;
+    }
 }
 
 
 static void
-arm_stop_cb (void *cls, enum GNUNET_ARM_RequestStatus status, const char *servicename, enum GNUNET_ARM_Result result)
+arm_stop_cb(void *cls,
+            enum GNUNET_ARM_RequestStatus status,
+            enum GNUNET_ARM_Result result)
 {
-  GNUNET_break (status == GNUNET_ARM_REQUEST_SENT_OK);
-  GNUNET_break (result == GNUNET_ARM_RESULT_STOPPED);
-  LOG ("ARM service stopped\n");
-  GNUNET_SCHEDULER_add_now (trigger_disconnect, NULL);
+  GNUNET_break(status == GNUNET_ARM_REQUEST_SENT_OK);
+  GNUNET_break(result == GNUNET_ARM_RESULT_STOPPED);
+  LOG("ARM service stopped\n");
+  GNUNET_SCHEDULER_shutdown();
 }
 
 
 static void
-srv_status (void *cls, const char *service, enum GNUNET_ARM_ServiceStatus status)
+srv_status(void *cls,
+           const char *service,
+           enum GNUNET_ARM_ServiceMonitorStatus status)
 {
-  LOG ("Service %s is %u, phase %u\n", service, status, phase);
   if (status == GNUNET_ARM_SERVICE_MONITORING_STARTED)
-  {
-    phase++;
-    GNUNET_ARM_request_service_start (arm, SERVICE,
-        GNUNET_OS_INHERIT_STD_OUT_AND_ERR, TIMEOUT, NULL, NULL);
-    return;
-  }
-  if (phase == 1)
-  {
-    GNUNET_break (status == GNUNET_ARM_SERVICE_STARTING);
-    GNUNET_break (0 == strcasecmp (service, SERVICE));
-    GNUNET_break (phase == 1);
-    LOG ("do-nothing is starting\n");
-    phase++;
-    ok = 1;
-    GNUNET_SCHEDULER_add_delayed (GNUNET_TIME_UNIT_SECONDS, &kill_task, NULL);
-  }
-  else if ((phase == 2) && (strcasecmp (SERVICE, service) == 0))
-  {
-    /* We passively monitor ARM for status updates. ARM should tell us
-     * when do-nothing dies (no need to run a service upness test ourselves).
-     */
-    if (status == GNUNET_ARM_SERVICE_STARTING)
     {
-      LOG ("do-nothing is starting\n");
-      GNUNET_SCHEDULER_add_now (kill_task, &ok);
+      LOG("ARM monitor started, starting mock service\n");
+      phase++;
+      GNUNET_ARM_request_service_start(arm,
+                                       SERVICE,
+                                       GNUNET_OS_INHERIT_STD_OUT_AND_ERR,
+                                       NULL,
+                                       NULL);
+      return;
     }
-    else if ((status == GNUNET_ARM_SERVICE_STOPPED) && (trialCount == 14))
+  if (0 != strcasecmp(service, SERVICE))
+    return; /* not what we care about */
+  if (phase == 1)
     {
+      GNUNET_break(status == GNUNET_ARM_SERVICE_STARTING);
+      GNUNET_break(phase == 1);
+      LOG("do-nothing is starting\n");
       phase++;
-      GNUNET_ARM_request_service_stop (arm, "arm", TIMEOUT, arm_stop_cb, NULL);
+      ok = 1;
+      GNUNET_assert(NULL == kt);
+      startedWaitingAt = GNUNET_TIME_absolute_get();
+      kt = GNUNET_SCHEDULER_add_delayed(GNUNET_TIME_UNIT_SECONDS,
+                                        &kill_task,
+                                        NULL);
+    }
+  else if (phase == 2)
+    {
+      /* We passively monitor ARM for status updates. ARM should tell us
+       * when do-nothing dies (no need to run a service upness test ourselves).
+       */
+      if (status == GNUNET_ARM_SERVICE_STARTING)
+        {
+          waitedFor = GNUNET_TIME_absolute_get_duration(startedWaitingAt);
+          LOG("Waited for: %s\n",
+              GNUNET_STRINGS_relative_time_to_string(waitedFor,
+                                                     GNUNET_YES));
+
+          LOG("do-nothing is starting, killing it...\n");
+          GNUNET_assert(NULL == kt);
+          kt = GNUNET_SCHEDULER_add_now(&kill_task, &ok);
+        }
+      else if ((status == GNUNET_ARM_SERVICE_STOPPED) && (trialCount == 14))
+        {
+          phase++;
+          LOG("do-nothing stopped working %u times, we are done here\n",
+              (unsigned int)trialCount);
+          GNUNET_ARM_request_service_stop(arm,
+                                          "arm",
+                                          &arm_stop_cb,
+                                          NULL);
+        }
     }
-  }
 }
 
 
 static void
-arm_start_cb (void *cls, enum GNUNET_ARM_RequestStatus status, const char *servicename, enum GNUNET_ARM_Result result)
+arm_start_cb(void *cls,
+             enum GNUNET_ARM_RequestStatus status,
+             enum GNUNET_ARM_Result result)
 {
-  GNUNET_break (status == GNUNET_ARM_REQUEST_SENT_OK);
-  GNUNET_break (result == GNUNET_ARM_RESULT_STARTING);
-  GNUNET_break (phase == 0);
-  LOG ("Sent 'START' request for arm to ARM %s\n",
-       (status == GNUNET_ARM_REQUEST_SENT_OK) ? "successfully" : "unsuccessfully");
+  GNUNET_break(status == GNUNET_ARM_REQUEST_SENT_OK);
+  GNUNET_break(result == GNUNET_ARM_RESULT_STARTING);
+  GNUNET_break(phase == 0);
+  LOG("Sent 'START' request for arm to ARM %s\n",
+      (status == GNUNET_ARM_REQUEST_SENT_OK) ? "successfully" : "unsuccessfully");
 }
 
 
 static void
-task (void *cls, char *const *args, const char *cfgfile,
-      const struct GNUNET_CONFIGURATION_Handle *c)
+task(void *cls,
+     char *const *args,
+     const char *cfgfile,
+     const struct GNUNET_CONFIGURATION_Handle *c)
 {
-  char *armconfig;
   cfg = c;
-  if (NULL != cfgfile)
-  {
-    if (GNUNET_OK != GNUNET_CONFIGURATION_get_value_filename (cfg, "arm",
-        "CONFIG", &armconfig))
+  arm = GNUNET_ARM_connect(cfg, NULL, NULL);
+  if (NULL == arm)
     {
-      GNUNET_CONFIGURATION_set_value_string ((struct GNUNET_CONFIGURATION_Handle
-                                              *) cfg, "arm", "CONFIG",
-                                             cfgfile);
+      GNUNET_break(0);
+      return;
     }
-    else
-      GNUNET_free (armconfig);
-  }
-
-  arm = GNUNET_ARM_connect (cfg, NULL, NULL);
-  if (NULL != arm)
-  {
-    mon = GNUNET_ARM_monitor (cfg, &srv_status, NULL);
-    if (NULL != mon)
+  mon = GNUNET_ARM_monitor_start(cfg,
+                                 &srv_status,
+                                 NULL);
+  if (NULL == mon)
     {
-#if START_ARM
-      GNUNET_ARM_request_service_start (arm, "arm",
-          GNUNET_OS_INHERIT_STD_OUT_AND_ERR, GNUNET_TIME_UNIT_ZERO, arm_start_cb, NULL);
-#else
-      arm_start_cb (NULL, arm, GNUNET_ARM_REQUEST_SENT_OK, "arm", GNUNET_ARM_SERVICE_STARTING);
-#endif
-    }
-    else
-    {
-      GNUNET_ARM_disconnect_and_free (arm);
+      GNUNET_break(0);
+      GNUNET_ARM_disconnect(arm);
       arm = NULL;
+      return;
     }
-  }
+  GNUNET_ARM_request_service_start(arm,
+                                   "arm",
+                                   GNUNET_OS_INHERIT_STD_OUT_AND_ERR,
+                                   &arm_start_cb,
+                                   NULL);
+  GNUNET_SCHEDULER_add_shutdown(&trigger_disconnect,
+                                NULL);
 }
 
 
 static int
-check ()
+check()
 {
   char *const argv[] = {
     "test-exponential-backoff",
@@ -406,46 +313,68 @@ check ()
   };
 
   /* Running ARM  and running the do_nothing task */
-  GNUNET_assert (GNUNET_OK ==
-                GNUNET_PROGRAM_run ((sizeof (argv) / sizeof (char *)) - 1,
-                                    argv, "test-exponential-backoff",
-                                    "nohelp", options, &task, NULL));
-
-
+  GNUNET_assert(GNUNET_OK ==
+                GNUNET_PROGRAM_run((sizeof(argv) / sizeof(char *)) - 1,
+                                   argv,
+                                   "test-exponential-backoff",
+                                   "nohelp",
+                                   options,
+                                   &task,
+                                   NULL));
   return ok;
 }
 
 
+#ifndef PATH_MAX
+/**
+ * Assumed maximum path length (for the log file name).
+ */
+#define PATH_MAX 4096
+#endif
+
+
 static int
-init ()
+init()
 {
   struct GNUNET_CONFIGURATION_Handle *cfg;
   char pwd[PATH_MAX];
   char *binary;
 
-  cfg = GNUNET_CONFIGURATION_create ();
-  if (GNUNET_OK != GNUNET_CONFIGURATION_parse (cfg,
-                                               "test_arm_api_data.conf"))
-    return GNUNET_SYSERR;
-  if (NULL == getcwd (pwd, PATH_MAX))
-    return GNUNET_SYSERR;
-  GNUNET_assert (0 < GNUNET_asprintf (&binary, "%s/%s", pwd, BINARY));
-  GNUNET_CONFIGURATION_set_value_string (cfg, SERVICE, "BINARY", binary);
-  GNUNET_free (binary);
-  if (GNUNET_OK != GNUNET_CONFIGURATION_write (cfg, CFGFILENAME))
-  {
-    GNUNET_CONFIGURATION_destroy (cfg);
+  cfg = GNUNET_CONFIGURATION_create();
+  if (GNUNET_OK != GNUNET_CONFIGURATION_parse(cfg,
+                                              "test_arm_api_data.conf"))
+    {
+      GNUNET_CONFIGURATION_destroy(cfg);
+      return GNUNET_SYSERR;
+    }
+  if (NULL == getcwd(pwd, PATH_MAX))
     return GNUNET_SYSERR;
-  }
-  GNUNET_CONFIGURATION_destroy (cfg);
+  GNUNET_assert(0 < GNUNET_asprintf(&binary,
+                                    "%s/%s",
+                                    pwd,
+                                    BINARY));
+  GNUNET_CONFIGURATION_set_value_string(cfg,
+                                        SERVICE,
+                                        "BINARY",
+                                        binary);
+  GNUNET_free(binary);
+  if (GNUNET_OK != GNUNET_CONFIGURATION_write(cfg,
+                                              CFGFILENAME))
+    {
+      GNUNET_CONFIGURATION_destroy(cfg);
+      return GNUNET_SYSERR;
+    }
+  GNUNET_CONFIGURATION_destroy(cfg);
 
 #if LOG_BACKOFF
-  killLogFileName = GNUNET_DISK_mktemp ("exponential-backoff-waiting.log");
-  if (NULL == (killLogFilePtr = FOPEN (killLogFileName, "w")))
+  killLogFileName = GNUNET_DISK_mktemp("exponential-backoff-waiting.log");
+  if (NULL == (killLogFilePtr = fopen(killLogFileName,
+                                      "w")))
     {
-      GNUNET_log_strerror_file (GNUNET_ERROR_TYPE_WARNING, "fopen",
-                               killLogFileName);
-      GNUNET_free (killLogFileName);
+      GNUNET_log_strerror_file(GNUNET_ERROR_TYPE_WARNING,
+                               "fopen",
+                               killLogFileName);
+      GNUNET_free(killLogFileName);
       return GNUNET_SYSERR;
     }
 #endif
@@ -454,29 +383,29 @@ init ()
 
 
 static void
-houseKeep ()
+houseKeep()
 {
 #if LOG_BACKOFF
-  GNUNET_assert (0 == fclose (killLogFilePtr));
-  GNUNET_free (killLogFileName);
+  GNUNET_assert(0 == fclose(killLogFilePtr));
+  GNUNET_free(killLogFileName);
 #endif
-  (void) unlink (CFGFILENAME);
+  (void)unlink(CFGFILENAME);
 }
 
 
 int
-main (int argc, char *argv[])
+main(int argc, char *argv[])
 {
   int ret;
 
-  GNUNET_log_setup ("test-exponential-backoff",
-                   "WARNING",
-                   NULL);
+  GNUNET_log_setup("test-exponential-backoff",
+                   "WARNING",
+                   NULL);
 
-  if (GNUNET_OK != init ())
+  if (GNUNET_OK != init())
     return 1;
-  ret = check ();
-  houseKeep ();
+  ret = check();
+  houseKeep();
   return ret;
 }