diff options
author | Sybren A. Stüvel <sybren@blender.org> | 2021-04-08 19:54:55 +0300 |
---|---|---|
committer | Sybren A. Stüvel <sybren@blender.org> | 2021-04-08 19:59:22 +0300 |
commit | b51562ed76d5428ed4d511cd248d1b059e341661 (patch) | |
tree | 43e6955839b8257e07ee190569e8b9f2daee644b /intern | |
parent | 8777a44ccbb04f5015eb6a71c70f8c1efe83e61b (diff) |
Fix T87232: Crash when evaluating object with unsupported modifier
Fix `nullptr` redeference when setting 'orig_data' pointers on CoW copies,
by stopping the loop also when `element_cow == nullptr`. This avoids a
crash of Blender when the original list of pointers is longer than the
CoW list of pointers.
I've also added a `BLI_assert()` that checks for equal lengths of the
two `ListBase`s, so that problems like these aren't hidden away completely.
The root cause of the crash was actually a modifier that was assigned to
an object of the wrong type (an Armature object doesn't support modifiers).
This caused the list of modifiers on the CoW copy to be shorter than the
list of modifiers on the original Object. It's still a mystery how that
object got that modifier in the first place.
Diffstat (limited to 'intern')
0 files changed, 0 insertions, 0 deletions