A mi código agregale un (char *) antes del malloc, algo así:

(char *)malloc(....)

El compilador no debería de pedirlo, es extraño.

¡Buenas!

Ya te han dado varias soluciones, voy a intentar darte una explicacion de porque no funcionaba tu codigo.

No se si sabes lo que es la pila de un programa. Es una parte de la memoria en la que se van acumulando datos de llamadas a funciones, alli se ponen, entre otras cosas, los parametros que pasas a las funciones y el valor de retorno. Una vez que se sale de la funcion, se "recupera" la memoria que se ha utilizado, incluida la que utilizan las variables locales, por lo tanto, como ya han dicho en alguno de las respuestas, los valores que contienen se pueden perder. Esto quiere decir que si devuelves un puntero a una variable que se encuentre dentro de una funcion, lo que estas devolviendo, en principio, es un puntero a alguna parte de la pila, que esta modificandose de forma continua. De ahi la "basura" que encuentras.

Tienes varias formas de solucionar el problema (que yo sepa 4 evidentes... ¿Alguien da mas?), dos muy parecidas, una de ellas ya te la han dado.

Puedes asignar memoria a un puntero local, esto no se perdera, ya que estaras devolviendo un valor a una posicion de memoria no de la pila, sino del heap.

Otra opcion es pasar a la funcion una cadena desde la funcion llamadora.

Si en principio no sabes cuanto va a ocupar el vector (porque por ejemplo ese dato lo obtienes dentro de la funcion) tambien puedes pasar en la lista de parametros un puntero a un puntero y actuar como en el primer caso, asignando la memoria de forma dinamica.

Y para terminar, puedes declarar la variable dentro de la propia funcion como static, de esta forma, cuando salgas de la funcion te aseguraras de que mantiene la informacion, pero esta cambiara de una llamada a otra, por lo que despues de cada llamada tendras que guardar o procesar inmediatamente el dato para que en posteriores llamadas no pierdas informacion que te pueda interesar.

¡Saludos!