Revision: 48456
          http://brlcad.svn.sourceforge.net/brlcad/?rev=48456&view=rev
Author:   starseeker
Date:     2012-01-12 00:10:50 +0000 (Thu, 12 Jan 2012)
Log Message:
-----------
Coverity found a path that gets to this point in animedit with test_hold null - 
check for the null, but may be symptomatic of deeper issue.  CID 309

Modified Paths:
--------------
    brlcad/trunk/src/mged/animedit.c

Modified: brlcad/trunk/src/mged/animedit.c
===================================================================
--- brlcad/trunk/src/mged/animedit.c    2012-01-12 00:02:31 UTC (rev 48455)
+++ brlcad/trunk/src/mged/animedit.c    2012-01-12 00:10:50 UTC (rev 48456)
@@ -2613,6 +2613,7 @@
        ssp = (struct solve_stack *) solve_head.forw;
 
        i = (2<<6) - 1;         /* Six degrees of freedom */
+        if (test_hold) {  /* make sure we've got test_hold */
        for (BU_LIST_FOR(jh, jointH, &test_hold->j_head)) {
            if (ssp->jp != jh->p) {
                i &= jh->flag;
@@ -2625,6 +2626,7 @@
            /* All joints, all freedoms */
            test_hold->flag |= HOLD_FLAG_TRIED;
        }
+        }
        reject_move();
        goto Middle;
     }

This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.


------------------------------------------------------------------------------
Ridiculously easy VDI. With Citrix VDI-in-a-Box, you don't need a complex
infrastructure or vast IT resources to deliver seamless, secure access to
virtual desktops. With this all-in-one solution, easily deploy virtual 
desktops for less than the cost of PCs and save 60% on VDI infrastructure 
costs. Try it free! http://p.sf.net/sfu/Citrix-VDIinabox
_______________________________________________
BRL-CAD Source Commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/brlcad-commits

Reply via email to