[FIXME: On NetBSD, dlsym() doesn't set errno != 0 when the system cannot be

        found; this leads to an incorrect return value of g_module_symbol.]

2000-01-13  Martin Baulig  <martin@home-of-linux.org>

	* gmodule.c (g_module_open): Check whether `check_init' is not NULL
	before we attempt to call it.

diff --git a/gmodule/ChangeLog b/gmodule/ChangeLog
index 0039146..040f881 100644 (file)
--- a/gmodule/ChangeLog
+++ b/gmodule/ChangeLog
@@ -1,3 +1,8 @@
+2000-01-13  Martin Baulig  <martin@home-of-linux.org>
+
+       * gmodule.c (g_module_open): Check whether `check_init' is not NULL
+       before we attempt to call it.
+
 Sun Oct  3 19:30:52 PDT 1999 Manish Singh <yosh@gimp.org>
 
        * gmodule.h

diff --git a/gmodule/gmodule.c b/gmodule/gmodule.c
index d2a6f99..0a4b211 100644 (file)
--- a/gmodule/gmodule.c
+++ b/gmodule/gmodule.c
@@ -256,7 +256,8 @@ g_module_open (const gchar    *file_name,
       
       /* check initialization */
       if (g_module_symbol (module, "g_module_check_init", (gpointer) &check_init))
-       check_failed = check_init (module);
+        if (check_init)
+          check_failed = check_init (module);
       
       /* we don't call unload() if the initialization check failed. */
       if (!check_failed)