Fehler bei scanf_s

[Bruno (Gast)]

Hallo zusammen

habe wie oben gesagt ein Problem mit der scanf_s. Ich möchte die CRC16 
für eine Modbuscom berechnen. Wenn ich die Werte fest vergebe ist das 
auch kein Problem. Nur wenn ich sie über scanf_s einlese, bekomme ich 
fehler in der Berechnung. Bzw ein Array wird nicht mit Daten gefüllt. 
Jemand ne Idee? Anbei der Code

#include <stdio.h>

// Berechnung CRC16
// ----------------
unsigned int CRC16(unsigned int crc, unsigned int data)
{
const unsigned int Poly16=0xA001;  //Generatorpolynom
unsigned int LSB, i;
crc = ((crc^data) | 0xFF00) & (crc | 0x00FF);
for (i=0; i<8; i++)
{
LSB=(crc & 0x0001);
crc=crc/2;
if (LSB)
crc=crc^Poly16;
}
return(crc);
}

int main()
{

/*unsigned*/ char NumData;  
/*unsigned*/ char InputByte; 
/*unsigned*/ char i,data[255]/*={01,43,14,01,00}*/; // Telegrammbuffer
unsigned char Crc_HByte,Crc_LByte; //
unsigned int Crc;

/*Einlesen des Wortes Problemzone*/
printf("Anzahl der Bytes: ");
scanf_s("%d",&NumData);
printf("Anzahl der Bytes = %d\n",NumData);

for (i=0; i<NumData;i++)
{
  printf("Wert Byte:" );
  scanf_s("%d",&InputByte);   
  data[i]= InputByte;
}
printf("Wert data: %d\n %d\n %d\n %d\n %d\n" ,data[0],data[1],data[2],data[3],data[4]);

/*keine Probleme*/
Crc=0xFFFF;
for (i=0; i<NumData; i++) 
{
Crc = CRC16 (Crc, data[i] );
}
Crc_LByte = (Crc & 0x00FF); // Low-Byte bestimmen
Crc_HByte = (Crc & 0xFF00) / 256; // High-Byte bestimmen

printf("LByte %d\n",Crc_LByte);
printf("HByte %d\n",Crc_HByte);
while (1)
{;
;}
}

Alles auf der Basis von Visual c++ 2005.

[Teplotaxl X. (t3plot4x1)]

Falsches Forum?

[Rufus Τ. F. (rufus)]

/*unsigned*/ char NumData;  
...
scanf_s("%d",&NumData);

Das geht schief. "%d" instruiert scanf, einen int-Pointer zu verwenden.

sizeof int != sizeof char

Damit wird also nicht nur der Speicher beschrieben, der für "NumData" 
reserviert ist, sondern auch noch andere Dinge, die in der gleichen 
Gegend abgelegt sind, hier also potentiell noch "InputByte" und/oder 
"i".

Du solltest einen passenderen Formatspezifizierer verwenden, allein, der 
C-Standard sieht keinen für diesen Fall passenden vor.
Also musst Du in einen int einlesen und danach umkopieren.

[Bruno (Gast)]

Habe das ein wenig umdeklariert und jetzt läuft die Sache. Habe einfach 
Integer genommen. Danke.