Eso quiere decir que el programa debe funcionar, nada mas.

Pásate por esta web para conocer la legalidad con respecto a este tema: http://www.legaltoday.com/practica-juridica/penal/penal/la-comprobacion-de-antecedentes-penales-por-parte-de-las-empresas

Quédate con tu profesor:

int mayor(int n1, int n2, int n3, int n4, int n5) {
    int mayor = n1;
    for(int *p=&n2; p<=&n5; ++p)
        if(*p > mayor)
            mayor = *p;
    return mayor;
}

Un puntero. Es lo que tiene ocultar punteros, que se llega a confundir a la gente. Mala práctica.

A ver, es un trabajo de clase. Debes inventarte esa base de datos.

Aprovecha el bucle loop. Cambia los whiles por if.
Si hay un dato en puerto serie lo lees en va.
Si va vale 0 imprime "OFF"
Si va vale 1 imprime "Modo 1"
Si va vale 2 imprime "Modo 2"
Esperar 400ms

Prueba esto.

Sigue el programa:
Cuándo hay un dato entras en el if.
Si has pulsado 1 va valdrá 1.
Entonces entras en el primer while donde la condición es que se repita si va vale 1. Cómo no hay cambio al terminar el bucle va seguirá valiendo 1 y repetirá el bucle de nuevo. Y esto será infinito.
Cambia el valor de va antes de salir del bucle a 0, por ejemplo.

scanf hace el mismo comportamiento que scanf, pero en vez de la fuente ser stdin, es una cadena dada.

strtok consigue toques de una cadena dada. Se le pasa una segunda cadena, que puede cambiar en cada llamada, que indicará los delimitadores. El toquen obtenido será una cadena desde la posición de la última llamada (o es inició si es la primera) hasta encontrar uno de los caracteres delimitadores dados. El toquen es devuelto uno a cada llamada y debe ser recogido por un puntero a carácter. La cadena original se ve afectada. Si la cadena original es una cadena constante strtok puede tener efectos extraños.

En C++?
Si el & está en el prototipo de la función ese argumento será una referencia: la función puede modificar el valor en la función llamante. Sin el & el argumento es una copia, es decir, la función no puede modificar el valor en la función llamante.

La función acepta un único altelta, en cambio tu le pasas un array entero.

La única forma que se me ocurre es que mientras desconozcas el sistema operativo del cual se trata sean punteros a void. Ya que los punteros siempre tienen el mismo tamaño no habrá quejas por parte del compilador en ningún caso. Una vez conozca el tipo de sistema operativo que se trata puedes generar de forma dinámica el tipo de estructura que requieras mediante malloc. Lo único que tendrás que tener en cuenta es que debes liberarlo cuando cierres el programa en la función de limpiar recursos, en el main o con atexit, según hagas la limpieza.

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
int main() {
    void* pointer;
 
    pointer = malloc(sizeof(int));
    *(int*)pointer = 3;
    printf("%d\n", *(int*)pointer);
    free(pointer);
 
    pointer = malloc(sizeof(char)*(strlen("Una cadena")+1));
    strcpy((char*)pointer, "Una cadena");
    printf("%s\n", (char*)pointer);
    free(pointer);
}

Sí, el código es un poco feo, pero en C es la forma de tener un tipo de objeto genérico.