Rockbox

I'm not sure which category to choose for this bug.

There is a bug in "void block_data(char *stg, ULONG val)" in the file "utils/disassembler/arm/disasm_arm.c" which leads to a segmentation fault in some cases, probably if you feed the function with illegal code.

void block_data(char *stg, ULONG val)
{
char lst[80];
int i;

strcpy(lst, "{");
for(i=0; i<16; i++)
if(val & (1<<i))
sprintf(lst+strlen(lst), "%s, ", regs[i]);
strcpy(lst+strlen(lst)-2, "}");

This strcpy-function is illegal if no reg is found in the loop before, because the string "}" is then copied to the position -1, which corrupts other variables. There should be an additional if-clause like this:

if (strlen(lst)>2)
strcpy(lst+strlen(lst)-2, "}");
else
strcpy(lst+strlen(lst),"}");

I think a patch file is not necessary, as it is very easy to patch witch copy-and-paste from this message.