[Opensync-commits] r5564 - plugins/ldap-sync/src

[<svn...@op...>]

Author: scriptor
Date: Wed Apr  8 21:47:45 2009
New Revision: 5564
URL: http://www.opensync.org/changeset/5564

Log:
Removed several osync_error_unref() calls, so that the error
keeps being set for the calling functions.

Modified:
   plugins/ldap-sync/src/ldap_connect.c
   plugins/ldap-sync/src/ldap_plugin.c

Modified: plugins/ldap-sync/src/ldap_connect.c
==============================================================================
--- plugins/ldap-sync/src/ldap_connect.c	Wed Apr  8 20:08:33 2009	(r5563)
+++ plugins/ldap-sync/src/ldap_connect.c	Wed Apr  8 21:47:45 2009	(r5564)
@@ -571,8 +571,6 @@
 
 	osync_context_report_osyncwarning(ctx, *error);
 	osync_trace(TRACE_EXIT_ERROR, "%s: %s", __func__, osync_error_print(error));
-	osync_error_unref(error);
-
 	return FALSE;
 }
 
@@ -724,8 +722,6 @@
 
 	osync_context_report_osyncwarning(ctx, *error);
 	osync_trace(TRACE_EXIT_ERROR, "%s: %s", __func__, osync_error_print(error));
-	osync_error_unref(error);
-
 	return FALSE;
 }
 
@@ -1195,8 +1191,6 @@
 
 	osync_context_report_osyncwarning(ctx, *error);
 	osync_trace(TRACE_EXIT_ERROR, "%s: %s", __func__, osync_error_print(error));
-	osync_error_unref(error);
-
 	return FALSE;
 }
 
@@ -1568,7 +1562,6 @@
 		osync_error_set(error, OSYNC_ERROR_GENERIC, "Unknown reason.\n");
 
 	osync_trace(TRACE_EXIT_ERROR, "%s: %s", __func__, osync_error_print(error));
-	osync_error_unref(error);
 	return FALSE;
 }
 
@@ -1750,8 +1743,6 @@
 
 	osync_context_report_osyncwarning(ctx, *error);
 	osync_trace(TRACE_EXIT_ERROR, "%s: %s", __func__, osync_error_print(error));
-	osync_error_unref(error);
-
 	return NULL;
 }
 
@@ -2154,8 +2145,6 @@
 
 	osync_context_report_osyncwarning(ctx, *error);
 	osync_trace(TRACE_EXIT_ERROR, "%s: %s", __func__, osync_error_print(error));
-	osync_error_unref(error);
-
 	return FALSE;
 }
 
@@ -2456,8 +2445,6 @@
 
 	osync_context_report_osyncwarning(ctx, *error);
 	osync_trace(TRACE_EXIT_ERROR, "%s: %s", __func__, osync_error_print(error));
-	osync_error_unref(error);
-
 	return NULL;
 }
 
@@ -2593,8 +2580,6 @@
 
 	osync_context_report_osyncwarning(ctx, *error);
 	osync_trace(TRACE_EXIT_ERROR, "%s: %s", __func__, osync_error_print(error));
-	osync_error_unref(error);
-
 	return FALSE;
 }
 
@@ -2735,8 +2720,6 @@
 
 	osync_context_report_osyncwarning(ctx, *error);
 	osync_trace(TRACE_EXIT_ERROR, "%s: %s", __func__, osync_error_print(error));
-	osync_error_unref(error);
-	
 	return FALSE;
 }
 
@@ -2826,8 +2809,6 @@
 
 	osync_context_report_osyncwarning(ctx, *error);
 	osync_trace(TRACE_EXIT_ERROR, "%s: %s", __func__, osync_error_print(error));
-	osync_error_unref(error);
-	
 	return FALSE;
 }
 
@@ -3246,7 +3227,6 @@
 
 	osync_context_report_osyncwarning(ctx, *error);
 	osync_trace(TRACE_EXIT_ERROR, "%s: %s", __func__, osync_error_print(error));
-	osync_error_unref(error);
 
 	/* String array contains the same values - no modify */
 	return FALSE;
@@ -3340,8 +3320,6 @@
 
 	osync_context_report_osyncwarning(ctx, *error);
 	osync_trace(TRACE_EXIT_ERROR, "%s: %s", __func__, osync_error_print(error));
-	osync_error_unref(error);
-
 	return NULL;
 }
 
@@ -3807,8 +3785,6 @@
 
 	osync_context_report_osyncwarning(ctx, *error);
 	osync_trace(TRACE_EXIT_ERROR, "%s: %s", __func__, osync_error_print(error));
-	osync_error_unref(error);
-
 	return;
 }
 
@@ -3949,8 +3925,6 @@
 		osync_error_set(error, OSYNC_ERROR_GENERIC, "Unknown reason.\n");
 
 	osync_trace(TRACE_EXIT_ERROR, "%s: %s", __func__, osync_error_print(error));
-	osync_error_unref(error);
-
 	return FALSE;
 }
 
@@ -4152,8 +4126,6 @@
 		osync_error_set(error, OSYNC_ERROR_GENERIC, "Unknown reason.\n");
 
 	osync_trace(TRACE_EXIT_ERROR, "%s: %s", __func__, osync_error_print(error));
-	osync_error_unref(error);
-
 	return FALSE;
 }
 
@@ -4381,8 +4353,6 @@
 
 	osync_context_report_osyncwarning(ctx, *error);
 	osync_trace(TRACE_EXIT_ERROR, "%s: %s", __func__, osync_error_print(error));
-	osync_error_unref(error);
-
 	return NULL;
 }
 

Modified: plugins/ldap-sync/src/ldap_plugin.c
==============================================================================
--- plugins/ldap-sync/src/ldap_plugin.c	Wed Apr  8 20:08:33 2009	(r5563)
+++ plugins/ldap-sync/src/ldap_plugin.c	Wed Apr  8 21:47:45 2009	(r5564)
@@ -508,8 +508,6 @@
 
 	osync_context_report_osyncwarning(ctx, *error);
 	osync_trace(TRACE_EXIT_ERROR, "%s: %s", __func__, osync_error_print(error));
-	osync_error_unref(error);
-
 	return FALSE;
 }