[PATCH] gas/config/tc-tic4x.c: Let insn name full use its space and always zero terminated

Alan Modra amodra@gmail.com
Mon Nov 24 03:28:00 GMT 2014


On Mon, Nov 17, 2014 at 11:47:08PM +0800, Chen Gang wrote:
> strncpy() can not be sure of insn name must be zero terminated, so need
> set it explicitly.

OK.

> At present, insn name is only used in tic4x_insn_check() which only test
> insn name within 9 chars explicitly. So it is harmless to use full space
> of insn name for strncpy().

Not OK.  You missed noticing that md_assemble is called recursively
for parallel instructions, and does strcat (insn->name, "_").  So
space must be reserved.

-- 
Alan Modra
Australia Development Lab, IBM



More information about the Binutils mailing list