[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [25372] trunk/blender/source/blender/ render/intern/source/occlusion.c: Fix #20364 and #20357: AAO crashes for certain combinations of settings.

[Brecht Van Lommel]

Revision: 25372
          
http://projects.blender.org/plugins/scmsvn/viewcvs.php?view=rev&root=bf-blender&revision=25372
Author:   blendix
Date:     2009-12-14 16:05:09 +0100 (Mon, 14 Dec 2009)

Log Message:
-----------
Fix #20364 and #20357: AAO crashes for certain combinations of settings.

Modified Paths:
--------------
    trunk/blender/source/blender/render/intern/source/occlusion.c

Modified: trunk/blender/source/blender/render/intern/source/occlusion.c
===================================================================
--- trunk/blender/source/blender/render/intern/source/occlusion.c       
2009-12-14 14:51:42 UTC (rev 25371)
+++ trunk/blender/source/blender/render/intern/source/occlusion.c       
2009-12-14 15:05:09 UTC (rev 25372)
@@ -356,18 +356,18 @@
 {
        OccNode *child;
        float occ, area, totarea, rad[3];
-       int a, b;
+       int a, b, indirect= tree->doindirect;
 
        occ= 0.0f;
        totarea= 0.0f;
-       zero_v3(rad);
+       if(indirect) zero_v3(rad);
 
        for(b=0; b<TOTCHILD; b++) {
                if(node->childflag & (1<<b)) {
                        a= node->child[b].face;
                        occ_face(&tree->face[a], 0, 0, &area);
                        occ += area*tree->occlusion[a];
-                       madd_v3_v3fl(rad, tree->rad[a], area);
+                       if(indirect) madd_v3_v3fl(rad, tree->rad[a], area);
                        totarea += area;
                }
                else if(node->child[b].node) {
@@ -375,18 +375,18 @@
                        occ_sum_occlusion(tree, child);
 
                        occ += child->area*child->occlusion;
-                       madd_v3_v3fl(rad, child->rad, child->area);
+                       if(indirect) madd_v3_v3fl(rad, child->rad, child->area);
                        totarea += child->area;
                }
        }
 
        if(totarea != 0.0f) {
                occ /= totarea;
-               mul_v3_fl(rad, 1.0f/totarea);
+               if(indirect) mul_v3_fl(rad, 1.0f/totarea);
        }
        
        node->occlusion= occ;
-       copy_v3_v3(node->rad, rad);
+       if(indirect) copy_v3_v3(node->rad, rad);
 }
 
 static int occ_find_bbox_axis(OcclusionTree *tree, int begin, int end, float 
*min, float *max)
@@ -1613,6 +1613,7 @@
 void make_occ_tree(Render *re)
 {
        OcclusionThread othreads[BLENDER_MAX_THREADS];
+       OcclusionTree *tree;
        StrandSurface *mesh;
        ListBase threads;
        float ao[3], indirect[3], (*faceao)[3], (*faceindirect)[3];
@@ -1624,13 +1625,13 @@
        re->i.infostr= "Occlusion preprocessing";
        re->stats_draw(re->sdh, &re->i);
        
-       re->occlusiontree= occ_tree_build(re);
+       re->occlusiontree= tree= occ_tree_build(re);
        
-       if(re->occlusiontree) {
+       if(tree) {
                if(re->wrld.ao_approx_passes > 0)
-                       occ_compute_passes(re, re->occlusiontree, 
re->wrld.ao_approx_passes);
-               if(re->wrld.ao_indirect_bounces > 1)
-                       occ_compute_bounces(re, re->occlusiontree, 
re->wrld.ao_indirect_bounces);
+                       occ_compute_passes(re, tree, re->wrld.ao_approx_passes);
+               if(tree->doindirect && re->wrld.ao_indirect_bounces > 1)
+                       occ_compute_bounces(re, tree, 
re->wrld.ao_indirect_bounces);
 
                for(mesh=re->strandsurface.first; mesh; mesh=mesh->next) {
                        if(!mesh->face || !mesh->co || !mesh->ao)


_______________________________________________
Bf-blender-cvs mailing list
Bf-blender-cvs@blender.org
http://lists.blender.org/mailman/listinfo/bf-blender-cvs