>[оверквотинг удален]
> new_s = (char*) malloc(n*sizeof(int));
> Тут Вы выделяете, грубо говоря, выделяете память под n int'ов, предполагая работать
> с нею как с char'овскими данными.
> Ошибка заключается в том, что обычно sizeof (int) = 4, когда sizeof
> (char) = 1.
> Т. е. Вы выделяете памяти с четырёхкратным запасом! Для учебного примера это
> не так критично. Но, если Вам надо было обработать строку в
> 10000 символов, Ваш код сожрал бы на это почти 40 кб
> вместо 10. Иногда это бывает весьма критично. Особенно если не освобождать
> память при помощи free ()

Просто для для проверки понимания:
Т.е. должно быть

так примерно int n = strlen(argv[1]);
а потом > new_s = (char*) malloc(n*sizeof(char));
И в конце free(new_s)
Верно ли теперь?