I am getting a heap corruption on my team’s robot and I would like to verify that my robot’s image is valid. Can someone please compute and post the checksums of your copy of the 2014 image. The problem appeared when I reimaged the cRIO to 2014v52. While running the 2013 image the robot does not crash. When I instantiate a Gyro, WPILib silently corrupts memory. I can demonstrate this by running (https://gist.github.com/electromatter/8995248) in test mode. When I run that program in Test mode and switch to Teleop, the robot crashes within a second. However, when I run the program only in Teleop, the robot does not crash. The jump to 0xEEEEEEEE is not consistent.
related files:
C:\WindRiver\WPILib\cRIO_Images\FRC_2014_v52.zip
C:\WindRiver\WPILib\WPILibC++Source20140101rev3876.zip
C:\WindRiver\vxworks-6.3 arget\lib\WPILib.a
Do you get similar behavior when switching from Test to Autonomous?
I have a few theories, but don’t want to get too in depth without knowing a little bit more. Re-imaging the cRIO shouldn’t hurt it even if it’s not the cause of the error, so I wouldn’t hesitate to try that, though all of your checksums match what I got here.
The corruption or crash could be caused by the user code, by WPILib, or by a corrupted image. As mentioned already, it will not hurt to reimage, though the file system is quite robust.
I suspect that the symptom will remain. I’d suggest debugging the Test code. If DriverStationLCD::GetInstance(); doesn’t return a valid object, the next line will crash. I also don’t know much about the WPILib C++ implementation, but it could be responsible. Independently put in the LCD and gyro code and see which seems responsible.
Since the crash happens when switching from Test to OperatorControl, and both methods use the DriverStationLCD method, I’m wondering if the bug is somewhere in the ~Gyro stack, which would explain that it happens when switching to other methods that don’t contain that object.
If this is the case, it might be mitigated by making your Gyro object a member of the class rather than an object local to a particular method.
pcfens: I moved the Gyro to the stack to make the test more localized. I have tested code with Gyro as a member, it had the same problem.
Greg: GetInstance should throw std::bad_alloc if allocation fails. Otherwise, GetInstance should return a valid object.
The test code that was posted does assume that a valid object pointer is returned. If the LCD class is implemented properly, that isn’t a problem. But if it isn’t, the test code will dereference and crash.
I can’t debug the code, but that is one of the things I’d check.
This is probably something you’ve already checked, but can you confirm that your analog module (NI 9201) is plugged into slot 1 on the cRio?
At some point these were switched on ours, and I found that when I tried to instantiate a gyro, it would throw an exception. I didn’t dive too deep into the details of what the exception was so I don’t know if it matches what you’re getting.