Crash in AbcExport, nHair and Maya 2013

[GoogleCodeExporter]

What steps will reproduce the problem?
1. create a nHair hair nodes.
2. parent or group the nucleus node.
3. export the scene to Alembic.

What is the expected output? What do you see instead?

AbcExport will crash when trying to export the scene.

What version of the product are you using? On what operating system?

Maya 2013, using both the exporter bundled and using a build from the 
repository, latest release.

Please provide any additional information below.

The problem is here:

void AbcWriteJob::setup(double iFrame, MayaTransformWriterPtr iParent, 
GetMembersMap& gmMap)
{
    ...
    ...

    else if (ob.hasFn(MFn::kTransform))
    {
        ...

        unsigned int numChild = mCurDag.childCount();
        for (unsigned int i = 0; i < numChild; ++i)
        {
            mCurDag.push(mCurDag.child(i));
            setup(iFrame, trans, gmMap);
            mCurDag.pop();
        }
    }
    ...
}

MDagPath::push fails when trying to move to a nucleus node in Maya 2013:

// Warning: |hairFront_hairGp MDagPath::push failed, childType = kNucleus //
Maya error: MDagPath::push, error = (kInvalidParameter): Object is incompatible 
with this method

and setup calls itself without moving to the child, entering infinite recursion 
and finally crashing with a stack overflow.

Original issue reported on code.google.com by ramen...@gmail.com on 16 Apr 2014 at 2:38

Attachments:

• test_abc_export_nhair_crash.ma

[GoogleCodeExporter]

Original comment by miller.lucas on 11 Jul 2014 at 4:45

• Changed state: Accepted

[GoogleCodeExporter]

Fix pushed here, it will make it's way into the next release:

http://code.google.com/r/millerlucas-dev/source/detail?r=c22bbf4cab1f20b14222068
763bfbc6b8ef4a54d

Original comment by miller.lucas on 11 Jul 2014 at 4:59

[GoogleCodeExporter]

Thanks!

Original comment by ramen...@gmail.com on 11 Jul 2014 at 5:02

[GoogleCodeExporter]

This has been released in Alembic 1.5.5

Original comment by miller.lucas on 15 Jul 2014 at 6:35

• Changed state: Verified

[GoogleCodeExporter]

Thanks.

Original comment by pbe...@gmail.com on 16 Jul 2014 at 12:57