diff options
author | Alfredo de Greef <eeshlo@yahoo.com> | 2004-11-23 09:10:03 +0300 |
---|---|---|
committer | Alfredo de Greef <eeshlo@yahoo.com> | 2004-11-23 09:10:03 +0300 |
commit | 88b556fd57701b6b6cfc5b47ab1c4f31676ade85 (patch) | |
tree | 22f212db9c2e9847f3ea26aa01e39eb06c37d852 /source/blender/yafray/intern/export_Plugin.cpp | |
parent | f3c8c47adb6ff3e3ca88418f4c26e9795029613f (diff) |
Possibly related to bug #1851:
orco coordinates were assumed to be valid when pointers were non-null,
this seems to have been a wrong assumption, pointers were probably unitialized
(but only for some objects like surfaces?), so when the export code tried to
access them, it crashed.
Now the rendermaterial texco flag is tested instead, which probably is how
it should have been done in the first place...
Diffstat (limited to 'source/blender/yafray/intern/export_Plugin.cpp')
-rw-r--r-- | source/blender/yafray/intern/export_Plugin.cpp | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/source/blender/yafray/intern/export_Plugin.cpp b/source/blender/yafray/intern/export_Plugin.cpp index a46f30b4be1..9a511cc5d7c 100644 --- a/source/blender/yafray/intern/export_Plugin.cpp +++ b/source/blender/yafray/intern/export_Plugin.cpp @@ -1123,7 +1123,12 @@ void yafrayPluginRender_t::writeObject(Object* obj, const vector<VlakRen*> &VLR_ tr = face0mat->ray_mirror; caus_rcolor.set(face0mat->mirr*tr, face0mat->mirg*tr, face0mat->mirb*tr); } - bool has_orco = (face0->v1->orco!=NULL); + + // Export orco coords test. + // Previously was done by checking orco pointer, however this can be non-null but still not initialized. + // Test the rendermaterial texco flag instead. + bool has_orco = ((face0mat->ren->texco & TEXCO_ORCO)!=0); + bool no_auto = true; //in case non-mesh, or mesh has no autosmooth float sm_angle = 0.1f; if (obj->type==OB_MESH) |