diff options
| author | Roman Rakus <rrakus@redhat.com> | 2013-07-01 12:17:14 +0200 |
|---|---|---|
| committer | Roman Rakus <rrakus@redhat.com> | 2013-07-03 09:55:18 +0200 |
| commit | 38cf8f7ebb9298a930c821f6e43fae168f425d30 (patch) | |
| tree | 7a8a7df795bde7e630e79ed68d6b86bf7a114cc0 | |
| parent | 07d7a7b16b4a0308876b3a3a1e67c0ab7e40fa2b (diff) | |
| download | openlmi-providers-38cf8f7ebb9298a930c821f6e43fae168f425d30.tar.gz openlmi-providers-38cf8f7ebb9298a930c821f6e43fae168f425d30.tar.xz openlmi-providers-38cf8f7ebb9298a930c821f6e43fae168f425d30.zip | |
Account: Don't use uninitialized fields
Fix coverity report for using uninitialized fields
Signed-off-by: Roman Rakus <rrakus@redhat.com>
| -rw-r--r-- | src/account/LMI_AccountProvider.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/src/account/LMI_AccountProvider.c b/src/account/LMI_AccountProvider.c index a34593e..d7b0dac 100644 --- a/src/account/LMI_AccountProvider.c +++ b/src/account/LMI_AccountProvider.c @@ -307,9 +307,10 @@ static CMPIStatus LMI_AccountModifyInstance( lu_ent_add(lue, (ATTR), &val);\ #define PARAMLONG(ATTR, VAR)\ - g_value_set_long(&val, (VAR).value);\ - lu_ent_clear(lue, (ATTR));\ - if (!(VAR).null) lu_ent_add(lue, (ATTR), &val);\ + if (!(VAR).null) {\ + g_value_set_long(&val, (VAR).value);\ + lu_ent_clear(lue, (ATTR));\ + lu_ent_add(lue, (ATTR), &val);}\ /* This macro will get property named NAME and save it into `value' variable */ #define GETSTRVALUE(NAME)\ |