Revert the context "fix" - on error is good to know the error from the context.
authorFrantišek Dvořák <valtri@civ.zcu.cz>
Fri, 6 Jun 2008 14:04:31 +0000 (14:04 +0000)
committerFrantišek Dvořák <valtri@civ.zcu.cz>
Fri, 6 Jun 2008 14:04:31 +0000 (14:04 +0000)
org.glite.lbjp-common.db/src/db.c

index 9827820..5738141 100644 (file)
@@ -192,11 +192,8 @@ int glite_lbu_InitDBContext(glite_lbu_DBContext *ctx) {
        pthread_mutex_lock(&db_handle.lock);
        if (!db_handle.lib) {
                if ((!MYSQL_LIBPATH[0] || (db_handle.lib = dlopen(MYSQL_LIBPATH, RTLD_LAZY | RTLD_LOCAL)) == NULL) &&
-                   (db_handle.lib = dlopen("libmysqlclient.so", RTLD_LAZY | RTLD_LOCAL)) == NULL) {
-                       free(*ctx);
-                       *ctx = NULL;
+                   (db_handle.lib = dlopen("libmysqlclient.so", RTLD_LAZY | RTLD_LOCAL)) == NULL)
                        return ERR(*ctx, ENOENT, "can't load '%s' or 'libmysqlclient.so' (%s)", MYSQL_LIBPATH, dlerror());
-               }
                do {
                        LOAD(mysql_init, "mysql_init");
                        LOAD(mysql_get_client_version, "mysql_get_client_version");
@@ -244,8 +241,6 @@ int glite_lbu_InitDBContext(glite_lbu_DBContext *ctx) {
                        dlclose(db_handle.lib);
                        db_handle.lib = NULL;
                        pthread_mutex_unlock(&db_handle.lock);
-                       free(*ctx);
-                       *ctx = NULL;
                        return err;
                }
        } else pthread_mutex_unlock(&db_handle.lock);