[SOLUCIONADO] ¿Por que se produce este buffer overflow?

[Pedritogj11]

Muy buenas, estoy empezando a programar asi que soy un poco novatillo en esto, la verdad es que para tener 15 años creo que esta bien lo que conozco sobre programación. Estoy haciendo un pequeño "minijuego" en C y no se porqué se crashea cada vez que elijo un opción en el menú, no consigo encontrar el error :(. Aquí dejo el código. Gracias de antemano.

[URL="https://gyazo.com/7eeb98bd351b772cdb267146b0b7bed6"]https://gyazo.com/7eeb98bd351b772cdb267146b0b7bed6[/URL]

Código:

/* Programa realizado por @PedroGabaldon el xx-xx-xxxx. Consiste en un pequeño juego. Enjoy it! :) */
#include <stdio.h>
#include <conio.h>

int main (void)
{
	char clave = 'P';
	int opcion;
	printf("##############################################\n#                                            #\n#                                            #\n#                                            #\n#              Mini-juego en C               #\n#              Pedrito - 2015                #\n                                             #\n##############################################");
	printf("\n\n________Menu________");
	printf("\n1. Abrir la caja de Pedrito.");
	printf("\n2. Descifrar clave.");
	printf("\n3. Salir.");
	printf("\n\nSeleccione una opcion:");
	scanf("%s", opcion);
	
	switch (opcion)
	{
		case 1: printf("\nEscribe el codigo:");
				scanf("%s, clave");
				if (clave == 'P')
				{
					printf("Enhorabuena, has abierto la caja. :)");
			    }
			    break;	
	}
	printf("Pulse intro para salir:");
	fflush(stdin);
	getchar();
	
	return 0;
}

[eferion]

clave únicamente puede almacenar un char... no una cadena de caracteres.

Para almacenar cadenas de caracteres tienes que usar un array:

Código C:

char clave[200]; // Cambia 200 por la longitud máxima a almacenar más 1
 

Por otro lado, el contenido de una cadena de caracteres no se puede comprobar con el operador de igualdad porque es una operación compleja... para eso existe la función strcmp, que podrás encontrar en la librería string.h

Código C:

if( strcmp( clave, "P") == 0 ) // 0 representa una igualdad
{
}
 

Si, en cambio, quieres leer únicamente un caracter, cambia el scanf por este otro:

Código C:

scanf("%c",&clave);
 

%c indica que vas a leer únicamente un carácter.

El & hay que ponerlo porque la función espera un puntero... y clave, declarado tal que char clave; no lo es :)

Un saludo.

[Pedritogj11]

Iniciado por eferion

clave únicamente puede almacenar un char... no una cadena de caracteres.

Para almacenar cadenas de caracteres tienes que usar un array:

Código C:

char clave[200]; // Cambia 200 por la longitud máxima a almacenar más 1
 

Por otro lado, el contenido de una cadena de caracteres no se puede comprobar con el operador de igualdad porque es una operación compleja... para eso existe la función strcmp, que podrás encontrar en la librería string.h

Código C:

if( strcmp( clave, "P") == 0 ) // 0 representa una igualdad
{
}
 

Si, en cambio, quieres leer únicamente un caracter, cambia el scanf por este otro:

Código C:

scanf("%c",&clave);
 

%c indica que vas a leer únicamente un carácter.

El & hay que ponerlo porque la función espera un puntero... y clave, declarado tal que char clave; no lo es :)

Un saludo.

Vale muchas gracias por la ayuda. Me gustaria que me dijeras porque en el array hay que indicar la longitud maxima MAS 1? Por que uno mas? Y la biblioteca stdio.h, ¿permite la programacion orientada a objetos o es solo para C y no para C++?
Un saludo :)

[eferion]

Iniciado por Pedritogj11

Me gustaria que me dijeras porque en el array hay que indicar la longitud maxima MAS 1? Por que uno mas?

Porque al que en su momento le tocó decidir cómo gestionar las cadenas de caracteres se le ocurrió que éstas iban a terminar con el caracter nulo '\0'. Luego si quieres almacenar "HOLA" necesitas una variable de, al menos, 5 bytes (4 para la palabra y 1 para el terminador).

Iniciado por Pedritogj11

Y la biblioteca stdio.h, ¿permite la programacion orientada a objetos o es solo para C y no para C++?

stdio es una biblioteca con funciones de entrada salida básicas... puedes llamar a dichas funciones desde C y desde C++, ya que ambos lenguajes tienen disponible esta librería. Te daría más detalles, pero es mejor que asientes conocimientos antes de darte más información de la que vas a ser capaz de entender.

Un saludo.