Corregir este error con Win32

Iniciado por Meta, 7 Abril 2018, 12:04 PM

0 Miembros y 1 Visitante están viendo este tema.

Meta

Hola:

He hecho un mini programa que abre y cierra la bandeja del lector. Lo que no entiendo a la hora de ejecutar la aplicación me sale este mensaje de error.
CitarGravedad   Código   Descripción   Proyecto   Archivo   Línea   Estado suprimido
Error   LNK1104   no se puede abrir el archivo 'C:\Users\Usuario\Documents\Visual Studio 2017\Projects\Bandeja_consola_win32_cpp\Debug\Bandeja_consola_win32_cpp.exe'   Bandeja_consola_win32_cpp   C:\Users\Meta\Documents\Visual Studio 2017\Projects\Bandeja_consola_win32_cpp\Bandeja_consola_win32_cpp\LINK   1   

El código es:
Código (cpp) [Seleccionar]

#include "stdafx.h"
#include "Windows.h"
#include "iostream"

using namespace std;

int main()
{
// Título de la ventana.
SetConsoleTitle(L"Consola C++ Win32 2017");

// Variable.
char entrada[] = "\0"; // Guarda A, a, C, y c tipo string que introduces desde la consola.

while (true)
{
// Muestra en pantalla textos.
cout << "Control bandeja del lector: " << endl << endl;
cout << "A - Abrir bandeja." << endl;
cout << "C - Cerrar bandeja." << endl;
cout << "==========================" << endl;

cin >> entrada; // Aquí introduces letras A, a, C, y c.

cout << "\n" << endl;

// Abrir bandeja.
if ((entrada[0] == 'a') || (entrada[0] == 'A'))
{
cout << "Abriendo..." << endl << endl; // Muestra en pantalla textos.
mciSendString(L"set cdaudio door open", nullptr, 0, nullptr);
cout << "Abierto." << endl << endl; // Muestra en pantalla textos.
}
// Cerrar bandeja.
else if ((entrada[0] == 'c') || (entrada[0] == 'C'))
{
cout << "Cerrando..." << endl << endl; // Muestra en pantalla textos.
mciSendString(L"set cdaudio door closed", nullptr, 0, nullptr);
cout << "Cerrado." << endl << endl; // Muestra en pantalla textos.
}
// Si haz pulsado otro caracter distinto de A, C, a, y c aparece
else
{
cout << "Solo pulsar A o C." << endl << endl; // este mensaje.

}
}
return EXIT_SUCCESS;
}


Antes me funcionaba, actualizo Visual studio y pasa esto.

Saludos.
Tutoriales Electrónica y PIC: http://electronica-pic.blogspot.com/

MAFUS

No te lo va a arreglar pero hay un error bastante malo:
char entrada[] = "\0";
Hace que la cadena 'entrada' solo aloje espacio para un (1) único carácter. Cualquier adquisición de una cadena diferente provocará un desbordamiento pues al capturar una cadena de un (1) único carácter estarás guardado dos (2) el que te interesa seguido del carácter nulo ('\0').

Meta

Buenas:

Curiosamente he cerrado Visual Studio, lo volví abrir y ya funciona. No aparece ningún error, el programa funciona al 100 %, puedo abrir y cerrar la bandeja.

En cuanto a lo del:

Código (cpp) [Seleccionar]
char entrada[] = "\0";

No provoca ningún error y funciona de mil maravillas. Aún así te escucho por si acaso.

Saludos.
Tutoriales Electrónica y PIC: http://electronica-pic.blogspot.com/

MAFUS

#3
No es error de tiempo de compilación, pero puede llevarte a errores en tiempo de compilación por buffer overflow. Una cadena bien construida podría permitir la ejecución de código arbitrario (supongo que te suena esto). Por lo normal, si se escribe una cadena lo suficientemente larga el S.O. generará una excepción  porqué se ha intentado escribir en un área no admitida de memoria.

Para ver los efectos del desbordamiento de buffer:
#include <stdio.h>
#include <string.h>

int i;

int main() {
    int a = 0;
    char b[] = "\0";
    int c = 0;

    for(i=0; i<10;++i) {
        strcat(b, "a");
        printf("b = %s\n", b);
        printf("a = %d\nc = %d\n\n", a, c);
    }
}

Meta

Me salta esto.
Gravedad   Código   Descripción   Proyecto   Archivo   Línea   Estado suprimido
Error   C4996   'strcat': This function or variable may be unsafe. Consider using strcat_s instead. To disable deprecation, use _CRT_SECURE_NO_WARNINGS. See online help for details.   EfectosDesbordamientoBuffer_Consola_Win32   c:\users\usuario\documents\visual studio 2017\projects\efectosdesbordamientobuffer_consola_win32\efectosdesbordamientobuffer_consola_win32\efectosdesbordamientobuffer_consola_win32.cpp   16   
Tutoriales Electrónica y PIC: http://electronica-pic.blogspot.com/

MAFUS

Eso es que el compilador encuentra peligrosa strcat y te dice que puede provocar el desbordamiento del búffer, precisamente lo que queremos hacer. Para ello te pide que desactives los warnings

Escribe #define _CRT_SECURE_NO_WARNINGS como primera línea del programa para saltarte esa protección.

Meta

Código (cpp) [Seleccionar]
#include "stdafx.h"
#include <stdio.h>
#include <string.h>
#define _CRT_SECURE_NO_WARNINGS

int i;

int main() {
int a = 0;
char b[] = "\0";
int c = 0;

for (i = 0; i<10; ++i) {
strcat(b, "a");
printf("b = %s\n", b);
printf("a = %d\nc = %d\n\n", a, c);
}
//return 0;
}


CitarGravedad   Código   Descripción   Proyecto   Archivo   Línea   Estado suprimido
Error   C4996   'strcat': This function or variable may be unsafe. Consider using strcat_s instead. To disable deprecation, use _CRT_SECURE_NO_WARNINGS. See online help for details.   EfectosDesbordamientoBuffer_Consola_Win32   c:\users\usuario\documents\visual studio 2017\projects\efectosdesbordamientobuffer_consola_win32\efectosdesbordamientobuffer_consola_win32\efectosdesbordamientobuffer_consola_win32.cpp   17   
Tutoriales Electrónica y PIC: http://electronica-pic.blogspot.com/

MAFUS

Antes de cualquier include. Esto sirve para modificar el comportamiento de las librerías y por tanto debe ir antes de que estás hagan su trabajo.

Meta

Código (cpp) [Seleccionar]
#define _CRT_SECURE_NO_WARNINGS
#include "stdafx.h"
#include <stdio.h>
#include <string.h>


int i;

int main() {
int a = 0;
char b[] = "\0";
int c = 0;

for (i = 0; i<10; ++i) {
strcat(b, "a");
printf("b = %s\n", b);
printf("a = %d\nc = %d\n\n", a, c);
}
//return 0;
}


CitarGravedad   Código   Descripción   Proyecto   Archivo   Línea   Estado suprimido
Advertencia   C4603   '_CRT_SECURE_NO_WARNINGS': no se ha definido la macro o la definición es diferente después del uso del encabezado precompilado   EfectosDesbordamientoBuffer_Consola_Win32   c:\users\usuario\documents\visual studio 2017\projects\efectosdesbordamientobuffer_consola_win32\efectosdesbordamientobuffer_consola_win32\efectosdesbordamientobuffer_consola_win32.cpp   4   

CitarGravedad   Código   Descripción   Proyecto   Archivo   Línea   Estado suprimido
Error   C4996   'strcat': This function or variable may be unsafe. Consider using strcat_s instead. To disable deprecation, use _CRT_SECURE_NO_WARNINGS. See online help for details.   EfectosDesbordamientoBuffer_Consola_Win32   c:\users\usuario\documents\visual studio 2017\projects\efectosdesbordamientobuffer_consola_win32\efectosdesbordamientobuffer_consola_win32\efectosdesbordamientobuffer_consola_win32.cpp   18   
Tutoriales Electrónica y PIC: http://electronica-pic.blogspot.com/

MAFUS

La cabecera precompilada, creo, te está fastidiando. Es verdad que son más rápidas a la hora de compilar pero no son tan dinámicas. Deberás borrar el archivo precompilado para que lo haga de nuevo y acepte el define.