Revision e5ae1bb46564689e56a38f3509daffa4aca3b29c authored by Daniel Black on 27 June 2018, 19:09:50 UTC, committed by Facebook Github Bot on 27 June 2018, 19:23:07 UTC
Summary: Various rearrangements of the cch maths failed or replacing = '\0' with memset failed to convince the compiler it was nul terminated. So took the perverse option of changing strncpy to strcpy. Return null if memory couldn't be allocated. util/status.cc: In static member function ‘static const char* rocksdb::Status::CopyState(const char*)’: util/status.cc:28:15: error: ‘char* strncpy(char*, const char*, size_t)’ output truncated before terminating nul copying as many bytes from a string as its length [-Werror=stringop-truncation] std::strncpy(result, state, cch - 1); ~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~ util/status.cc:19:18: note: length computed here std::strlen(state) + 1; // +1 for the null terminator ~~~~~~~~~~~^~~~~~~ cc1plus: all warnings being treated as errors make: *** [Makefile:645: shared-objects/util/status.o] Error 1 closes #2705 Closes https://github.com/facebook/rocksdb/pull/3870 Differential Revision: D8594114 Pulled By: anand1976 fbshipit-source-id: ab20f3a456a711e4d29144ebe630e4fe3c99ec25
1 parent a16e00b
File | Mode | Size |
---|---|---|
.gitignore | -rw-r--r-- | 176 bytes |
Makefile | -rw-r--r-- | 2.2 KB |
README.md | -rw-r--r-- | 120 bytes |
c_simple_example.c | -rw-r--r-- | 2.6 KB |
column_families_example.cc | -rw-r--r-- | 1.9 KB |
compact_files_example.cc | -rw-r--r-- | 5.7 KB |
compaction_filter_example.cc | -rw-r--r-- | 2.9 KB |
optimistic_transaction_example.cc | -rw-r--r-- | 4.1 KB |
options_file_example.cc | -rw-r--r-- | 4.0 KB |
rocksdb_option_file_example.ini | -rw-r--r-- | 4.6 KB |
simple_example.cc | -rw-r--r-- | 2.4 KB |
transaction_example.cc | -rw-r--r-- | 4.1 KB |
![swh spinner](/static/img/swh-spinner.gif)
Computing file changes ...