X-Git-Url: https://git.octo.it/?p=collectd.git;a=blobdiff_plain;f=src%2Fdaemon%2Fcommon.c;h=c80abcbee65cb1e95a8ef14dc065d7082f83c4c2;hp=e4894493222e91455fedc43d7b571aa7b6887938;hb=8c5927c52f4eefebaad3a6ecadc253ee9007ebb5;hpb=9c962b99a3acd77f1d6e2499052b47356819511a

diff --git a/src/daemon/common.c b/src/daemon/common.c
index e4894493..c80abcbe 100644
--- a/src/daemon/common.c
+++ b/src/daemon/common.c
@@ -339,50 +339,60 @@ int strsplit (char *string, char **fields, size_t size)
 	return ((int) i);
 }
 
-int strjoin (char *buffer, size_t buffer_size,
-		char **fields, size_t fields_num,
-		const char *sep)
-{
-	size_t avail;
-	char *ptr;
-	size_t sep_len;
+int strjoin(char *buffer, size_t buffer_size, char **fields, size_t fields_num,
+            const char *sep) {
+  size_t avail = 0;
+  char *ptr = buffer;
+  size_t sep_len = 0;
 
-	if ((buffer_size < 1) || (fields_num == 0))
-		return (-1);
+  size_t buffer_req = 0;
 
-	memset (buffer, 0, buffer_size);
-	ptr = buffer;
-	avail = buffer_size - 1;
+  if (((fields_num != 0) && (fields == NULL)) ||
+      ((buffer_size != 0) && (buffer == NULL)))
+    return (-EINVAL);
 
-	sep_len = 0;
-	if (sep != NULL)
-		sep_len = strlen (sep);
+  if (buffer != NULL)
+    buffer[0] = 0;
 
-	for (size_t i = 0; i < fields_num; i++)
-	{
-		size_t field_len;
+  if (buffer_size != 0)
+    avail = buffer_size - 1;
 
-		if ((i > 0) && (sep_len > 0))
-		{
-			if (avail < sep_len)
-				return (-1);
+  if (sep != NULL)
+    sep_len = strlen(sep);
 
-			memcpy (ptr, sep, sep_len);
-			ptr += sep_len;
-			avail -= sep_len;
-		}
+  for (size_t i = 0; i < fields_num; i++) {
+    size_t field_len = strlen(fields[i]);
 
-		field_len = strlen (fields[i]);
-		if (avail < field_len)
-			return (-1);
+    if (i != 0)
+      buffer_req += sep_len;
+    buffer_req += field_len;
 
-		memcpy (ptr, fields[i], field_len);
-		ptr += field_len;
-		avail -= field_len;
-	}
+    if ((i != 0) && (sep_len > 0)) {
+      if (sep_len >= avail) {
+        /* prevent subsequent iterations from writing to the
+         * buffer. */
+        avail = 0;
+        continue;
+      }
+
+      memcpy(ptr, sep, sep_len);
+
+      ptr += sep_len;
+      avail -= sep_len;
+    }
 
-	assert (buffer[buffer_size - 1] == 0);
-	return ((int) strlen (buffer));
+    if (field_len > avail)
+      field_len = avail;
+
+    memcpy(ptr, fields[i], field_len);
+    ptr += field_len;
+
+    avail -= field_len;
+    if (ptr != NULL)
+      *ptr = 0;
+  }
+
+  return (int)buffer_req;
 }
 
 int escape_string (char *buffer, size_t buffer_size)
@@ -1204,6 +1214,28 @@ int parse_values (char *buffer, value_list_t *vl, const data_set_t *ds)
 	return (0);
 } /* int parse_values */
 
+int parse_value_file (char const *path, value_t *ret_value, int ds_type)
+{
+	FILE *fh;
+	char buffer[256];
+
+	fh = fopen (path, "r");
+	if (fh == NULL)
+		return (-1);
+
+	if (fgets (buffer, sizeof (buffer), fh) == NULL)
+	{
+		fclose (fh);
+		return (-1);
+	}
+
+	fclose (fh);
+
+	strstripnewline (buffer);
+
+	return parse_value (buffer, ret_value, ds_type);
+} /* int parse_value_file */
+
 #if !HAVE_GETPWNAM_R
 int getpwnam_r (const char *name, struct passwd *pwbuf, char *buf,
 		size_t buflen, struct passwd **pwbufp)