Ya que mencionas que sólo puedes usar `<stdlib.h>` y punteros te recomendaría que no usaras [`scanf`][1] y usaras [`fgets`][2] 

[`fgets`][2] es una mejor manera de leer datos que scanf.

Su firma es la siguiente:
```C
char * fgets(char *s, int size, FILE *stream);
```

Como dice la función lee como máximo "*size* - 1" caracteres o hasta un salto de línea (**\n**) o un **EOF**.

Esto evita que guardes más elementos de los que tu arreglo en realidad es capaz de leer, cosa que sucede con `scanf`, donde se sobrescribe memoria si es que tu arreglo no tiene la capacidad de almacenamiento suficiente.

Solamente que deberías especificar como archivo que se leerá de `stdin`

Algo así:

```C
char cadena[20];
fgets(cadena, 20, stdin);
```
De una vez te menciono que no es necesario hacer esta parte de tu código puesto que realmente estás duplicando el valor que guarda cadena (un puntero a la primera posición de memoria de tu arreglo)

```C
char *p;
p=cadena;
```

  [1]: https://linux.die.net/man/3/scanf
  [2]: https://linux.die.net/man/3/fgets