[OpenOCD-devel] Old -coreid setresult bug?

[Tim N. <ti...@si...>]

A coworker noticed this snipped of code in target.c:
case TCFG_COREID:
if (goi->isconfigure) {
e = Jim_GetOpt_Wide(goi, &w);
if (e != JIM_OK)
return e;
target->coreid = (int32_t)w;
} else {
if (goi->argc != 0)
goto no_params;
}
Jim_SetResult(goi->interp, Jim_NewIntObj(goi->interp,
target->working_area_size));
/* loop for more */
break;

He remarked that the result should probably be target->coreid instead of
target->working_area_size, and I'm inclined to agree. But since I don't
really understand this code, nor know how to really test it, I'm not
comfortable making the change or even declaring it a real bug. But now at
least it's been passed up the chain. If this means something to somebody,
it can be fixed. :-)

Tim