diff options
author | Rich Megginson <rmeggins@redhat.com> | 2010-02-25 20:29:35 -0700 |
---|---|---|
committer | Rich Megginson <rmeggins@redhat.com> | 2010-02-26 11:20:01 -0700 |
commit | 9c0edff0cffc5d5aa62b4edb74133008b72b8e94 (patch) | |
tree | 0cd6584e4174004d3211c09034f7869a8caff1d6 | |
parent | 38d13f961c0d8033aa3dd3e5a3a8f32bc1650691 (diff) | |
download | ds-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.c | 2 |
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); |