diff options
author | Brecht Van Lommel <brechtvanlommel@gmail.com> | 2019-11-06 22:03:22 +0300 |
---|---|---|
committer | Brecht Van Lommel <brechtvanlommel@gmail.com> | 2019-11-06 22:06:33 +0300 |
commit | 5abd006c78597f4d3a87e04ca63c1cca367c10bd (patch) | |
tree | ca4fd7695ee35b5ae48a104896a7af81e1825c95 /source/blender/blenloader | |
parent | 6528fcaeea6243c376699b987e1770ed155e19ae (diff) |
Fix T70279: crash loading certain old, possibly corrupt files with nodes
This test should not be needed and the cause is unclear, but better to avoid
the crash. Possibly caused by files saved with development versions that had
different node type IDs.
Diffstat (limited to 'source/blender/blenloader')
-rw-r--r-- | source/blender/blenloader/intern/versioning_cycles.c | 22 |
1 files changed, 11 insertions, 11 deletions
diff --git a/source/blender/blenloader/intern/versioning_cycles.c b/source/blender/blenloader/intern/versioning_cycles.c index 2c4ba4a1102..5610222e85d 100644 --- a/source/blender/blenloader/intern/versioning_cycles.c +++ b/source/blender/blenloader/intern/versioning_cycles.c @@ -221,7 +221,7 @@ static void square_roughness_node_insert(bNodeTree *ntree) static void mapping_node_order_flip(bNode *node) { /* Flip euler order of mapping shader node */ - if (node->type == SH_NODE_MAPPING) { + if (node->type == SH_NODE_MAPPING && node->storage) { TexMapping *texmap = node->storage; float quat[4]; @@ -233,7 +233,7 @@ static void mapping_node_order_flip(bNode *node) static void vector_curve_node_remap(bNode *node) { /* Remap values of vector curve node from normalized to absolute values */ - if (node->type == SH_NODE_CURVE_VEC) { + if (node->type == SH_NODE_CURVE_VEC && node->storage) { CurveMapping *mapping = node->storage; mapping->flag &= ~CUMA_DO_CLIP; @@ -300,11 +300,11 @@ static void image_node_colorspace(bNode *node) } int color_space; - if (node->type == SH_NODE_TEX_IMAGE) { + if (node->type == SH_NODE_TEX_IMAGE && node->storage) { NodeTexImage *tex = node->storage; color_space = tex->color_space; } - else if (node->type == SH_NODE_TEX_ENVIRONMENT) { + else if (node->type == SH_NODE_TEX_ENVIRONMENT && node->storage) { NodeTexEnvironment *tex = node->storage; color_space = tex->color_space; } @@ -766,7 +766,7 @@ static void update_vector_math_node_average_operator(bNodeTree *ntree) static void update_noise_node_dimensions(bNodeTree *ntree) { for (bNode *node = ntree->nodes.first; node; node = node->next) { - if (node->type == SH_NODE_TEX_NOISE) { + if (node->type == SH_NODE_TEX_NOISE && node->storage) { NodeTexNoise *tex = (NodeTexNoise *)node->storage; tex->dimensions = 3; } @@ -923,7 +923,7 @@ static void update_mapping_node_inputs_and_properties(bNodeTree *ntree) static void update_musgrave_node_dimensions(bNodeTree *ntree) { for (bNode *node = ntree->nodes.first; node; node = node->next) { - if (node->type == SH_NODE_TEX_MUSGRAVE) { + if (node->type == SH_NODE_TEX_MUSGRAVE && node->storage) { NodeTexMusgrave *tex = (NodeTexMusgrave *)node->storage; tex->dimensions = 3; } @@ -951,7 +951,7 @@ static void update_musgrave_node_color_output(bNodeTree *ntree) static void update_voronoi_node_dimensions(bNodeTree *ntree) { for (bNode *node = ntree->nodes.first; node; node = node->next) { - if (node->type == SH_NODE_TEX_VORONOI) { + if (node->type == SH_NODE_TEX_VORONOI && node->storage) { NodeTexVoronoi *tex = (NodeTexVoronoi *)node->storage; tex->dimensions = 3; } @@ -966,7 +966,7 @@ static void update_voronoi_node_dimensions(bNodeTree *ntree) static void update_voronoi_node_f3_and_f4(bNodeTree *ntree) { for (bNode *node = ntree->nodes.first; node; node = node->next) { - if (node->type == SH_NODE_TEX_VORONOI) { + if (node->type == SH_NODE_TEX_VORONOI && node->storage) { NodeTexVoronoi *tex = (NodeTexVoronoi *)node->storage; if (ELEM(tex->feature, 2, 3)) { tex->feature = SHD_VORONOI_F2; @@ -1014,7 +1014,7 @@ static void update_voronoi_node_crackle(bNodeTree *ntree) bool need_update = false; for (bNode *node = ntree->nodes.first; node; node = node->next) { - if (node->type == SH_NODE_TEX_VORONOI) { + if (node->type == SH_NODE_TEX_VORONOI && node->storage) { NodeTexVoronoi *tex = (NodeTexVoronoi *)node->storage; bNodeSocket *sockDistance = nodeFindSocket(node, SOCK_OUT, "Distance"); bNodeSocket *sockColor = nodeFindSocket(node, SOCK_OUT, "Color"); @@ -1106,7 +1106,7 @@ static void update_voronoi_node_coloring(bNodeTree *ntree) LISTBASE_FOREACH_BACKWARD_MUTABLE (bNodeLink *, link, &ntree->links) { bNode *node = link->fromnode; - if (node && node->type == SH_NODE_TEX_VORONOI) { + if (node && node->type == SH_NODE_TEX_VORONOI && node->storage) { NodeTexVoronoi *tex = (NodeTexVoronoi *)node->storage; if (tex->coloring == 0) { bNodeSocket *sockColor = nodeFindSocket(node, SOCK_OUT, "Color"); @@ -1143,7 +1143,7 @@ static void update_voronoi_node_square_distance(bNodeTree *ntree) bool need_update = false; for (bNode *node = ntree->nodes.first; node; node = node->next) { - if (node->type == SH_NODE_TEX_VORONOI) { + if (node->type == SH_NODE_TEX_VORONOI && node->storage) { NodeTexVoronoi *tex = (NodeTexVoronoi *)node->storage; bNodeSocket *sockDistance = nodeFindSocket(node, SOCK_OUT, "Distance"); if (tex->distance == SHD_VORONOI_EUCLIDEAN && |