summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRich Megginson <rmeggins@redhat.com>2010-02-25 20:29:35 -0700
committerRich Megginson <rmeggins@redhat.com>2010-02-26 11:20:01 -0700
commit9c0edff0cffc5d5aa62b4edb74133008b72b8e94 (patch)
tree0cd6584e4174004d3211c09034f7869a8caff1d6
parent38d13f961c0d8033aa3dd3e5a3a8f32bc1650691 (diff)
downloadds-9c0edff0cffc5d5aa62b4edb74133008b72b8e94.tar.gz
ds-9c0edff0cffc5d5aa62b4edb74133008b72b8e94.tar.xz
ds-9c0edff0cffc5d5aa62b4edb74133008b72b8e94.zip
389 DS segfaults on libsyntax-plugin.so - part 3
https://bugzilla.redhat.com/show_bug.cgi?id=516611 Resolves: bug 516611 Bug Description: 389 DS segfaults on libsyntax-plugin.so - part 3 Reviewed by: nkinder (Thanks!) Branch: HEAD Fix Description: bin_filter_ava should check for null bvals Platforms tested: RHEL5 x86_64 Flag Day: no Doc impact: no
-rw-r--r--ldap/servers/plugins/syntaxes/bin.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/ldap/servers/plugins/syntaxes/bin.c b/ldap/servers/plugins/syntaxes/bin.c
index 3d2d88ac..25f21298 100644
--- a/ldap/servers/plugins/syntaxes/bin.c
+++ b/ldap/servers/plugins/syntaxes/bin.c
@@ -251,7 +251,7 @@ bin_filter_ava( Slapi_PBlock *pb, struct berval *bvfilter,
{
int i;
- for ( i = 0; bvals[i] != NULL; i++ ) {
+ for ( i = 0; (bvals != NULL) && (bvals[i] != NULL); i++ ) {
const struct berval *bv = slapi_value_get_berval(bvals[i]);
int rc = slapi_berval_cmp(bv, bvfilter);