On Wed, Aug 28, 2013 at 04:35:14PM +0200, Michal Židek wrote:
Just a small typo... i think this can be amended before
pushing.
On 08/20/2013 10:32 AM, Lukas Slebodnik wrote:
>- if (hash > mcc->ht_size) {
>- /* Invalid hash. This should never happen, but better
>- * return than trying to access out of bounds memory */
>+ if (hash > MC_HT_ELEMS(mcc->ht_size)) {
>+ /* It can happen if rec->hash1 and rec->has2 was the same.
^^^^ s/has2/hash2
>+ * or it is invalid hash. It is better to return
>+ * than trying to access out of bounds memory
>+ */
> return;
> }
But he code itself is OK.
Michal
Thanks for the second look, I tested the code and memcache works OK for
me, but the memcache is one area of SSSD I don't really know well.