[Rseding91] [0.15.12] LuaEntity.set_command crashes with empty compound

[folk]

The subject line is not long enough for me to clarify, so here it is:
[0.15.12] LuaEntity.set_command crashes when passed an invalid defines.command.compound

Probably the case with LuaUnitGroup as well, then. I have not tested.

Essentially, all you have to do is;

Code: Select all

		LuaEntity.set_command({
			type = defines.command.compound,
			structure_type = defines.compound_command.return_last,
			commands = {}
		})

And you instantly crash:

Error CrashHandler.cpp:232: Received SIGSEGV
Factorio crashed. Generating symbolized stacktrace, please wait ...
#0 0x000000000073a350 in CrashHandler::writeStackTrace() at /tmp/factorio-rGYOt2/src/Util/CrashHandler.cpp:105
#1 0x000000000073a5d4 in CrashHandler::SignalHandler(int) at /tmp/factorio-rGYOt2/src/Util/CrashHandler.cpp:237
#2 0x00007f5f4a0277f0 in ?? at ??:0
#3 0x00007f5ed80007d8 in ?? at ??:0
1695.600 Error CrashHandler.cpp:106: Map tick at moment of crash: 20652930
1695.600 Error Util.cpp:80: Unexpected error occurred. If you're running the latest version of the game you can help us solve the problem by posting the contents of the log file on the Factorio forums.

[folk]

Still present in 0.15.13

58.315 Error CrashHandler.cpp:232: Received SIGSEGV
Factorio crashed. Generating symbolized stacktrace, please wait ...
#0 0x000000000073fae0 in CrashHandler::writeStackTrace() at /tmp/factorio-ydwwTf/src/Util/CrashHandler.cpp:105
#1 0x000000000073fd64 in CrashHandler::SignalHandler(int) at /tmp/factorio-ydwwTf/src/Util/CrashHandler.cpp:237
#2 0x00007fb3fd2f47f0 in ?? at ??:0
#3 0x00007fb38c000268 in ?? at ??:0
59.752 Error CrashHandler.cpp:106: Map tick at moment of crash: 20648160

[folk]

Tested with LuaUnitGroup.set_command as well, same result.

[folk]

Someone has tagged this bug report with [Rseding97], which I presume is going to have little effect on whatever script you're using internally to assign forum bug reports

But I cant change it, it seems. Or my browser is broken - which is entirely possible, since I build it from source.

[folk]

I'm guessing I'll have to keep bumping this until some moderator notices the problem in the post title.

[Rseding91]

I'm guessing I'll have to keep bumping this until some moderator notices the problem in the post title.

91/97 it's only 6 years difference