Why does UtPrintx have an infinite loop in it?

[asgibson]

Describe the bug
Any use of UtPrintx results in an infinite printing of the given Memory and Length.

To Reproduce
Steps to reproduce the behavior:

1. Put UtPrintx in a unit test, give it an address and size (from 0 to max uint32)
2. Run tests
3. Enter loop

Expected behavior

1. Print the number of bytes given as Length starting at the given address Memory (with newline at end)
2. Return

Code snips

osal/ut_assert/src/uttools.c

Lines 188 to 207 in b37da18

	void UtPrintx(const void *Memory, uint32 Length)
	{
	uint32 i;
	uint32 j;
	const uint8 *Byte_ptr = Memory;
	char OutputLine[50];
	char * OutPtr;
	i = 0;
	while (1)
	{
	OutPtr = OutputLine;
	for (j = 0; j < 16 && i < Length; j++, i++)
	{
	sprintf(OutPtr, "%02X ", Byte_ptr[i]);
	OutPtr += 3;
	}
	UtPrintf("%s\n", OutputLine);
	}
	}

System observed on:

• RHEL 7.8

Reporter Info
Alan Gibson
NASA/GSFC 587

[skliper]

Guessing the intent was to break the output into lines, but definitely needs a fix.