Howard Chu authored Jul 14, 2019 and Quanah Gibson-Mount committed Jul 15, 2019

Don't search for scopes of entries with no children

Howard Chu authored Jan 27, 2015 and Quanah Gibson-Mount committed Jan 27, 2015

Remove assert. This is merely a cache of parent DNs, failure to
insert doesn't affect outcome.

Leonid Yuriev authored Jan 04, 2015 and Quanah Gibson-Mount committed Jan 05, 2015

dn2id.c:
  + assertion for mdb_id2l_insert() result;
  - unnecessary assignment;

search.c:
  - unused variable 'first';
  + range-check for mdb_idl_search() result;

Leonid Yuriev authored Oct 14, 2014 and Quanah Gibson-Mount committed Dec 10, 2014

heap corruption due to returning a reference to a local variable

Howard Chu authored Aug 28, 2014 and Quanah Gibson-Mount committed Aug 28, 2014

Check for duplicate scopes in search_aliases, not later.

Howard Chu authored Aug 28, 2014 and Quanah Gibson-Mount committed Aug 28, 2014

This reverts commit cac00c6a.

Howard Chu authored Aug 26, 2014 and Quanah Gibson-Mount committed Aug 27, 2014

Aliases pointing within the original search scope could cause
the same entry to be returned multiple times. Check for subtree
scope and duplicate scopes.

Howard Chu authored May 15, 2014 and Quanah Gibson-Mount committed May 29, 2014

Better check for missing IDs

Howard Chu authored May 15, 2014 and Quanah Gibson-Mount committed May 29, 2014

mdb_idscopes was remembering the IDs of every entry it checked;
it should only have been saving the IDs of entries that were
actually in the search scope. The extra entries filled the scopes
array, causing a loop searching for a parent entry that was
never inserted.

Howard Chu authored Jan 29, 2013 and Quanah Gibson-Mount committed Jan 29, 2013

uninit'd size in dn2id_walk, bogus pagedresults

Howard Chu authored Jan 26, 2013 and Quanah Gibson-Mount committed Jan 26, 2013

Non-recursive tree walker.

Howard Chu authored Jan 11, 2013 and Quanah Gibson-Mount committed Jan 26, 2013

This is only a 10% gain, but still worth having for now.

Howard Chu authored Oct 06, 2011 and Quanah Gibson-Mount committed Nov 04, 2011

So they can be reused, and avoid unnecessary page_searches

Howard Chu authored Sep 13, 2011 and Quanah Gibson-Mount committed Nov 04, 2011

Tighten up entry_alloc/entry_decode
Track parent nodes in idscopes

Hallvard Furuseth authored Sep 12, 2011 and Quanah Gibson-Mount committed Nov 04, 2011

Also do not cast between berval and MDB_val.  That breaks when
sizeof(berval.bv_len) != sizeof(MDB_val.mv_size).