Cambié algunas cosas, la función la puse arriba del `main`, hice que devuelva un **entero** (`int`) en vez un `void`, y que no reciba el segundo argumento. También hice que no sea necesario especificar la longitud del array, ahora lo detecta con `sizeof`. Además el main debe devolver un entero de valor `0` para verificar que se ejecutó correctamente. Hacer esto, `int ventasSemanales[semanas]` está mal, no es la manera de recibir el argumento `semanas`, y además no es necesario, ya es posible conocer la longitud de tal array. Código final: int semanaMenorVent(int ventasSemanales[],int numeroMagico) { int semanaMenor = -1; int numeroMenor = ventasSemanales[0]; int i; for (i = 0; ventasSemanales[i]!=numeroMagico; i++) { if (ventasSemanales[i] < numeroMenor) { numeroMenor = ventasSemanales[i]; semanaMenor = i; } printf("%i\n", semanaMenor); } printf("longitudArray=%i\n",i); return semanaMenor; } int main() { int numeroMagico=0; int ventasSemanales[]={9,5,6,9,2,4,3,9,3,3,numeroMagico}; int menor = semanaMenorVent(ventasSemanales,numeroMagico); printf("la semana de menores ventas fue la numero: %d\n", menor+1); return 0; }