XMODEM送信プログラムのパケット生成がわからない

Question

前提・実現したいこと

学校の課題で困っているので助けてください。
XMODEMの送信プログラムを作成する課題で、パケット生成で躓いています。235行目でdataSize = ReadDataFromFile(fp, &packet[3], DATA_SIZE, 10);とあり。これでファイルから最大DATA_SIZEバイト読み込むとするのであればpacket[3]=dataSizeとすればpacket[3],packet[4],packet[5]にファイルから読み込んだデータが代入されるのでは無いかと思いました。しかし、実際に通信させてみると受信側の端末でなにも変化が起きず失敗に終わりました。
もう何が何だかわからず、心が折れそうです。このサイトが最後の頼みの綱です。どうかご回答お願いします。
厳しいお言葉は投げかけないでください。勉強不足なのは重々承知の上です。

発生している問題・エラーメッセージ

expected identifier or ‘(’ before ‘}’ token
}

該当のソースコード

#include <sys/types.h>
#include <sys/stat.h>
#include <termios.h>
#include <sys/file.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <fcntl.h>
#include <unistd.h>
#include <signal.h>
#include <sys/time.h>

#define SOH 0x01
#define EOT 0x04
#define ACK 0x06
#define NAK 0x15
#define CAN 0x18
#define SUB 0x1a

#define DATA_SIZE 3
#define TIMEOUT 10

enum States {
NAKReceived, /NAK受信/
SamePacket10Sent, /同一パケットを10回送信/
ACKReceived, /ACK受信/
CAN2Received, /2連続CAN受信/
AllSent, /全パケット送信/
Continue, /ループ周回/
TimeOut /タイムアウト/
};

unsigned char CheckSum(unsigned char *p, size_t size);

/*** 時刻測定:オマジナイ ***/
double GettimeofdaySec();

int Receive(int fd, unsigned char *receiveData, int dataSize, int time);

int Send(int fd, unsigned char sendData, int time);

int ReadDataFromFile(FILE *fp, unsigned char *characterPosition, int readSize, int number);

void printState(enum States);

/*** 初期設定 オマジナイ***/
int Initialize(int *pfd,
FILE **pfp,
char *filename,
double *startTime,
struct termios *pold_tio,
struct termios *pnew_tio);

enum States InitReceive(int fd);

enum States MidState(int fd);

void EndByCAN2Received(int fd, FILE *fp, struct termios *pold_tio);

int EndByTimeOut(int fd, FILE *fp, struct termios *pold_tio);

int EndByAllSent(int *pfd,
FILE *fp,
double startTime,
int packetNumber,
struct termios *pold_tio);

void EndBySamePacket10Sent(int fd, FILE *fp, struct termios *pold_tio);

int End(int fd, FILE *fp, struct termios *pold_tio);

void PrintPacketInfo(double startTime, int packetNumber);

int main(int argc, char **argv) {
int fd;/デバイス識別のためのファイルデスクリプタ/
struct termios old_tio, new_tio;/シリアルデバイス設定/
FILE *fp;/ファイル読込のためのファイル型へのポインタ/
char *filename = argv[1];/送信ファイル名/
int dataSize;/ファイルから読み込んだバイト数/

unsigned char packet[DATA_SIZE+4];/パケット/
unsigned char packetIndex = 0;/パケット番号(1バイト)/
int packetNumber = 0;/パケット番号(int型)/
int SamePacketSentTimes;/同一パケット送信回数/

double startTime;/通信開始時刻(通信時間計測に使用)/

int i;

enum States state;/状態/

/初期設定/
if(Initialize(&fd,
&fp,
filename,
&startTime,
&old_tio,
&new_tio)){
printf("Init Error\n");
exit(1);
}

/初めのNAK受信を試みて状態を獲得する。/
state=InitReceive(fd);
if(/ここに何か書く/state==NAKReceived){/NAK受信したときの処理/
while(1){/ファイルから読込、パケット送信とその応答処理/
/ファイルから最大DATA_SIZEバイト読込;/
dataSize = ReadDataFromFile(fp, &packet[3], DATA_SIZE, 10);
if(dataSize<0){
perror("file read failed\n");
return -1;
}

  if(dataSize==0){
state=AllSent;
break;
  }
  else  if(dataSize==1){
   	dataSize=dataSize+2*SUB;
  }
  else if(dataSize==2){
dataSize=dataSize+SUB;
  }
 
  packetIndex++;
  packetNumber++;
  /**パケット生成**/
  /***ここに何か書く***/
  packet[0]=SOH;
  packet[1]=packetIndex;
  packet[2]=0xff-packetIndex;
  packet[3]=dataSize;
  packet[6]=CheckSum(packet+3,DATA_SIZE);
 
  /*同一パケット送信回数リセット*/
  /***ここに何か書く***/
  SamePacketSentTimes=0;
  
  while(1){
if(SamePacketSentTimes>10){
  state=SamePacket10Sent;
break;
}
  
Send(fd,dataSize,60);


SamePacketSentTimes++;

    state=MidState(fd);

if(state!=NAKReceived && state!=TimeOut){
  break;
}
  }/*while(1)
  if(state!=ACKReceived){
break;
  }
}/*while(1)ファイルから読込、パケット送信とその応答処理*/

}/if(state==NAK受信)/
/**状態に応じて、関数 */
/*EndByCAN2Received(fd, fp, &old_tio)); */
/*EndByTimeOut(fd, fp, &old_tio); */
/EndByAllSent(&fd, fp, startTime, packetNumber, &old_tio);/
/*EndBySamePacket10Sent(fd, fp, &old_tio); */
/*を呼び出すようにswitch文を書く **/
/ここに何か書く/
switch(state){
case CAN2Received:
EndByCAN2Received(fd, fp, &old_tio);
break;

case TimeOut:
  EndByTimeOut(fd, fp, &old_tio);
  break;

case AllSent:
  EndByAllSent(&fd, fp, startTime, packetNumber, &old_tio);
  break;

case SamePacket10Sent:
  EndBySamePacket10Sent(fd, fp, &old_tio);
  break;
}

return 0;
}
}

/初期設定 オマジナイ/
int Initialize(int *pfd,
FILE **pfp,
char *filename,
double *startTime,
struct termios *pold_tio,
struct termios *pnew_tio){
int state;
int full_data_size;
int blockSize;

/* デバイスのオープン */

/* 読み書き用 (O_RDWR) で開く */
if((*pfd = open("/dev/tnt0", O_RDWR)) < 0) {
perror("open of /dev/tnt0 failed");
return 1;
}

/* 現在のシリアルポートの設定を退避させる */
if((state = tcgetattr(pfd, pold_tio)) < 0) {
perror("tcgetatrr failed");
return 1;
}
/ 新しいポートの設定の構造体をクリア */
memset(pnew_tio, 0, sizeof(*pnew_tio));

/* パラメータの設定 */

/* フレームエラーおよびパリティエラーを無視する /
pnew_tio->c_iflag = IGNPAR;
/ rawモード(入力したキーをそのまま前処理せずに渡す /
pnew_tio->c_oflag = 0;
/ 非カノニカル入力(read毎に決まった数の文字を扱う) /
pnew_tio->c_lflag = 0;
/ ボーレート(B9600) :9600Bps
文字サイズ(CS8) : 8bit
受信を有効にする(CREAD)
モデムの状態信号を無視する(CLOCAL) */
pnew_tio->c_cflag = ( B9600 | CS8 | CREAD | CLOCAL);

/* ノンブロッキングモード /
pnew_tio->c_cc[VMIN] = 0;
/ キャラクタ間タイマを1[sec]に設定 */
pnew_tio->c_cc[VTIME] = TIMEOUT;

/* ポートのクリア */
if((state = tcflush(*pfd, TCIFLUSH)) < 0 ) {
perror("tcflush failed");
return 1;

}

/* 以上の設定を反映させるために"TCSANOW"を行なう */
if(tcsetattr(*pfd, TCSANOW, pnew_tio) < 0) {
perror("tscsetattr (TCSANOW) failed");
return 1;
}

/* 送信ファイルのオープン */
if((*pfp = fopen(filename, "rb")) == NULL) {
perror("file open failed");
return 1;
}
fseek(*pfp, 0, SEEK_END);
full_data_size = ftell(*pfp);
blockSize = (full_data_size + (DATA_SIZE - 1)) / DATA_SIZE;
fseek(*pfp, 0, SEEK_SET);
printf("Sending %s, %d blocks: Give your local XMODEM receive command now.\n", filename, blockSize);
*startTime = GettimeofdaySec();

return 0;
}

enum States InitReceive(int fd){
enum States state;
unsigned char c;

while(1){
if(!Receive(fd, &c, 1, 60)){

Comments

[y_waiwai]

このままではコードが読みづらいので、質問を編集し、<code>ボタンを押し、出てくる’’’の枠の中にコードを貼り付けてください

[y_waiwai]

んで、提示されてるエラーメッセージはコンパイルエラーと思いますが、
これがでていたら実行ファイルができないと思うんですが、どうやって実行させてるんでしょうか

Answer 1

xmodemなんてもうすっかり忘れましたけど...

235行目で

少なくとも貼り付けてあるソースでは #include <sys/types.h>を1行目として
235行目は
if(tcsetattr(*pfd, TCSANOW, pnew_tio) < 0) {
です。その近辺に dataSize = ReadDataFromFile(... という内容のものはありません。
質問の文章と対応していないソースコードということでしょうか。それでは何も言えません。

packet[3]=dataSizeとすれば

packet[3]の内容はプログラムの動作に関係ないのでは。で、ありがちなパターンから言えば、datasizeはファイルから読みだしたデータサイズが入っているのではないのですか?

packet[3],packet[4],packet[5]にファイルから読み込んだデータが代入されるのでは無いかと思いました。

思うのではなく、ReadDataFromFile関数の定義を確認しましょう。ReadDataFromFile関数が示されていないので、こちらではなにも言えません。

しかし、実際に通信させてみると受信側の端末でなにも変化が起きず失敗に終わりました。

基本的な通信は出来ているのですか? ファイルの読み出しとは別のところに原因がある気配が濃厚に思えます。