Fix T43718: FBX: Do not 'crash' importer when there is too much data in a layer.

FBX... Looks like even using the SDK does not avoid many mistakes (I would suspect the countrary, actually).
This commit is contained in:
Bastien Montagne 2015-02-18 13:00:13 +01:00
parent 17a28df49c
commit de7d011110
Notes: blender-bot 2023-02-14 19:58:43 +01:00
Referenced by issue #43718, FBX: exception during fbx importing
1 changed files with 16 additions and 2 deletions

View File

@ -702,15 +702,29 @@ def blen_read_geom_layerinfo(fbx_layer):
def blen_read_geom_array_setattr(generator, blen_data, blen_attr, fbx_data, stride, item_size, descr, xform):
"""Generic fbx_layer to blen_data setter, generator is expected to yield tuples (ble_idx, fbx_idx)."""
max_idx = len(blen_data) - 1
print_error = True
def check_skip(blen_idx, fbx_idx):
nonlocal print_error
if fbx_idx == -1:
return True
if blen_idx > max_idx:
if print_error:
print("ERROR: too much data in this layer, compared to elements in mesh, skipping!")
print_error = False
return True
return False
if xform is not None:
for blen_idx, fbx_idx in generator:
if fbx_idx == -1:
if check_skip(blen_idx, fbx_idx):
continue
setattr(blen_data[blen_idx], blen_attr,
xform(fbx_data[fbx_idx] if (item_size == 1) else fbx_data[fbx_idx:fbx_idx + item_size]))
else:
for blen_idx, fbx_idx in generator:
if fbx_idx == -1:
if check_skip(blen_idx, fbx_idx):
continue
setattr(blen_data[blen_idx], blen_attr,
fbx_data[fbx_idx] if (item_size == 1) else fbx_data[fbx_idx:fbx_idx + item_size])