Skip to main content
se añadieron 416 caracteres en el cuerpo
Origen Enlace
MrDave1999
  • 7.6k
  • 1
  • 11
  • 22

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);

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);
}

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);
Origen Enlace
MrDave1999
  • 7.6k
  • 1
  • 11
  • 22

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);
}