https://bugs.kde.org/show_bug.cgi?id=385386

Reimar Döffinger <k...@reimardoeffinger.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |k...@reimardoeffinger.de

--- Comment #7 from Reimar Döffinger <k...@reimardoeffinger.de> ---
This seems to be a logic bug in the realloc_CEnt function that was never
adjusted for compressed symbol support.
alloc_CEnt has this logic:
   if (fromC) {                                                                
                                              // szB can be arbitrary
   } else {                                                                    
                                              vg_assert(szB ==
CACHE_ENTRY_SIZE);
   }                                                                            

However realloc_CEnt does not have such a fromC argument and unconditionally
checks
vg_assert(szB >= CACHE_ENTRY_SIZE);
Shouldn't these simply be aligned in behaviour?
Unfortunately I can't share any examples, but I would greatly appreciate if
someone could check my logic and consider a patch based on that.
I think it requires a rather large binary with lots of debug symbols, as the
cache re-uses compressed entries last, and that is when this bug happens.

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to