diff options
author | Hans Goudey <h.goudey@me.com> | 2020-10-15 18:08:39 +0300 |
---|---|---|
committer | Hans Goudey <h.goudey@me.com> | 2020-10-15 18:08:39 +0300 |
commit | e7ab802ab04d25532ee16153c55eaac290c877b0 (patch) | |
tree | 79323d094c9b7f15c85769153c359986d06ddc50 /source/blender/blenkernel/BKE_lib_remap.h | |
parent | 15d78ea85b602cfda4e4b825d44bede71a06d86f (diff) |
Fix T81697: Property search crash with python handlers
Previously I used `CTX_copy` to create a mutable copy of the context in
order to set its area and region fields to temporary variables. This was
a tradeoff to avoid casting away `const` for `bContext`.
However, `bpy.context` is set to this new temporary value, which is fine
for a single `wm_draw_update` pass, but in the next main loop,
`bpy.context` is still set to this value, which was freed at the
end of `property_search_all_tabs`. It would be possible to reset the
`bpy.context` variable ath the end of the function, but this patch
contains an alternate solution: just don't copy the context. It looks
like this was the only use of `CTX_copy` anyway, maybe for good reason.
Differential Revision: https://developer.blender.org/D9216
Diffstat (limited to 'source/blender/blenkernel/BKE_lib_remap.h')
0 files changed, 0 insertions, 0 deletions