add ENABLE_SANITY_CHECKS #define instead of using '1'

src/include/taler_util.h

@@ -171,6 +171,10 @@ TALER_mhd_is_https (struct MHD_Connection *connection);
 /**
  * Make an absolute URL with query parameters.
  *
+ * If a 'value' is given as NULL, both the key and the value are skipped. Note
+ * that a NULL value does not terminate the list, only a NULL key signals the
+ * end of the list of arguments.
+ *
  * @param base_url absolute base URL to use
  * @param path path of the url
  * @param ... NULL-terminated key-value pairs (char *) for query parameters,
@@ -186,6 +190,10 @@ TALER_url_join (const char *base_url,
 /**
  * Make an absolute URL for the given parameters.
  *
+ * If a 'value' is given as NULL, both the key and the value are skipped. Note
+ * that a NULL value does not terminate the list, only a NULL key signals the
+ * end of the list of arguments.
+ *
  * @param proto protocol for the URL (typically https)
  * @param host hostname for the URL
  * @param prefix prefix for the URL
@@ -205,6 +213,10 @@ TALER_url_absolute_raw (const char *proto,
 /**
  * Make an absolute URL for the given parameters.
  *
+ * If a 'value' is given as NULL, both the key and the value are skipped. Note
+ * that a NULL value does not terminate the list, only a NULL key signals the
+ * end of the list of arguments.
+ *
  * @param proto protocol for the URL (typically https)
  * @param host hostname for the URL
  * @param prefix prefix for the URL

src/util/crypto.c

@@ -62,7 +62,8 @@ TALER_gcrypt_init ()
              "libgcrypt version mismatch\n");
     abort ();
   }
-  /* Disable secure memory.  */
+  /* Disable secure memory (we should never run on a system that
+     even uses swap space for memory). */
   gcry_control (GCRYCTL_DISABLE_SECMEM, 0);
   gcry_control (GCRYCTL_INITIALIZATION_FINISHED, 0);
 }
@@ -83,8 +84,7 @@ TALER_test_coin_valid (const struct TALER_CoinPublicInfo *coin_public_info,
                        const struct TALER_DenominationPublicKey *denom_pub)
 {
   struct GNUNET_HashCode c_hash;
-#if 1 /* sanity check of invariant, could probably be disabled in production
+#if ENABLE_SANITY_CHECKS
-         for slightly more performance */
   struct GNUNET_HashCode d_hash;
 
   GNUNET_CRYPTO_rsa_public_key_hash (denom_pub->rsa_public_key,

src/util/url.c

@@ -157,7 +157,10 @@ calculate_argument_length (va_list args)
 
 /**
  * Take the key-value pairs in @a args and serialize them into
- * @a buf, using URL encoding for the values.
+ * @a buf, using URL encoding for the values.  If a 'value' is
+ * given as NULL, both the key and the value are skipped. Note
+ * that a NULL value does not terminate the list, only a NULL
+ * key signals the end of the list of arguments.
  *
  * @param buf where to write the values
  * @param args NULL-terminated key-value pairs (char *) for query parameters,
@@ -201,6 +204,10 @@ serialize_arguments (struct GNUNET_Buffer *buf,
 /**
  * Make an absolute URL with query parameters.
  *
+ * If a 'value' is given as NULL, both the key and the value are skipped. Note
+ * that a NULL value does not terminate the list, only a NULL key signals the
+ * end of the list of arguments.
+ *
  * @param base_url absolute base URL to use
  * @param path path of the url
  * @param ... NULL-terminated key-value pairs (char *) for query parameters,
@@ -265,6 +272,10 @@ TALER_url_join (const char *base_url,
 /**
  * Make an absolute URL for the given parameters.
  *
+ * If a 'value' is given as NULL, both the key and the value are skipped. Note
+ * that a NULL value does not terminate the list, only a NULL key signals the
+ * end of the list of arguments.
+ *
  * @param proto protocol for the URL (typically https)
  * @param host hostname for the URL
  * @param prefix prefix for the URL
@@ -308,6 +319,10 @@ TALER_url_absolute_raw_va (const char *proto,
 /**
  * Make an absolute URL for the given parameters.
  *
+ * If a 'value' is given as NULL, both the key and the value are skipped. Note
+ * that a NULL value does not terminate the list, only a NULL key signals the
+ * end of the list of arguments.
+ *
  * @param proto protocol for the URL (typically https)
  * @param host hostname for the URL
  * @param prefix prefix for the URL