NetworkTable crash in unmodified SimpleRobot template

[IBeMad]

I'm getting a null pointer exeption in

NetworkTable::Initialize()

on the following line:

staticProvider = new NetworkTableProvider(*(mode->CreateNode(ipAddress.c_str(), port, threadManager)));

This crash occurs in unmodified SimpleRobotTemplate code. How does something like this even get through QA, or is there even any QA to begin with? Sure seems like there isn't any...

Earlier on in the call stack, the crash stems from

SimpleRobot::StartCompetition()

when it calls

LiveWindow *lw = LiveWindow::GetInstance();

which creates a new instance of LiveWindow since it's the first time GetInstance() is called, which results in it calling

NetworkTable::GetTable("LiveWindow")

.

When will this be fixed in WPILib, and how can I work around it in the meantime?

[mitchellweb1]

Hi,

Are you sure that this is unmodified. We just ran the template project again and it worked fine. If you are still having issues would it be possible to do a little more debugging and figure out exactly what is null on that line.

--Mitchell

[IBeMad]

Originally Posted by mitchellweb1

Hi,

Are you sure that this is unmodified. We just ran the template project again and it worked fine. If you are still having issues would it be possible to do a little more debugging and figure out exactly what is null on that line.

--Mitchell

Yes, I'm sure that it's umodified. I guess I'll just have to fix the bug in WPILib myself and submit it.

[Alan Anderson]

I suggest that you reinstall the WindRiver update before blaming the library itself.

[IBeMad]

Originally Posted by Alan Anderson

I suggest that you reinstall the WindRiver update before blaming the library itself.

No luck reinstalling everything, however it does work on other computers. Just not mine. Any other ideas?

[mitchellweb1]

If you could run the debugger and determine what value on that line is null then I may be able to try to help a little more.

[Fifthparallel]

staticProvider = new NetworkTableProvider(*(mode->CreateNode(ipAddress.c_str(), port, threadManager)));

Out of curiosity, why is there an asterisk in the argument for NetworkTableProvider? (I always get hazy around dereference operators)

Anyway, a null pointer exception would seem to indicate that something on that line is an uninitialized pointer. How/where did you declare staticProvider?

[nightpool]

Originally Posted by Fifthparallel

staticProvider = new NetworkTableProvider(*(mode->CreateNode(ipAddress.c_str(), port, threadManager)));

Out of curiosity, why is there an asterisk in the argument for NetworkTableProvider? (I always get hazy around dereference operators)

Anyway, a null pointer exception would seem to indicate that something on that line is an uninitialized pointer. How/where did you declare staticProvider?

Well, not looking at the lib or anything, but it seems pretty clear that CreateNode returns a pointer, but the NetworkTableProvider constructor expects a value.

[Fifthparallel]

It looks like it expects an address to a node -- and if createNode returns a pointer, he has an asterisk in there that he doesn't need. I think that's the cause of his null pointer exception -- that asterisk creates an uninitialized pointer.

[mitchellweb1]

createNode will always return a non-null pointer. The NetworkTableProvider constructor takes a reference to the pointer type returned by createNode. This code has run fine on under linux and the crio so it is weird that it is failing here.

staticProvider is declared in NetworkTable.h

[StephenNutt]

Could ipAddress contain the null string? If so, ipAddress.c_str() would return nullptr.