(ITS#6719) syncrepl_add_glue_ancestors() uninitialized return

[h.b.furuseth@usit.uio.no]

Full_Name: Hallvard B Furuseth
Version: 2.4.23, HEAD
OS: 
URL: ftp://ftp.openldap.org/incoming/
Submission from: (NULL) (129.240.6.233)
Submitted by: hallvard


syncrepl_add_glue_ancestors() returns 'rc' uninitialized if the 'else'
branch of 'if ( ndn.bv_len > be->be_nsuffix[0].bv_len )' is taken: Then
the loop is not entered.

This also means the 'else' branch is useless.  Should it be doing
something more?

Found with gcc -O -Wuninitialized .  'make test' does not seem to enter
the 'else'.