RNA: report error on struct naming collision
authorCampbell Barton <ideasman42@gmail.com>
Wed, 23 Aug 2017 04:59:14 +0000 (14:59 +1000)
committerCampbell Barton <ideasman42@gmail.com>
Wed, 23 Aug 2017 05:02:21 +0000 (15:02 +1000)
Fixes T52463, error instead of crash.

source/blender/makesrna/RNA_access.h
source/blender/makesrna/intern/rna_ID.c
source/blender/makesrna/intern/rna_access.c
source/blender/makesrna/intern/rna_animation.c
source/blender/makesrna/intern/rna_nodetree.c
source/blender/makesrna/intern/rna_render.c
source/blender/makesrna/intern/rna_ui.c
source/blender/makesrna/intern/rna_userdef.c
source/blender/makesrna/intern/rna_wm.c

index a1af3f982744a7159a54dca0251c79a4c226b4dc..2a1554bcb8c6ec6489a497c52065e18d7e31e1d8 100644 (file)
@@ -783,6 +783,8 @@ const struct ListBase *RNA_struct_type_functions(StructRNA *srna);
 
 char *RNA_struct_name_get_alloc(PointerRNA *ptr, char *fixedbuf, int fixedlen, int *r_len);
 
+bool RNA_struct_available_or_report(struct ReportList *reports, const char *identifier);
+
 /* Properties
  *
  * Access to struct properties. All this works with RNA pointers rather than
index a74758a4f71763df6c00c69932f0d8d35793a210..9d7dd7f424aa32ca3e823b30b71e67f2bf11d96c 100644 (file)
@@ -273,6 +273,10 @@ StructRNA *rna_PropertyGroup_register(Main *UNUSED(bmain), ReportList *reports,
                return NULL;
        }
 
+       if (!RNA_struct_available_or_report(reports, identifier)) {
+               return NULL;
+       }
+
        return RNA_def_struct_ptr(&BLENDER_RNA, identifier, &RNA_PropertyGroup);  /* XXX */
 }
 
index 89348bb8f6ce4c78baebccd50502b9a5040bdebf..15b535625df0d833915f397f19da99e56a197636 100644 (file)
@@ -814,6 +814,38 @@ char *RNA_struct_name_get_alloc(PointerRNA *ptr, char *fixedbuf, int fixedlen, i
        return NULL;
 }
 
+bool RNA_struct_available_or_report(ReportList *reports, const char *identifier)
+{
+       const StructRNA *srna_exists = RNA_struct_find(identifier);
+       if (UNLIKELY(srna_exists != NULL)) {
+               /* Use comprehensive string construction since this is such a rare occurrence
+                * and information here may cut down time troubleshooting. */
+               DynStr *dynstr = BLI_dynstr_new();
+               BLI_dynstr_appendf(dynstr, "Type identifier '%s' is already in use: '", identifier);
+               BLI_dynstr_append(dynstr, srna_exists->identifier);
+               int i = 0;
+               if (srna_exists->base) {
+                       for (const StructRNA *base = srna_exists->base; base; base = base->base) {
+                               BLI_dynstr_append(dynstr, "(");
+                               BLI_dynstr_append(dynstr, base->identifier);
+                               i += 1;
+                       }
+                       while (i--) {
+                               BLI_dynstr_append(dynstr, ")");
+                       }
+               }
+               BLI_dynstr_append(dynstr, "'.");
+               char *result = BLI_dynstr_get_cstring(dynstr);
+               BLI_dynstr_free(dynstr);
+               BKE_report(reports, RPT_ERROR, result);
+               MEM_freeN(result);
+               return false;
+       }
+       else {
+               return true;
+       }
+}
+
 /* Property Information */
 
 const char *RNA_property_identifier(PropertyRNA *prop)
index f271bccd326ca850caa47c4b20a0bbc0fc002af2..b0a51fd8c73d148262eb9d3c5f3777f7ba340f0d 100644 (file)
@@ -270,9 +270,13 @@ static StructRNA *rna_KeyingSetInfo_register(Main *bmain, ReportList *reports, v
        
        /* check if we have registered this info before, and remove it */
        ksi = ANIM_keyingset_info_find_name(dummyksi.idname);
-       if (ksi && ksi->ext.srna)
+       if (ksi && ksi->ext.srna) {
                rna_KeyingSetInfo_unregister(bmain, ksi->ext.srna);
-       
+       }
+       if (!RNA_struct_available_or_report(reports, identifier)) {
+               return NULL;
+       }
+
        /* create a new KeyingSetInfo type */
        ksi = MEM_callocN(sizeof(KeyingSetInfo), "python keying set info");
        memcpy(ksi, &dummyksi, sizeof(KeyingSetInfo));
index 00b881dabc7539785df88c2d577bafb3989ca818..f15605d0f83f2741de1543b83f1beec85ccf1e32 100644 (file)
@@ -630,9 +630,13 @@ static StructRNA *rna_NodeTree_register(
 
        /* check if we have registered this tree type before, and remove it */
        nt = ntreeTypeFind(dummynt.idname);
-       if (nt)
+       if (nt) {
                rna_NodeTree_unregister(bmain, nt->ext.srna);
-       
+       }
+       if (!RNA_struct_available_or_report(reports, identifier)) {
+               return NULL;
+       }
+
        /* create a new node tree type */
        nt = MEM_callocN(sizeof(bNodeTreeType), "node tree type");
        memcpy(nt, &dummynt, sizeof(dummynt));
@@ -1389,11 +1393,18 @@ static bNodeType *rna_Node_register_base(Main *bmain, ReportList *reports, Struc
                            identifier, (int)sizeof(dummynt.idname));
                return NULL;
        }
+       if (!RNA_struct_available_or_report(reports, identifier)) {
+               return NULL;
+       }
 
        /* check if we have registered this node type before, and remove it */
        nt = nodeTypeFind(dummynt.idname);
-       if (nt)
+       if (nt) {
                rna_Node_unregister(bmain, nt->ext.srna);
+       }
+       if (!RNA_struct_available_or_report(reports, identifier)) {
+               return NULL;
+       }
        
        /* create a new node type */
        nt = MEM_callocN(sizeof(bNodeType), "node type");
index a66c160ed1a3ebbb9491891db8098d45f6e5edb8..0c0484e633b5f24411d95b62c6632320a4120980 100644 (file)
@@ -321,6 +321,9 @@ static StructRNA *rna_RenderEngine_register(Main *bmain, ReportList *reports, vo
                        break;
                }
        }
+       if (!RNA_struct_available_or_report(reports, identifier)) {
+               return NULL;
+       }
        
        /* create a new engine type */
        et = MEM_callocN(sizeof(RenderEngineType), "python render engine");
index 64b41ac789fc3356d045141ce4b56c7a93107482..0fc0739f86045f224f3544154c8301ffcb764d08 100644 (file)
@@ -229,6 +229,9 @@ static StructRNA *rna_Panel_register(Main *bmain, ReportList *reports, void *dat
                        break;
                }
        }
+       if (!RNA_struct_available_or_report(reports, identifier)) {
+               return NULL;
+       }
        
        /* create a new panel type */
        pt = MEM_callocN(sizeof(PanelType), "python buttons panel");
@@ -488,8 +491,12 @@ static StructRNA *rna_UIList_register(Main *bmain, ReportList *reports, void *da
 
        /* check if we have registered this uilist type before, and remove it */
        ult = WM_uilisttype_find(dummyult.idname, true);
-       if (ult && ult->ext.srna)
+       if (ult && ult->ext.srna) {
                rna_UIList_unregister(bmain, ult->ext.srna);
+       }
+       if (!RNA_struct_available_or_report(reports, identifier)) {
+               return NULL;
+       }
 
        /* create a new menu type */
        ult = MEM_callocN(sizeof(uiListType) + over_alloc, "python uilist");
@@ -592,6 +599,9 @@ static StructRNA *rna_Header_register(Main *bmain, ReportList *reports, void *da
                        break;
                }
        }
+       if (!RNA_struct_available_or_report(reports, identifier)) {
+               return NULL;
+       }
        
        /* create a new header type */
        ht = MEM_callocN(sizeof(HeaderType), "python buttons header");
@@ -712,8 +722,12 @@ static StructRNA *rna_Menu_register(Main *bmain, ReportList *reports, void *data
 
        /* check if we have registered this menu type before, and remove it */
        mt = WM_menutype_find(dummymt.idname, true);
-       if (mt && mt->ext.srna)
+       if (mt && mt->ext.srna) {
                rna_Menu_unregister(bmain, mt->ext.srna);
+       }
+       if (!RNA_struct_available_or_report(reports, identifier)) {
+               return NULL;
+       }
        
        /* create a new menu type */
        if (_menu_descr[0]) {
index 19a25b4df11d55d2b89940f2b2a54a5b70355b1a..3ecacd11a5c7713bbb1cf6c33d7f77479b9f040a 100644 (file)
@@ -638,10 +638,11 @@ static StructRNA *rna_AddonPref_register(Main *bmain, ReportList *reports, void
 
        /* check if we have registered this header type before, and remove it */
        apt = BKE_addon_pref_type_find(dummyaddon.module, true);
-       if (apt) {
-               if (apt->ext.srna) {
-                       rna_AddonPref_unregister(bmain, apt->ext.srna);
-               }
+       if (apt && apt->ext.srna) {
+               rna_AddonPref_unregister(bmain, apt->ext.srna);
+       }
+       if (!RNA_struct_available_or_report(reports, identifier)) {
+               return NULL;
        }
 
        /* create a new header type */
index 4aee03025f62da1382114b5e91c9bca3f06da3e3..a7afc405048ffb4da16d3941894f8bfb6d8356af 100644 (file)
@@ -1147,6 +1147,16 @@ static StructRNA *rna_Operator_register(
        if (validate(&dummyotr, data, have_function) != 0)
                return NULL;
 
+       /* check if we have registered this operator type before, and remove it */
+       {
+               wmOperatorType *ot = WM_operatortype_find(dummyot.idname, true);
+               if (ot && ot->ext.srna)
+                       rna_Operator_unregister(bmain, ot->ext.srna);
+       }
+       if (!RNA_struct_available_or_report(reports, identifier)) {
+               return NULL;
+       }
+
        {   /* convert foo.bar to FOO_OT_bar
                 * allocate the description and the idname in 1 go */
 
@@ -1214,13 +1224,6 @@ static StructRNA *rna_Operator_register(
                }
        }
 
-       /* check if we have registered this operator type before, and remove it */
-       {
-               wmOperatorType *ot = WM_operatortype_find(dummyot.idname, true);
-               if (ot && ot->ext.srna)
-                       rna_Operator_unregister(bmain, ot->ext.srna);
-       }
-
        /* XXX, this doubles up with the operator name [#29666]
         * for now just remove from dir(bpy.types) */
 
@@ -1323,6 +1326,16 @@ static StructRNA *rna_MacroOperator_register(
                return NULL;
        }
 
+       /* check if we have registered this operator type before, and remove it */
+       {
+               wmOperatorType *ot = WM_operatortype_find(dummyot.idname, true);
+               if (ot && ot->ext.srna)
+                       rna_Operator_unregister(bmain, ot->ext.srna);
+       }
+       if (!RNA_struct_available_or_report(reports, identifier)) {
+               return NULL;
+       }
+
        {   /* convert foo.bar to FOO_OT_bar
                 * allocate the description and the idname in 1 go */
                const uint idname_len = strlen(temp_buffers.idname) + 4;
@@ -1349,13 +1362,6 @@ static StructRNA *rna_MacroOperator_register(
                dummyot.undo_group = ch;
        }
 
-       /* check if we have registered this operator type before, and remove it */
-       {
-               wmOperatorType *ot = WM_operatortype_find(dummyot.idname, true);
-               if (ot && ot->ext.srna)
-                       rna_Operator_unregister(bmain, ot->ext.srna);
-       }
-
        /* XXX, this doubles up with the operator name [#29666]
         * for now just remove from dir(bpy.types) */