質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.52%

  • Linux

    4365questions

    Linuxは、Unixをベースにして開発されたオペレーティングシステムです。日本では「リナックス」と呼ばれています。 主にWebサーバやDNSサーバ、イントラネットなどのサーバ用OSとして利用されています。 上位500のスーパーコンピュータの90%以上はLinuxを使用しています。 携帯端末用のプラットフォームAndroidは、Linuxカーネル上に構築されています。

  • C++

    4295questions

    C++はC言語をもとにしてつくられた最もよく使われるマルチパラダイムプログラミング言語の1つです。オブジェクト指向、ジェネリック、命令型など広く対応しており、多目的に使用されています。

  • ポインタ

    130questions

    ポインタはアドレスを用いてメモリに格納された値を"参照する"変数です。

  • ファイルI/O

    42questions

    ファイルI/Oは、コンピューターにおけるファイルの入出力です。これは生成/削除やファイルを読み込んだり、出力をファイルに書き込むようなディレクトリやファイルの運用を含みます。

ファイル転送のプロトコルをC++で実装しているが、チェックサムのCRC16が合わない?どうして?

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 1,449

NEWBIEEBIEE

score 39

ファイルの転送プロトコルのxmodemをLinux上で実装しているのですが、
ファイルを小分けにしたデータとそのチェックサムを送信して、
別の端末で受け取っています。しかし、このデータをその受け取った端末で
チェックサムを計算して、送られたチェックサムと照合すると一致しません。
写真では9404がデータと一緒に送られるのですが、データから再計算して出したチェックサムには7541とでます。
全く良くわからないのですが,これはファイルの受信のポインタ処理に問題があるのでしょうか、
ファイルを受け取る際にread関数で、配列をwhileで回すことによって、1024バイトのデータを取得しています。
そこでのポインタ処理に問題があるのでしょうか。バイナリとポインタは慣れていないので感覚がつかめません。

それとも、別の問題があるのでしょうか。

実行とエラー

送信側の処理(xymodem_send)

chunk.crc = swap16(crc16(chunk.payload, sizeof(chunk.payload)));

受け取り側の処理 (xmodem_receive)

    uint8_t recData[1024]; // receive data from chunk
    uint16_t recChksum;
    uint16_t recCRC;

    // recData is data payload
    ret = read(serial_fd, recData, sizeof(recData));

    printf("Data buffer is %c\n", &recData);
    fflush(stdout);

    // processing  up to 1024bytes
    if (ret != sizeof(recData)) {
        rec_chunk_data += ret;
        while(rec_chunk_data < sizeof(recData)){
            ret = read(serial_fd, recData + (sizeof(uint8_t) * rec_chunk_data), (sizeof(recData) - rec_chunk_data));
            rec_chunk_data += ret;
                    printf("ret is proceeding: %d\n", ret);
                    fflush(stdout);
        }
    }

    ret = read(serial_fd, &recChksum, sizeof(recChksum));

    printf("Check sum is %d\n", recChksum);
    fflush(stdout);
    // Calculating checksum from data payload
    recCRC = swap16(crc16(recData, sizeof(recData)));
    // data integrity check
    if(recChksum != recCRC){
        printf("Check sum is %d and %d\n", recChksum, swap16(crc16(recData, sizeof(recData))));
        perror("swap16");
        return -errno;
    }

関係のある処理(コード)

#include <stdio.h>
#include <stdlib.h>
#include <stdint.h>
#include <string.h>
#include <errno.h>
#include <unistd.h>
#include <fcntl.h>
#include <termios.h>
#include <sys/types.h>
#include <sys/stat.h>
#include <sys/mman.h>
#include <string>

#define X_STX 0x02
#define X_ACK 0x06
#define X_NAK 0x15
#define X_EOF 0x04

 #define MAX_RETRY (9)

#define min(a, b)       ((a) < (b) ? (a) : (b))

struct xmodem_chunk {
        uint8_t start;
        uint8_t block;
        uint8_t block_neg;
        uint8_t payload[1024];
        uint16_t crc;
} __attribute__((packed));

#define CRC_POLY 0x1021
static uint16_t crc_update(uint16_t crc_in, int incr)
{
    uint16_t xor1 = crc_in >> 15;
    uint16_t out1 = crc_in << 1;

    if(incr)
        out1++;

    if(xor1)
        out1 ^= CRC_POLY; // xor 0b1000000100001

    return out1;
}


static uint16_t crc16(const uint8_t *data, uint16_t size)
{
        uint16_t crc, i;

        for (crc = 0; size > 0; size--, data++)
                for (i = 0x80; i; i >>= 1)
                        crc = crc_update(crc, *data & i);

        for (i = 0; i < 16; i++)
                crc = crc_update(crc, 0);

        return crc;
}

static uint16_t swap16(uint16_t in)
{
        return (in >> 8) | ((in & 0xff) << 8);
}

enum {
        PROTOCOL_XMODEM,
        PROTOCOL_YMODEM,
};
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

+3

こんにちは。

sizeof(recData)とsizeof(chunk.payload)は同じ値になるでしょうか?
一般的には、このような計算をする時、受信側は受信したバイト数を使います。

もし同じ値だった場合、payloadはいっぱいになってますか? もし、送信したバイト数より大きかった場合、空き部分に入っている値は、受信側の同様の空き部分に入っている値と一致してますか?

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/01/14 15:50

    なるほど、確かにその通りでした。ちょっと確認してきます。

    キャンセル

  • 2017/01/14 15:54

    プログラムとしては、そこ、気になりますよね。

    キャンセル

checkベストアンサー

+2

直接の回答にはなっていません、ごめんなさい。

今回のような場合、

  1. 内容がごく短いテキストファイルを送信し、その内容を送り元、送り先の双方ですべて表示するようなデバッグ出力をし、その出力が同一かどうか検証する
  2. 同一である場合、それぞれ、「内容を埋め込んで CRC16 を計算する」
  3. sizeof(chunk.payload) 等、crc16 の引数の中身が同一か検証する

ことで、切り分けが可能ではないかと考えます。参考にしてください。

http://stackoverflow.com/questions/41647161/why-my-checksumcrc16-in-uart-is-not-matched
には書きませんでしたよ。

よろしくお願いいたします。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/01/14 15:56

    う〜ん、やってみます。(まさかみられてるとは・・・・・)

    キャンセル

  • 2017/01/14 16:33

    ありがとうございます。1と2をやってみたところ、どうやら再計算のところで間違えているようです。(とりあえず途中まで)

    キャンセル

  • 2017/01/14 17:24

    3に関しては2回め以降が一致しないようです。正直う〜んという感じです。

    キャンセル

  • 2017/01/14 17:42

    sizeof(recData) ということは、1024 固定ですよね、後ろの0埋めとかかもしれませんね。
    がんばって切り分けてみて下さい。

    キャンセル

同じタグがついた質問を見る

  • Linux

    4365questions

    Linuxは、Unixをベースにして開発されたオペレーティングシステムです。日本では「リナックス」と呼ばれています。 主にWebサーバやDNSサーバ、イントラネットなどのサーバ用OSとして利用されています。 上位500のスーパーコンピュータの90%以上はLinuxを使用しています。 携帯端末用のプラットフォームAndroidは、Linuxカーネル上に構築されています。

  • C++

    4295questions

    C++はC言語をもとにしてつくられた最もよく使われるマルチパラダイムプログラミング言語の1つです。オブジェクト指向、ジェネリック、命令型など広く対応しており、多目的に使用されています。

  • ポインタ

    130questions

    ポインタはアドレスを用いてメモリに格納された値を"参照する"変数です。

  • ファイルI/O

    42questions

    ファイルI/Oは、コンピューターにおけるファイルの入出力です。これは生成/削除やファイルを読み込んだり、出力をファイルに書き込むようなディレクトリやファイルの運用を含みます。

  • トップ
  • C++に関する質問
  • ファイル転送のプロトコルをC++で実装しているが、チェックサムのCRC16が合わない?どうして?