summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTomas Bzatek <tbzatek@redhat.com>2013-11-11 17:17:38 +0100
committerTomas Bzatek <tbzatek@redhat.com>2014-01-07 15:24:30 +0100
commita6bd74b75f0517fe24a7cb48b868d20b19fa1044 (patch)
tree06708f738a20443a88ef99a99850ff0694d67364
parent85a88c4acd01a47d7c4a37eeda9473ab315b996f (diff)
downloadopenlmi-providers-a6bd74b75f0517fe24a7cb48b868d20b19fa1044.tar.gz
openlmi-providers-a6bd74b75f0517fe24a7cb48b868d20b19fa1044.tar.xz
openlmi-providers-a6bd74b75f0517fe24a7cb48b868d20b19fa1044.zip
indmanager: Fix potential errorneous dereference
This ideally shouldn't happen in well writen providers.
-rw-r--r--src/indmanager/ind_manager.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/src/indmanager/ind_manager.c b/src/indmanager/ind_manager.c
index 2fae132..46c94eb 100644
--- a/src/indmanager/ind_manager.c
+++ b/src/indmanager/ind_manager.c
@@ -966,12 +966,11 @@ bool _im_add_filter(IMManager *manager, CMPISelectExp *se, IMError *err)
bool im_add_filter(IMManager *manager, CMPISelectExp *filter,
const CMPIContext *ctx, IMError *err)
{
- pthread_mutex_lock(&manager->_t_mutex);
if (!manager) {
*err = IM_ERR_MANAGER;
- pthread_mutex_unlock(&manager->_t_mutex);
return false;
}
+ pthread_mutex_lock(&manager->_t_mutex);
if (!ctx) {
*err = IM_ERR_CONTEXT;
pthread_mutex_unlock(&manager->_t_mutex);