for(int i = 0; i < n; i++)
you have the same loop 2 times, first to allocate memory and then to loop through the array, you can allocate the memory in that second loop and then you won't need the first one