nella malloc nel sizeof toglierei il char* e metterei char

la scanf vuole un puntatore e s[i] è già un puntatore no? non credo serva l'&