Infinite loop false alarm in unit test (easy fix) - static analysis warning

[skliper]

Is your feature request related to a problem? Please describe.
Static analysis warning on i being int8. Can be avoided if defined as size_t.

cFE/fsw/cfe-core/unit-test/ut_support.c

Lines 82 to 90 in c708d64

	int8 i;
	/* Copy the application name for later use */
	strncpy(UT_subsys, subsys, sizeof(UT_subsys)-1);
	UT_subsys[sizeof(UT_subsys)-1] = 0;
	snprintf(UT_appname, 80, "ut_cfe_%s", subsys);
	/* Convert to upper case */
	for (i = 0; i < strlen(UT_subsys); ++i)

Based on logic strlen(UT_subsys) will never be bigger than int8 (unless someone redefines UT_subsys as something unreasonable) so this is a false alarm but easy to fix and better pattern to follow.

Describe the solution you'd like
Change to size_t

Describe alternatives you've considered
None

Additional context
Please wait until #972 is resolved.

Requester Info
Jacob Hageman - NASA/GSFC

[himanshu007-creator]

I would like to work on this issue.

[himanshu007-creator]

@skliper Hi, as #972 is closed, so i assume it is resolved. So should i go for a PR for this issue?

[skliper]

@himanshu007-creator Yes please! You'll need to follow the PR template instructions to fill out and deliver a CLA before the PR can be merged. Thanks!