Segfault with inaccessible custom node

[dbr]

If I create a custom node, and register it like this:

class TestInput(Gaffer.ComputeNode):
   pass
IECore.registerRunTimeTyped(TestInput, typeName = "TestInput")

import GafferUI
nodeMenu = GafferUI.NodeMenu.acquire(application)
nodeMenu.append("/Example/TestInput", lambda: TestInput(), searchText = "TestInput")

(just started looking into Gaffer and shoved it in ~/gaffer/startup/gui/example.py as the most obvious place to get code to run)

This works fine when I create the node by calling TestInput() directly (like by clicking the menu item)

However if the node is created "indirectly" (either by saving the node into a .gfr script and loading it, or copypasting the node), the GUI will segfault:

*  1 0x0000003d8200f70f [libpthread.so.0 ] [funlockfile.c : 0]
>> 2 0x00007fbd63a8befc [libGafferBindings.so ] (anonymous namespace) [ScriptNodeBinding.cpp : 0]
*  3 0x00007fbd63a8befc [libGafferBindings.so ] [ScriptNodeBinding.cpp : 0]
*  4 0x00007fbd6401cab5 [libGaffer.so ] Gaffer::ScriptNode::execute(std::string const&, Gaffer::Node*, bool)
*  5 0x00007fbd6401cd70 [libGaffer.so ] Gaffer::ScriptNode::paste(Gaffer::Node*, bool)
[...snip...]

Fix was for me to be less lazy and define the module in a importable Python module, and specify the full path in typeName, e.g define it in blah.py somewhere and do:

IECore.registerRunTimeTyped(TestInput, typeName = "blah::TestInput")

This fixes the problem, but I'd expect a better error in the first case

[dbr]

Oh, this is with the prebuilt version 0.40.0 binaries on Linux

[johnhaddon]

Thanks for the report, and glad you've got things working. I've fixed the crash in PR #2320 so you'll now get a slightly more informative error - NameError: name 'TestInput' is not defined.

[dbr]

Thanks! That error looks far more descriptive \o/