muchas gracias

Hola, estoy tratando de hacer una funcion que copia los caracteres de una string en otra. el funcionamiento es igual que el de la funcion strcpy() pero no puedo usarla. tengo lo siguiente:

char *mi_strcpy(char *cadena1, char *cadena2) {
 
    for(int i=0; i < strlen(cadena2); i++) {
        cadena1[i] = cadena2[i];
    }
    cadena2[strlen(cadena2)]  = '\0';
 
    return cadena1;
}

el problema es que si la cadena2 es menor que la cadena 1, el resto de la longitud de s1 se completa con sus elementos. Ejemplo:
Cadena1 = Hola
Cadena2 = Si
Resultado = Sila