История изменений

1. Неявный int в качестве возвращаемого значения функции — deprecated и не обязан работать в C99 (gcc по умолчанию выдаёт предупреждение), поэтому лучше заменить тип main на такой: int main(void)
2. Возвращаемое значение swap тебя не интересует, поэтому лучше заменить его на void

По теме — работа с указателями правильная.

1. Неявный int в качестве возвращаемого значения функции — deprecated и не обязан работать в C99 (gcc по умолчанию выдаёт предупреждение), поэтому лучше заменить тип main на такой: int main(void)
2. Возвращаемое значение swap тебя не интересует, поэтому лучше заменить его на void