diff options
author | Noriko Hosoi <nhosoi@redhat.com> | 2010-06-29 14:34:37 -0700 |
---|---|---|
committer | Noriko Hosoi <nhosoi@redhat.com> | 2010-07-07 17:15:43 -0700 |
commit | 7fa74e30e36274f561ef61f3cae29abf68ce05ae (patch) | |
tree | 775891adf13ea3e405573ab1172f1a57a7b0e142 /lib | |
parent | 56c057e2559cec6acf845cb3196eabb1c7363f7e (diff) | |
download | ds-7fa74e30e36274f561ef61f3cae29abf68ce05ae.tar.gz ds-7fa74e30e36274f561ef61f3cae29abf68ce05ae.tar.xz ds-7fa74e30e36274f561ef61f3cae29abf68ce05ae.zip |
609255 - fix coverity Defect Type: Memory - illegal accesses issues
https://bugzilla.redhat.com/show_bug.cgi?id=609255
12291 USE_AFTER_FREE Triaged Unassigned Bug Minor Fix Required
dbconf_read_default_dbinfo_sub() ds/lib/ldaputil/dbconf.c
Comment:
should never happen - should always break out of the loop at 504
with a valid db_info - but we should set db_info = NULL after line
505 just to make sure
Diffstat (limited to 'lib')
-rw-r--r-- | lib/ldaputil/dbconf.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/lib/ldaputil/dbconf.c b/lib/ldaputil/dbconf.c index cc152168..746afc9a 100644 --- a/lib/ldaputil/dbconf.c +++ b/lib/ldaputil/dbconf.c @@ -503,6 +503,7 @@ int dbconf_read_default_dbinfo_sub (const char *file, { if (!strcmp(db_info->dbname, DBCONF_DEFAULT_DBNAME)) break; dbconf_free_dbinfo(db_info); + db_info = NULL; } if (rv != LDAPU_SUCCESS) { |