diff options
| author | Noriko Hosoi <nhosoi@redhat.com> | 2010-07-02 17:15:22 -0700 |
|---|---|---|
| committer | Noriko Hosoi <nhosoi@redhat.com> | 2010-07-23 13:35:13 -0700 |
| commit | c766b7ac3ce68eae18ace54805bdd3badcaf7a80 (patch) | |
| tree | e298bafa117c5ed8ed12903cbd0399f660defacd /lib/libaccess/register.cpp | |
| parent | c38c2a860937cda0cdbe8f05acaa8083feb8caf1 (diff) | |
| download | ds-c766b7ac3ce68eae18ace54805bdd3badcaf7a80.tar.gz ds-c766b7ac3ce68eae18ace54805bdd3badcaf7a80.tar.xz ds-c766b7ac3ce68eae18ace54805bdd3badcaf7a80.zip | |
610281 - fix coverity Defect Type: Control flow issues
https://bugzilla.redhat.com/show_bug.cgi?id=610281
11814 DEADCODE Triaged Unassigned Bug Moderate Fix Required
string_filter_sub() ds/ldap/servers/plugins/syntaxes/string.c
Comment:
A code to update tmpbufsize was missing. This "tpbufsize = len + 1;"
is needed before slapi_ch_realloc.
351 tmpbufsize = len + 1;
352 tmpbuf = (char *) slapi_ch_realloc( tmpbuf, tmpbufsize );
Also, if (len < tmpbufsize) were true (could not be true since
tmpbufsize never have been set), bvp->bv_val was copied to buf
which is not long enough for bvp->bv_val. The bug was also
fixed.
Diffstat (limited to 'lib/libaccess/register.cpp')
0 files changed, 0 insertions, 0 deletions