allow empty/NULL context message
[oweals/gnunet.git] / src / my / my_query_helper.c
index c094221602276e5adc7bbaa30579ad602d4ad212..17ade5a9274c2949401bc7456d538d7fd1d7e5d1 100644 (file)
@@ -21,6 +21,7 @@
  * @file my/my_query_helper.c
  * @brief library to help with access to a MySQL database
  * @author Christian Grothoff
+ * @author Christophe Genevey
  */
 #include "platform.h"
 #include <mysql/mysql.h>
  * by a #GNUNET_MY_QueryConverter.
  *
  * @param cls closure
- * @param rd result data to clean up
+ * @param qbind array of parameter to clean up
  */
 static void
 my_clean_query (void *cls,
-          MYSQL_BIND *qbind)
+                MYSQL_BIND *qbind)
 {
   GNUNET_free (qbind[0].buffer);
 }
@@ -56,7 +57,6 @@ my_conv_fixed_size (void *cls,
                     MYSQL_BIND *qbind)
 {
   GNUNET_assert (1 == qp->num_params);
-
   qbind->buffer = (void *) qp->data;
   qbind->buffer_length = qp->data_len;
   qbind->buffer_type = MYSQL_TYPE_BLOB;
@@ -89,95 +89,105 @@ GNUNET_MY_query_param_fixed_size (const void *ptr,
 
 
 /**
-  * Generate query parameter for a string
-  *
-  *@param ptr pointer to the string query parameter to pass
-  */
+ * Function called to convert input argument into SQL parameters.
+ *
+ * @param cls closure
+ * @param pq data about the query
+ * @param qbind array of parameters to initialize
+ * @return -1 on error
+ */
+static int
+my_conv_string (void *cls,
+                const struct GNUNET_MY_QueryParam *qp,
+                MYSQL_BIND *qbind)
+{
+  GNUNET_assert (1 == qp->num_params);
+
+  qbind->buffer = (void *) qp->data;
+  qbind->buffer_length = qp->data_len;
+  qbind->buffer_type = MYSQL_TYPE_STRING;
+
+  return 1;
+}
+
+
+/**
+ * Generate query parameter for a string
+ *
+ * @param ptr pointer to the string query parameter to pass
+ */
 struct GNUNET_MY_QueryParam
 GNUNET_MY_query_param_string (const char *ptr)
 {
-  return GNUNET_MY_query_param_fixed_size(ptr,
-                                         strlen(ptr));
+  struct GNUNET_MY_QueryParam qp = {
+    .conv = &my_conv_string,
+    .cleaner = NULL,
+    .conv_cls = NULL,
+    .num_params = 1,
+    .data = ptr,
+    .data_len = strlen (ptr)
+  };
+  return qp;
 }
 
 
 /**
 * Function called to convert input argument into SQL parameters
 *
 *@param cls closure
 *@param pq data about the query
+ * Function called to convert input argument into SQL parameters
+ *
@param cls closure
@param pq data about the query
  * @param qbind array of parameters to initialize
 *@return -1 on error
 */
@return -1 on error
+ */
 static int
 my_conv_uint16 (void *cls,
-                const struct GNUNET_MY_QueryParam * qp,
+                const struct GNUNET_MY_QueryParam *qp,
                 MYSQL_BIND *qbind)
 {
-  const uint16_t *u_hbo = qp->data;
-  uint16_t *u_nbo;
-
   GNUNET_assert (1 == qp->num_params);
-
-
-  u_nbo = GNUNET_new (uint16_t);
-  if (NULL == u_nbo)
-    return -1;
-
-  *u_nbo = *u_hbo;
-
-  qbind->buffer = (void *) u_nbo;
-  qbind->buffer_length = sizeof(uint16_t);
+  qbind->buffer = (void *) qp->data;
+  qbind->buffer_length = sizeof (uint16_t);
   qbind->buffer_type = MYSQL_TYPE_SHORT;
-
   return 1;
 }
 
 
 /**
 * Generate query parameter for an uint16_t in host byte order.
 *
 * @param x pointer to the query parameter to pass
 */
+ * Generate query parameter for an uint16_t in host byte order.
+ *
+ * @param x pointer to the query parameter to pass
+ */
 struct GNUNET_MY_QueryParam
 GNUNET_MY_query_param_uint16 (const uint16_t *x)
 {
   struct GNUNET_MY_QueryParam res = {
-      .conv = &my_conv_uint16,
-      .cleaner = &my_clean_query,
-      .conv_cls = NULL,
-      .num_params = 1,
-      .data = x,
-      .data_len = sizeof (*x)
-    };
+    .conv = &my_conv_uint16,
+    .cleaner = NULL,
+    .conv_cls = NULL,
+    .num_params = 1,
+    .data = x,
+    .data_len = sizeof (*x)
+  };
 
   return res;
 }
 
 
 /**
 * Function called to convert input argument into SQL parameters
 *
 *@param cls closure
 *@param pq data about the query
+ * Function called to convert input argument into SQL parameters
+ *
@param cls closure
@param pq data about the query
  * @param qbind array of parameters to initialize
 *@return -1 on error
 */
@return -1 on error
+ */
 static int
 my_conv_uint32 (void *cls,
                 const struct GNUNET_MY_QueryParam *qp,
                 MYSQL_BIND *qbind)
 {
-  const uint32_t *u_hbo = qp->data;
-  uint32_t * u_nbo;
-
   GNUNET_assert (1 == qp->num_params);
-
-  u_nbo = GNUNET_new (uint32_t);
-
-  *u_nbo = *u_hbo;
-
-  qbind->buffer = (void *) u_nbo;
+  qbind->buffer = (void *) qp->data;
   qbind->buffer_length = sizeof(uint32_t);
   qbind->buffer_type = MYSQL_TYPE_LONG;
 
@@ -186,16 +196,16 @@ my_conv_uint32 (void *cls,
 
 
 /**
 * Generate query parameter for an uint32_t in host byte order
 *
 *@param x pointer to the query parameter to pass
 */
+ * Generate query parameter for an uint32_t in host byte order
+ *
@param x pointer to the query parameter to pass
+ */
 struct GNUNET_MY_QueryParam
 GNUNET_MY_query_param_uint32 (const uint32_t *x)
 {
   struct GNUNET_MY_QueryParam res = {
     .conv = &my_conv_uint32,
-    .cleaner = &my_clean_query,
+    .cleaner = NULL,
     .conv_cls = NULL,
     .num_params = 1,
     .data = x,
@@ -207,46 +217,37 @@ GNUNET_MY_query_param_uint32 (const uint32_t *x)
 
 
 /**
 * Function called to convert input argument into SQL parameters
 *
 *@param cls closure
 *@param pq data about the query
+ * Function called to convert input argument into SQL parameters
+ *
@param cls closure
@param pq data about the query
  * @param qbind array of parameters to initialize
 *@return -1 on error
 */
@return -1 on error
+ */
 static int
 my_conv_uint64 (void *cls,
-              const struct GNUNET_MY_QueryParam *qp,
-              MYSQL_BIND * qbind)
+                const struct GNUNET_MY_QueryParam *qp,
+                MYSQL_BIND * qbind)
 {
-  const uint64_t * u_hbo = qp->data;
-  uint64_t *u_nbo;
-
   GNUNET_assert (1 == qp->num_params);
-
-  u_nbo = GNUNET_new(uint64_t);
-
-  *u_nbo = *u_hbo;
-
-  qbind->buffer = (void *) u_nbo;
+  qbind->buffer = (void *) qp->data;
   qbind->buffer_length = sizeof (uint64_t);
   qbind->buffer_type = MYSQL_TYPE_LONGLONG;
-
   return 1;
 }
 
 
 /**
 * Generate query parameter for an uint64_t in host byte order
 *
 *@param x pointer to the query parameter to pass
 */
+ * Generate query parameter for an uint64_t in host byte order
+ *
@param x pointer to the query parameter to pass
+ */
 struct GNUNET_MY_QueryParam
 GNUNET_MY_query_param_uint64 (const uint64_t *x)
 {
   struct GNUNET_MY_QueryParam res = {
     .conv = &my_conv_uint64,
-    .cleaner = &my_clean_query,
+    .cleaner = NULL,
     .conv_cls = NULL,
     .num_params = 1,
     .data = x,
@@ -258,13 +259,13 @@ GNUNET_MY_query_param_uint64 (const uint64_t *x)
 
 
 /**
 * Function called to convert input argument into SQL parameters
 *
 *@param cls closure
 *@param pq data about the query
+ * Function called to convert input argument into SQL parameters
+ *
@param cls closure
@param pq data about the query
  * @param qbind array of parameters to initialize
 *@return -1 on error
 */
@return -1 on error
+ */
 static int
 my_conv_rsa_public_key (void *cls,
                         const struct GNUNET_MY_QueryParam *qp,
@@ -275,7 +276,7 @@ my_conv_rsa_public_key (void *cls,
   size_t buf_size;
 
   GNUNET_assert(1 == qp->num_params);
-  // FIXME: this leaks memory right now...
+
   buf_size = GNUNET_CRYPTO_rsa_public_key_encode (rsa, &buf);
 
   qbind->buffer = (void *) buf;
@@ -287,12 +288,12 @@ my_conv_rsa_public_key (void *cls,
 
 
 /**
 * Generate query parameter for an RSA public key. The
 * database must contain a BLOB type in the respective position.
 *
 * @param x the query parameter to pass
 * @return array entry for the query parameters to use
 */
+ * Generate query parameter for an RSA public key. The
+ * database must contain a BLOB type in the respective position.
+ *
+ * @param x the query parameter to pass
+ * @return array entry for the query parameters to use
+ */
 struct GNUNET_MY_QueryParam
 GNUNET_MY_query_param_rsa_public_key (const struct GNUNET_CRYPTO_RsaPublicKey *x)
 {
@@ -339,12 +340,12 @@ my_conv_rsa_signature (void *cls,
 
 
 /**
 * Generate query parameter for an RSA signature. The
 * database must contain a BLOB type in the respective position
 *
 *@param x the query parameter to pass
 *@return array entry for the query parameters to use
 */
+ * Generate query parameter for an RSA signature. The
+ * database must contain a BLOB type in the respective position
+ *
@param x the query parameter to pass
@return array entry for the query parameters to use
+ */
 struct GNUNET_MY_QueryParam
 GNUNET_MY_query_param_rsa_signature (const struct GNUNET_CRYPTO_RsaSignature *x)
 {
@@ -361,12 +362,12 @@ GNUNET_MY_query_param_rsa_signature (const struct GNUNET_CRYPTO_RsaSignature *x)
 
 
 /**
 * Generate query parameter for an absolute time value.
 * The database must store a 64-bit integer.
 *
 *@param x pointer to the query parameter to pass
 *@return array entry for the query parameters to use
 */
+ * Generate query parameter for an absolute time value.
+ * The database must store a 64-bit integer.
+ *
@param x pointer to the query parameter to pass
@return array entry for the query parameters to use
+ */
 struct GNUNET_MY_QueryParam
 GNUNET_MY_query_param_absolute_time (const struct GNUNET_TIME_Absolute *x)
 {
@@ -375,11 +376,11 @@ GNUNET_MY_query_param_absolute_time (const struct GNUNET_TIME_Absolute *x)
 
 
 /**
 * Generate query parameter for an absolute time value.
 * The database must store a 64-bit integer.
 *
 *@param x pointer to the query parameter to pass
 */
+ * Generate query parameter for an absolute time value.
+ * The database must store a 64-bit integer.
+ *
@param x pointer to the query parameter to pass
+ */
 struct GNUNET_MY_QueryParam
 GNUNET_MY_query_param_absolute_time_nbo (const struct GNUNET_TIME_AbsoluteNBO *x)
 {
@@ -387,4 +388,4 @@ GNUNET_MY_query_param_absolute_time_nbo (const struct GNUNET_TIME_AbsoluteNBO *x
 }
 
 
-/* end of my_query_helper.c */
\ No newline at end of file
+/* end of my_query_helper.c */