gcc-8 Wstringop-усечение - PullRequest
Новая
       41

gcc-8 Wstringop-усечение

0 голосов
/ 10 сентября 2018

Я пытаюсь исправить код C, где gcc-8 жалуется на усечение Wstringop (код здесь )

При компиляции этого кода на сервере, который я не могу контролировать, я не могу ни добавить прагматические операторы, ни отключить диагностику усечения Wstringop, я получаю предупреждение: 

gcc-8  -I"/home/hornik/tmp/R/include" -DNDEBUG -I./cqdb/include -I./crf/src -I./liblbfgs/include -I./include -I"/home/hornik/lib/R/Library/3.6/x86_64-linux-gnu/Rcpp/include" -I/usr/local/include   -fpic  -g -O2 -Wall -pedantic -mtune=native -c cqdb/src/cqdb.c -o cqdb/src/cqdb.o
cqdb/src/cqdb.c: In function ‘cqdb_writer_close’:
cqdb/src/cqdb.c:270:5: warning: ‘strncpy’ output truncated before terminating nul copying 4 bytes from a string of the same length [-Wstringop-truncation]
     strncpy((char*)header.chunkid, CHUNKID, 4);
     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
cqdb/src/cqdb.c: In function ‘cqdb_reader’:
cqdb/src/cqdb.c:469:9: warning: ‘strncpy’ specified bound 4 equals destination size [-Wstringop-truncation]
         strncpy((char*)db->header.chunkid, (const char*)p, 4);
         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

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

strncpy((char*)header.chunkid, CHUNKID, 4);
with strncpy((char*)header.chunkid, CHUNKID, 5);

and strncpy((char*)db->header.chunkid, (const char*)p, 4);
with strncpy((char*)db->header.chunkid, (const char*)p, 5);

Соответствующий код в cqdb.c приведен ниже. Он в основном проверяет, имеет ли файл тип 'CQDB'. Отметьте, что хотя я действительно хотел бы получить доступ, у меня нет доступа к этой машине, поэтому я не могу проверить, будут ли исправления кода C работать. 

#define CHUNKID             "CQDB"

typedef struct {
    int8_t      chunkid[4]; /**< Chunk identifier, "CQDB". */
    uint32_t    size;       /**< Chunk size including this header. */
    uint32_t    flag;       /**< Global flags. */
    uint32_t    byteorder;  /**< Byte-order indicator. */
    uint32_t    bwd_size;   /**< Number of elements in the backward array. */
    uint32_t    bwd_offset; /**< Offset to the backward array. */
} header_t;

int cqdb_writer_close(cqdb_writer_t* dbw)
{
header_t header;
strncpy((char*)header.chunkid, CHUNKID, 4);
...
}

cqdb_t* cqdb_reader(const void *buffer, size_t size)
{
    cqdb_t* db = NULL;
    /* Check the file chunkid. */
    if (memcmp(buffer, CHUNKID, 4) != 0) {
        return NULL;
    }
    db = (cqdb_t*)calloc(1, sizeof(cqdb_t));
    const uint8_t* p = NULL;
    db->buffer = buffer;
    p = db->buffer;
    strncpy((char*)db->header.chunkid, (const char*)p, 4);
...
}

1 Ответ

0 голосов
/ 11 сентября 2018

Использование strncpy в вопросе действительно правильно (выравнивание по левому краю некоторых символов в буфере, заполнение справа нулевыми байтами), но предупреждение состоит в том, что эта функция часто используется людьми, пытающимися скопировать нулевое значение строка.

Для кода, показанного в вопросе, я бы заменил вызовы strncpy на: 

set_chunkid(&header);

где вы добавляете новую функцию: 

void set_chunkid(header_t *hdr)
{
    _Static_assert(sizeof CHUNKID == sizeof hdr->chunkid + 1, "chunk ID not 4 chars");

    memcpy(&hdr->chunkid, CHUNKID, sizeof hdr->chunkid);
}

Если есть другие варианты использования, которые эта функция не охватывает, обновите вопрос.

...