Al llamar desde main a Adquirir_Componente pongo de código 1 y de texto hola, pero al pasar a la instrucción siguiente, la de printf, la salida por pantalla que obtengo es 1, ?·@-.
Claro, es porque estás sobrescribiendo el contenido del parámetro texto
.
Cuando invocas a esta función:
Adquirir_Componente(&codigo,texto);
Le pasas al segundo parámetro la dirección base del arreglo y se supone que la función debe modificar dicho contenido, sin embargo, cuando se ejecute esta instrucción:
texto = (char*)malloc(MAX_CADENA+1);
Estarías sobrescribiendo la dirección de memoria que tenía guardado el puntero texto
. Por lo tanto, reservar memoria con malloc
es inútil, debido a que, cuando la función Adquirir_Componente
termine su ejecución, toda la memoria que consumía dicha rutina es liberada y por ende, se pierde el texto que haya introducido el usuario (ya que no tienes un puntero para poder acceder a dicho bloque que se haya reservado con malloc
) y de paso ocurre una fuga de memoria.
Solución: Elimina ese malloc
.
El código queda así:
void Adquirir_Componente(long *codigo, char *texto)
{
printf("Introduzca el código: \n");
scanf("%ld", codigo);
printf("\nIntroduzca el texto: ");
scanf("%s", texto);
}
Observación:
Posible desbordamiento de búfer en esta línea:
scanf("%s", texto);
Si el usuario ingresa más de 33 caracteres, desbordas el búfer y esto ocasiona que se sobrescriba una zona de memoria.
Solución rápida: Escribe el tamaño del búfer en el especificador de formato:
scanf("%33s", texto);