diff options
author | Campbell Barton <ideasman42@gmail.com> | 2013-11-27 06:42:24 +0400 |
---|---|---|
committer | Campbell Barton <ideasman42@gmail.com> | 2013-11-27 06:42:24 +0400 |
commit | 45eed8246a8803efb23693a2b09d020daa5dd22a (patch) | |
tree | 617ff1fce093c9aab048c581cee448f0db5aa160 | |
parent | 4dd949632acd352dcac9432a89d3f33e1d2bc3da (diff) |
BMesh Operators: BMO_op_finish now overwrites values in debug mode
This should prevent accidental use-after-free.
-rw-r--r-- | source/blender/bmesh/intern/bmesh_operators.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/source/blender/bmesh/intern/bmesh_operators.c b/source/blender/bmesh/intern/bmesh_operators.c index 611ee4d03b8..b4439703c4a 100644 --- a/source/blender/bmesh/intern/bmesh_operators.c +++ b/source/blender/bmesh/intern/bmesh_operators.c @@ -238,6 +238,9 @@ void BMO_op_finish(BMesh *bm, BMOperator *op) #ifdef DEBUG BM_ELEM_INDEX_VALIDATE(bm, "post bmo", bmo_opdefines[op->type]->opname); + + /* avoid accidental re-use */ + memset(op, 0xff, sizeof(*op)); #else (void)bm; #endif |