BOOST :: CR C создание контрольной суммы с помощью метода process.block () - PullRequest
1 голос
/ 03 августа 2020

В настоящее время я пытаюсь создать контрольную сумму по длине настраиваемой структуры UMGR_s. Я использовал метод BOOST :: cr c process.block () с начальным и конечным адресами моей структуры.

Проблема в том, что я буду получать другую контрольную сумму каждый раз, когда запускаю этот метод, даже хотя данные в структуре остаются согласованными. Я не уверен, что я здесь делаю не так.

int checksum_manager::createCRC(UMGR_s *CRCdata)
{
    boost::crc_32_type result;

    result.process_block(CRCdata, CRCdata + 1);

    return result.checksum();
};

1 Ответ

1 голос
/ 04 августа 2020

Оглядываясь на ваш старый вопрос:

#include <string>
#include <type_traits>

struct UMGR_s {
    std::string name;
    std::string description;
    std::string dlt_id;
    std::string log_mode;
    std::string log_level;
    std::string log_dir_path;
    uint ipc_port;
    uint reconnection_retry_offset;
    uint msg_buf_size;
    int checksum;
};

static_assert(std::is_standard_layout<UMGR_s>{});
static_assert(not std::is_trivial<UMGR_s>{});
static_assert(not std::is_pod<UMGR_s>{});

Это не POD. Как вы уже заметили, простое преобразование битов не является надежным.

Рассматривайте его как объект C ++ и напишите функцию контрольной суммы, например:

int calcCRC() {
    boost::crc_32_type crc;
    crc.process_bytes(name.data(), name.size());
    crc.process_bytes(description.data(), description.size());
    crc.process_bytes(dlt_id.data(), dlt_id.size());
    crc.process_bytes(log_mode.data(), log_mode.size());
    crc.process_bytes(log_level.data(), log_level.size());
    crc.process_bytes(log_dir_path.data(), log_dir_path.size());
    crc.process_block(&ipc_port, &ipc_port+1);
    crc.process_block(&reconnection_retry_offset, &reconnection_retry_offset+1);
    crc.process_block(&msg_buf_size, &msg_buf_size+1);

    return crc.checksum();
}

Посмотреть Live На Coliru .

Пример последовательно возвращает 0xbf21e978 и без предупреждений UB или valgrind / ASAN.

Еще лучше, сделайте меньше подвержен ошибкам:

struct CRC {
    boost::crc_32_type crc;
    void operator()(std::string_view s) {
        crc.process_bytes(s.data(), s.size());
    }
    template <typename T, typename = std::enable_if_t<std::is_integral_v<T>>>
    void operator()(T const& i) {
        static_assert(std::is_trivial_v<T>);
        static_assert(not std::is_class_v<T>);
        crc.process_bytes(&i, sizeof(i));
    }

    auto get() { return crc.checksum(); }
};

Это означает, что теперь вы можете просто написать:

int calcCRC() {
    CRC crc;
    crc(name);
    crc(description);
    crc(dlt_id);
    crc(log_mode);
    crc(log_level);
    crc(log_dir_path);
    crc(ipc_port);
    crc(reconnection_retry_offset);
    crc(msg_buf_size);

    return crc.get();
}

Live Demo

Live On Coliru

#include <string>
#include <iostream>
#include <type_traits>
#include <boost/crc.hpp>

struct CRC {
    boost::crc_32_type crc;
    void operator()(std::string_view s) {
        crc.process_bytes(s.data(), s.size());
    }
    template <typename T, typename = std::enable_if_t<std::is_integral_v<T>>>
    void operator()(T const& i) {
        static_assert(std::is_trivial_v<T>);
        static_assert(not std::is_class_v<T>);
        crc.process_bytes(&i, sizeof(i));
    }

    auto get() { return crc.checksum(); }
};

struct UMGR_s {
    std::string name;
    std::string description;
    std::string dlt_id;
    std::string log_mode;
    std::string log_level;
    std::string log_dir_path;
    uint ipc_port;
    uint reconnection_retry_offset;
    uint msg_buf_size;
    int checksum;

    int calcCRC() {
        CRC crc;
        crc(name);
        crc(description);
        crc(dlt_id);
        crc(log_mode);
        crc(log_level);
        crc(log_dir_path);
        crc(ipc_port);
        crc(reconnection_retry_offset);
        crc(msg_buf_size);

        return crc.get();
    }
};

static_assert(std::is_standard_layout<UMGR_s>{});
static_assert(not std::is_trivial<UMGR_s>{});
//static_assert(not std::is_pod<UMGR_s>{});

int main() {
    UMGR_s data {
         "UMGR",
         "UpdateManager",
         "1234",
         "kConsole",
         "kVerbose",
         "",
         33,
         0,
         1000,
         0,
    };

    data.checksum = data.calcCRC();

    std::cout << "crc: " << std::hex << std::showbase << data.checksum << "\n";
}

Обратите внимание, что он по-прежнему печатает тот же CR C:

crc: 0xbf21e978
...