[Opensync-commits] r5259 - in plugins/gpe: . src

[<dg...@su...>]

Author: Graham Cobb
Date: Wed Feb  4 01:57:21 2009
New Revision: 5259
URL: http://www.opensync.org/changeset/5259

Log:
gpe-sync: fix valgrind problems

Modified:
   plugins/gpe/ChangeLog
   plugins/gpe/src/utils.c

Modified: plugins/gpe/ChangeLog
==============================================================================
--- plugins/gpe/ChangeLog	Tue Feb  3 20:54:04 2009	(r5258)
+++ plugins/gpe/ChangeLog	Wed Feb  4 01:57:21 2009	(r5259)
@@ -1,3 +1,8 @@
+2009-02-04  Graham Cobb  <g+...@co...>
+
+	* src/utils.c (report_change): Include NUL terminator in change data
+	(report_change): Fix osync_trace format
+
 2009-01-17  Graham Cobb  <g+...@co...>
 
 	* src/utils.c (report_deleted): Fix end condition for uids loop

Modified: plugins/gpe/src/utils.c
==============================================================================
--- plugins/gpe/src/utils.c	Tue Feb  3 20:54:04 2009	(r5258)
+++ plugins/gpe/src/utils.c	Wed Feb  4 01:57:21 2009	(r5259)
@@ -78,12 +78,12 @@
  */
 osync_bool report_change (sink_environment *sinkenv, OSyncContext *ctx, gchar *type, gchar *uid, gchar *hash, char *string)
 {
-	osync_trace(TRACE_ENTRY, "GPE-SYNC %s(%p, %p, %p, %p, %p, %p, %p)", __func__, sinkenv, ctx, type, uid, hash, string);
+	osync_trace(TRACE_ENTRY, "GPE-SYNC %s(%p, %p, %p, %p, %p, %p)", __func__, sinkenv, ctx, type, uid, hash, string);
 	osync_trace(TRACE_SENSITIVE, "GPE-SYNC %s: reporting item type = %s, uid = %s, hash = %s, string = %s", __func__, type, uid, hash, string);
 
 	OSyncError *error = NULL;
 
-	OSyncData *data = osync_data_new(string, strlen(string), sinkenv->objformat, &error);
+	OSyncData *data = osync_data_new(string, strlen(string)+1, sinkenv->objformat, &error);
 	if (!data) {
 	  osync_context_report_osyncwarning(ctx, error);
 	  osync_error_unref(&error);