[Grbl_Esp32 Issue#456] Stack smashing protect failure!

#1 – bdring 于 2020-06-27

Could you please post the gcode that causes the problem. I can test it later today. Just a fewest lines needed to repeat the problem.

Have to tried manually sending the gcode via serial monitor? Many gcode senders have trouble with more than 3 axes and it would be helpful to see if that is part of the problem.

#2 – Claude54 于 2020-06-30

Sorry for the late reply but I was not available this weekend. I am sending you a piece of GCode which systematically makes GRBL
crash. I noticed that GRBL crashed when A arrived around -10000.

#3 – Claude54 于 2020-06-30

> I have tried with other files and always the same, GRBL crashed when A
> arrived around -10000
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/bdring/Grbl_Esp32/issues/456#issuecomment-652058434>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/AABOJ3JSO4YOU57KY4VXMNLRZJLBNANCNFSM4OJ5ZAQQ>
> .
>

#5 – bdring 于 2020-06-30

I was able to repeat the problem. I think it is due to such a large number not being properly planned for in function…

“
// formats axis values into a string and returns that string in rpt
static void reportutilaxisvalues(float axisvalue, char rpt) {
uint8_t idx;
char axisVal[10];
float unit_conv = 1.0; // unit conversion multiplier..default is mm
rpt[0] = '\0';
if (report_inches->get())
unitconv = 1.0 / MMPER_INCH;
for (idx = 0; idx < N_AXIS; idx++) { if (report_inches->get())
sprintf(axisVal, "%4.4f", axisvalue[idx] * unitconv); // Report inches to 4 decimals
else
sprintf(axisVal, "%4.3f", axisvalue[idx] * unitconv); // Report mm to 3 decimals
strcat(rpt, axisVal);
if (idx < (N_AXIS - 1)) strcat(rpt, ","); } } `

char axisVal[10]; does not have wnough room to store it.

This is in report.cpp. You can try changing it to char axisVal[20];`. It worked for me. I need to look for other areas before updating the code.

#6 - Claude54 于 2020-06-30

Yessss..
I have changed the size of axisVal array and it works now.
Thank you very much.