mirror of
https://github.com/dashpay/dash.git
synced 2024-12-26 04:22:55 +01:00
Squashed 'src/leveldb/' changes from c521b3ac65..64052c76c5
64052c76c5 Merge #15: Add filename to corruption errors 135ed0fb4e Add filename to corruption errors git-subtree-dir: src/leveldb git-subtree-split: 64052c76c567cff3dad32d1db0ef969d97b5882f
This commit is contained in:
parent
b13a68e129
commit
835a21b424
@ -414,7 +414,7 @@ Status DBImpl::RecoverLogFile(uint64_t log_number, bool last_log,
|
||||
status.ok()) {
|
||||
if (record.size() < 12) {
|
||||
reporter.Corruption(
|
||||
record.size(), Status::Corruption("log record too small"));
|
||||
record.size(), Status::Corruption("log record too small", fname));
|
||||
continue;
|
||||
}
|
||||
WriteBatchInternal::SetContents(&batch, record);
|
||||
|
@ -19,6 +19,7 @@ class StdoutPrinter : public WritableFile {
|
||||
virtual Status Close() { return Status::OK(); }
|
||||
virtual Status Flush() { return Status::OK(); }
|
||||
virtual Status Sync() { return Status::OK(); }
|
||||
virtual std::string GetName() const { return "[stdout]"; }
|
||||
};
|
||||
|
||||
bool HandleDumpCommand(Env* env, char** files, int num) {
|
||||
|
@ -186,7 +186,7 @@ uint64_t Reader::LastRecordOffset() {
|
||||
}
|
||||
|
||||
void Reader::ReportCorruption(uint64_t bytes, const char* reason) {
|
||||
ReportDrop(bytes, Status::Corruption(reason));
|
||||
ReportDrop(bytes, Status::Corruption(reason, file_->GetName()));
|
||||
}
|
||||
|
||||
void Reader::ReportDrop(uint64_t bytes, const Status& reason) {
|
||||
|
@ -203,7 +203,7 @@ class Repairer {
|
||||
while (reader.ReadRecord(&record, &scratch)) {
|
||||
if (record.size() < 12) {
|
||||
reporter.Corruption(
|
||||
record.size(), Status::Corruption("log record too small"));
|
||||
record.size(), Status::Corruption("log record too small", logname));
|
||||
continue;
|
||||
}
|
||||
WriteBatchInternal::SetContents(&batch, record);
|
||||
|
@ -176,6 +176,7 @@ class SequentialFileImpl : public SequentialFile {
|
||||
return Status::OK();
|
||||
}
|
||||
|
||||
virtual std::string GetName() const { return "[memenv]"; }
|
||||
private:
|
||||
FileState* file_;
|
||||
uint64_t pos_;
|
||||
@ -196,6 +197,7 @@ class RandomAccessFileImpl : public RandomAccessFile {
|
||||
return file_->Read(offset, n, result, scratch);
|
||||
}
|
||||
|
||||
virtual std::string GetName() const { return "[memenv]"; }
|
||||
private:
|
||||
FileState* file_;
|
||||
};
|
||||
@ -218,6 +220,7 @@ class WritableFileImpl : public WritableFile {
|
||||
virtual Status Flush() { return Status::OK(); }
|
||||
virtual Status Sync() { return Status::OK(); }
|
||||
|
||||
virtual std::string GetName() const { return "[memenv]"; }
|
||||
private:
|
||||
FileState* file_;
|
||||
};
|
||||
|
@ -191,6 +191,9 @@ class SequentialFile {
|
||||
// REQUIRES: External synchronization
|
||||
virtual Status Skip(uint64_t n) = 0;
|
||||
|
||||
// Get a name for the file, only for error reporting
|
||||
virtual std::string GetName() const = 0;
|
||||
|
||||
private:
|
||||
// No copying allowed
|
||||
SequentialFile(const SequentialFile&);
|
||||
@ -215,6 +218,9 @@ class RandomAccessFile {
|
||||
virtual Status Read(uint64_t offset, size_t n, Slice* result,
|
||||
char* scratch) const = 0;
|
||||
|
||||
// Get a name for the file, only for error reporting
|
||||
virtual std::string GetName() const = 0;
|
||||
|
||||
private:
|
||||
// No copying allowed
|
||||
RandomAccessFile(const RandomAccessFile&);
|
||||
@ -234,6 +240,9 @@ class WritableFile {
|
||||
virtual Status Flush() = 0;
|
||||
virtual Status Sync() = 0;
|
||||
|
||||
// Get a name for the file, only for error reporting
|
||||
virtual std::string GetName() const = 0;
|
||||
|
||||
private:
|
||||
// No copying allowed
|
||||
WritableFile(const WritableFile&);
|
||||
|
@ -82,7 +82,7 @@ Status ReadBlock(RandomAccessFile* file,
|
||||
}
|
||||
if (contents.size() != n + kBlockTrailerSize) {
|
||||
delete[] buf;
|
||||
return Status::Corruption("truncated block read");
|
||||
return Status::Corruption("truncated block read", file->GetName());
|
||||
}
|
||||
|
||||
// Check the crc of the type and the block contents
|
||||
@ -92,7 +92,7 @@ Status ReadBlock(RandomAccessFile* file,
|
||||
const uint32_t actual = crc32c::Value(data, n + 1);
|
||||
if (actual != crc) {
|
||||
delete[] buf;
|
||||
s = Status::Corruption("block checksum mismatch");
|
||||
s = Status::Corruption("block checksum mismatch", file->GetName());
|
||||
return s;
|
||||
}
|
||||
}
|
||||
@ -119,13 +119,13 @@ Status ReadBlock(RandomAccessFile* file,
|
||||
size_t ulength = 0;
|
||||
if (!port::Snappy_GetUncompressedLength(data, n, &ulength)) {
|
||||
delete[] buf;
|
||||
return Status::Corruption("corrupted compressed block contents");
|
||||
return Status::Corruption("corrupted compressed block contents", file->GetName());
|
||||
}
|
||||
char* ubuf = new char[ulength];
|
||||
if (!port::Snappy_Uncompress(data, n, ubuf)) {
|
||||
delete[] buf;
|
||||
delete[] ubuf;
|
||||
return Status::Corruption("corrupted compressed block contents");
|
||||
return Status::Corruption("corrupted compressed block contents", file->GetName());
|
||||
}
|
||||
delete[] buf;
|
||||
result->data = Slice(ubuf, ulength);
|
||||
@ -135,7 +135,7 @@ Status ReadBlock(RandomAccessFile* file,
|
||||
}
|
||||
default:
|
||||
delete[] buf;
|
||||
return Status::Corruption("bad block type");
|
||||
return Status::Corruption("bad block type", file->GetName());
|
||||
}
|
||||
|
||||
return Status::OK();
|
||||
|
@ -121,6 +121,8 @@ class PosixSequentialFile: public SequentialFile {
|
||||
}
|
||||
return Status::OK();
|
||||
}
|
||||
|
||||
virtual std::string GetName() const { return filename_; }
|
||||
};
|
||||
|
||||
// pread() based random-access
|
||||
@ -172,6 +174,8 @@ class PosixRandomAccessFile: public RandomAccessFile {
|
||||
}
|
||||
return s;
|
||||
}
|
||||
|
||||
virtual std::string GetName() const { return filename_; }
|
||||
};
|
||||
|
||||
// mmap() based random-access
|
||||
@ -206,6 +210,8 @@ class PosixMmapReadableFile: public RandomAccessFile {
|
||||
}
|
||||
return s;
|
||||
}
|
||||
|
||||
virtual std::string GetName() const { return filename_; }
|
||||
};
|
||||
|
||||
class PosixWritableFile : public WritableFile {
|
||||
@ -287,6 +293,8 @@ class PosixWritableFile : public WritableFile {
|
||||
}
|
||||
return s;
|
||||
}
|
||||
|
||||
virtual std::string GetName() const { return filename_; }
|
||||
};
|
||||
|
||||
static int LockOrUnlock(int fd, bool lock) {
|
||||
|
@ -78,6 +78,7 @@ public:
|
||||
virtual Status Read(size_t n, Slice* result, char* scratch);
|
||||
virtual Status Skip(uint64_t n);
|
||||
BOOL isEnable();
|
||||
virtual std::string GetName() const { return _filename; }
|
||||
private:
|
||||
BOOL _Init();
|
||||
void _CleanUp();
|
||||
@ -94,6 +95,7 @@ public:
|
||||
virtual ~Win32RandomAccessFile();
|
||||
virtual Status Read(uint64_t offset, size_t n, Slice* result,char* scratch) const;
|
||||
BOOL isEnable();
|
||||
virtual std::string GetName() const { return _filename; }
|
||||
private:
|
||||
BOOL _Init(LPCWSTR path);
|
||||
void _CleanUp();
|
||||
@ -114,6 +116,7 @@ public:
|
||||
virtual Status Flush();
|
||||
virtual Status Sync();
|
||||
BOOL isEnable();
|
||||
virtual std::string GetName() const { return filename_; }
|
||||
private:
|
||||
std::string filename_;
|
||||
::HANDLE _hFile;
|
||||
|
Loading…
Reference in New Issue
Block a user