Fix T45423: depsgraph: crash in IDDepsNode::tag_update
authorSergey Sharybin <sergey.vfx@gmail.com>
Tue, 14 Jul 2015 09:19:27 +0000 (11:19 +0200)
committerSergey Sharybin <sergey.vfx@gmail.com>
Tue, 14 Jul 2015 09:21:04 +0000 (11:21 +0200)
Two issues fixed in this commit:

- Clearing or adding animation via python should ensure relations are valid.
- Animation component animation data might be null caused by removing animation
  from python.

source/blender/depsgraph/intern/depsnode.cc
source/blender/makesrna/intern/rna_ID.c

index 1736aadf9999637cafd4761bb7ce7528ac60568b..7d4d6890c834adcda84ab003c770c67e23b2b25b 100644 (file)
@@ -250,8 +250,8 @@ void IDDepsNode::tag_update(Depsgraph *graph)
                bool do_component_tag = comp_node->type != DEPSNODE_TYPE_ANIMATION;
                if (comp_node->type == DEPSNODE_TYPE_ANIMATION) {
                        AnimData *adt = BKE_animdata_from_id(id);
-                       BLI_assert(adt != NULL);
-                       if (adt->recalc & ADT_RECALC_ANIM) {
+                       /* Animation data might be null if relations are tagged for update. */
+                       if (adt != NULL && (adt->recalc & ADT_RECALC_ANIM)) {
                                do_component_tag = true;
                        }
                }
index b87b455b36ff66bb0c9b4fbf82e4a7a0ec892181..f3bd5fc5cefeaac2a12145290c1f86f5876a801c 100644 (file)
@@ -85,6 +85,8 @@ EnumPropertyItem id_type_items[] = {
 
 #ifdef RNA_RUNTIME
 
+#include "DNA_anim_types.h"
+
 #include "BKE_font.h"
 #include "BKE_idprop.h"
 #include "BKE_library.h"
@@ -331,6 +333,19 @@ static void rna_ID_user_clear(ID *id)
        id->flag &= ~LIB_FAKEUSER;
 }
 
+static AnimData * rna_ID_animation_data_create(ID *id, Main *bmain)
+{
+       AnimData *adt = BKE_animdata_add_id(id);
+       DAG_relations_tag_update(bmain);
+       return adt;
+}
+
+static void rna_ID_animation_data_free(ID *id, Main *bmain)
+{
+       BKE_animdata_free(id);
+       DAG_relations_tag_update(bmain);
+}
+
 static void rna_IDPArray_begin(CollectionPropertyIterator *iter, PointerRNA *ptr)
 {
        IDProperty *prop = (IDProperty *)ptr->data;
@@ -835,12 +850,14 @@ static void rna_def_ID(BlenderRNA *brna)
        RNA_def_function_ui_description(func, "Clear the user count of a datablock so its not saved, "
                                        "on reload the data will be removed");
 
-       func = RNA_def_function(srna, "animation_data_create", "BKE_animdata_add_id");
+       func = RNA_def_function(srna, "animation_data_create", "rna_ID_animation_data_create");
+       RNA_def_function_flag(func, FUNC_USE_MAIN);
        RNA_def_function_ui_description(func, "Create animation data to this ID, note that not all ID types support this");
        parm = RNA_def_pointer(func, "anim_data", "AnimData", "", "New animation data or NULL");
        RNA_def_function_return(func, parm);
 
-       func = RNA_def_function(srna, "animation_data_clear", "BKE_animdata_free");
+       func = RNA_def_function(srna, "animation_data_clear", "rna_ID_animation_data_free");
+       RNA_def_function_flag(func, FUNC_USE_MAIN);
        RNA_def_function_ui_description(func, "Clear animation on this this ID");
 
        func = RNA_def_function(srna, "update_tag", "rna_ID_update_tag");