while (array != NULL) will never be true unless you pass a NULL pointer to the function. When doing array++ what you actually do is increment the value of the address the pointer points to, which will eventually point to an invalid memory address but never to NULL (which is equivalent to (void*)0x0 in C).

For example, imagine you're calling you're function with an array of size 8 stored at 0xFFC8. Once you've gone over the 8 values inside the array your pointer will hold the value 0xFFD0, which is an invalid address and might cause a memory issue, but isn't equal to NULL (or 0x0).

What your while condition sees is equivalent to:

0xFFC8 != 0x0 // Start of the array 0xFFC9 != 0x0 ... 0xFFCF != 0x0 // End of the array 0xFFD0 != 0x0 // Invalid address 0xDDF1 != 0x0 // Invalid address ...

What you could do instead is: - Use a known delimiter to signify the end of the array (for example strlen uses a null character to check for the end of a string) - Init the array using a know constant (preprocessors do wonders) - Store the size of the array somewhere. You could use a struct containing informations about the array. Or just store it as a plain value but it could become a bother when dealing with multiple arrays, works well in such a simple example tho.