Skip to content
Snippets Groups Projects
Verified Commit 695d3895 authored by Marek Vavruša's avatar Marek Vavruša Committed by Vladimír Čunát
Browse files

lru: fix case when inserting value with larger size than allocated

This fixes a case when inserting into LRU, and the entry for given
key exists, but has allocated smaller value than what's requested.
parent dac77c7c
Branches
Tags
1 merge request!708misc nitpicks, see commits for details
Pipeline #42459 canceled with stages
in 3 minutes and 50 seconds
......@@ -165,8 +165,15 @@ KR_EXPORT void * lru_get_impl(struct lru *lru, const char *key, uint key_len,
if (g->hashes[i] == khash_top) {
it = g->items[i];
if (likely(it && it->key_len == key_len
&& (key_len == 0 || memcmp(it->data, key, key_len) == 0)))
goto found; // to reduce huge nesting depth
&& (key_len == 0 || memcmp(it->data, key, key_len) == 0))) {
/* Found a key, but trying to insert a value larger than available
* space in the allocated slot, so the entry must be resized to fit. */
if (unlikely(do_insert && val_len > it->val_len)) {
goto insert;
} else {
goto found; // to reduce huge nesting depth
}
}
}
}
// key not found; first try an empty/counted-out place to insert
......
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment