[ODE] Small bug in collision_kernel.cpp

John Miles jmiles at pop.net
Mon Apr 19 10:59:45 MST 2004


I reported this a long time ago, and I thought they'd fixed it.  Do you have
the latest version of the source from CVS?

-- jm

> Hey all.  Found another bug, this time in collision_kernel.cpp.  In
> dGeomSetBody(), in the case where the body is zero, memcpy() is used
> to copy the Body's position and orientation back into the Geom.  The
> size arguments should be sizeof(dVector3) and sizeof(dMatrix3), as
> they are elsewhere, but currently are sizeof(b->pos) and sizeof(b->R).
> Since the pos and R elements of dxBody are in fact both dReal*, this
> results in the position and orientation of a Geom not being properly
> updated when it is detached from a Body.
>
> Old: (collision_kernel.cpp:324)
> <  memcpy (g->pos,g->body->pos,sizeof(g->pos));
> <  memcpy (g->R,g->body->R,sizeof(g->R));
> New:
> >  memcpy (g->pos,g->body->pos,sizeof(dVector3));
> >  memcpy (g->R,g->body->R,sizeof(dMatrix3));
>
>



More information about the ODE mailing list