[英]gcc-8 Wstringop-truncation
我正在嘗試修復gcc-8抱怨Wstringop-truncation的一些C代碼(代碼在這里 )
在無法控制的服務器上編譯該代碼時,既不能添加編譯指示也不能禁用Wstringop-truncation診斷,我收到的警告是:
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);
...
}
問題中使用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);
}
如果還有此功能無法涵蓋的其他用例,請更新問題。
聲明:本站的技術帖子網頁,遵循CC BY-SA 4.0協議,如果您需要轉載,請注明本站網址或者原文地址。任何問題請咨詢:yoyou2525@163.com.