X-Git-Url: http://git.tuebingen.mpg.de/?p=paraslash.git;a=blobdiff_plain;f=mysql_selector.c;h=bbc05bdf39ddcf9a4f8e9c503d3d21efff72ae67;hp=dedc96cc8ec6961bf118c78586bb3c5be037fb04;hb=de25f9d0d999b2a911ecc93d19511ff437211d18;hpb=843950e7a85730b796bb6238b6e91111f7209c25

diff --git a/mysql_selector.c b/mysql_selector.c
index dedc96cc..bbc05bdf 100644
--- a/mysql_selector.c
+++ b/mysql_selector.c
@@ -67,11 +67,11 @@ static const struct para_macro mysql_macro_list[] = {
 };
 
 /**
- * simple search and replace routine
+ * Simple search and replace routine.
  *
- * \param src source string
- * \param macro_name the name of the macro
- * \param replacement the replacement format string
+ * \param src Source String.
+ * \param macro_name The name of the macro.
+ * \param replacement The replacement format string.
  *
  * In \p src, replace each occurence of \p macro_name(arg) by the string
  * determined by the \p replacement format string. \p replacement may (but
@@ -79,7 +79,7 @@ static const struct para_macro mysql_macro_list[] = {
  * replaced by \p arg.
  *
  * \return A string in which all matches in \p src are replaced, or \p NULL if
- * an syntax error was encountered. Caller must free the result.
+ * an error was encountered. Caller must free the result.
  *
  * \sa regcomp(3)
  */
@@ -95,7 +95,8 @@ __must_check __malloc static char *s_a_r(const char *src, const char* macro_name
 
 	if (!macro_name || !replacement || !src)
 		return para_strdup(src);
-	regcomp(&preg, macro_name, 0);
+	if (regcomp(&preg, macro_name, 0) != 0)
+		return NULL;
 	while (regexec(&preg,  bufptr, nmatch, pmatch, eflags)
 			!= REG_NOMATCH) {
 		char *tmp, *arg, *o_bracket, *c_bracket;
@@ -2270,6 +2271,7 @@ static void shutdown_connection(void)
 		ret = mutex_destroy(mysql_lock);
 		if (ret < 0)
 			PARA_ERROR_LOG("%s\n", PARA_STRERROR(-ret));
+		mysql_lock = 0;
 	}
 }