Blame SOURCES/openchange-2.3-fix-connection-args.patch

afa884
diff -up openchange-openchange-2.3-VULCAN/libmapi/IMSProvider.c.fix-connection-args openchange-openchange-2.3-VULCAN/libmapi/IMSProvider.c
afa884
--- openchange-openchange-2.3-VULCAN/libmapi/IMSProvider.c.fix-connection-args	2015-05-16 17:22:04.000000000 +0200
afa884
+++ openchange-openchange-2.3-VULCAN/libmapi/IMSProvider.c	2016-04-07 19:10:30.340959912 +0200
afa884
@@ -91,6 +91,7 @@ static char *build_binding_string(struct
afa884
 				  struct mapi_profile *profile)
afa884
 {
afa884
 	char	*binding;
afa884
+	bool	any_string_added = false;
afa884
 
afa884
 	/* Sanity Checks */
afa884
 	if (!profile) return NULL;
afa884
@@ -101,15 +102,23 @@ static char *build_binding_string(struct
afa884
 
afa884
 	/* If dump-data option is enabled */
afa884
 	if (mapi_ctx->dumpdata == true) {
afa884
-		binding = talloc_strdup_append(binding, "print,");
afa884
+		binding = talloc_strdup_append(binding, "print");
afa884
+		any_string_added = true;
afa884
 	}
afa884
 	/* If seal option is enabled in the profile */
afa884
 	if (profile->seal == true) {
afa884
-		binding = talloc_strdup_append(binding, "seal,");
afa884
+		if (any_string_added) {
afa884
+			binding = talloc_strdup_append(binding, ",");
afa884
+		}
afa884
+		binding = talloc_strdup_append(binding, "seal");
afa884
+		any_string_added = true;
afa884
 	}
afa884
 	/* If localaddress parameter is available in the profile */
afa884
 	if (profile->localaddr) {
afa884
-		binding = talloc_asprintf_append(binding, "localaddress=%s,", profile->localaddr);
afa884
+		if (any_string_added) {
afa884
+			binding = talloc_strdup_append(binding, ",");
afa884
+		}
afa884
+		binding = talloc_asprintf_append(binding, "localaddress=%s", profile->localaddr);
afa884
 	}
afa884
 	
afa884
 	binding = talloc_strdup_append(binding, "]");