Changeset 3552

Timestamp:

08/18/08 09:40:26 (4 months ago)

Author:

bricks

Message:

added some sanity checks

Location:

trunk/opensync

Files:

• module/opensync_module.c (modified) (5 diffs)
• plugin/opensync_plugin_env.c (modified) (1 diff)

trunk/opensync/module/opensync_module.c

@@ -76 +76 @@
 void *osync_module_get_function(OSyncModule *module, const char *name, OSyncError **error)
 {
+        osync_assert(module);
+        osync_assert(name);
+        
         void *function = NULL;
         if (!module->module) {
@@ -83 +86 @@
         
         if (!g_module_symbol(module->module, name, &function)) {
-                osync_error_set(error, OSYNC_ERROR_PARAMETER, "Unable to locate symbol %s: %s", name, g_module_error());
+                osync_error_set(error, OSYNC_ERROR_PARAMETER, "Unable to locate symbol %s: %s", __NULLSTR(name), g_module_error());
                 return NULL;
         }
@@ -164 +167 @@
 int osync_module_get_version(OSyncModule *module)
 {
+        osync_assert(module);
+        
         void *function = NULL;
         int (* fct_version)(void) = NULL;
@@ -217 +222 @@
 {
         osync_trace(TRACE_ENTRY, "%s(%p, %s, %p)", __func__, module, path, error);
+        osync_assert(module);
         osync_assert(!module->module);
         
@@ -250 +256 @@
 {
         osync_trace(TRACE_ENTRY, "%s(%p)", __func__, module);
+        osync_assert(module);
         osync_assert(module->module);
         

trunk/opensync/plugin/opensync_plugin_env.c

@@ -82 +82 @@
 {
         osync_trace(TRACE_ENTRY, "%s(%p)", __func__, env);
-        g_assert(env);
+        osync_assert(env);
         
         /* Free the plugins */