gvsafronov/futriix
1
0
Fork 0
Code Issues Pull Requests Actions 4 Packages Projects Releases Wiki Activity

Log the real reason for why posix_fadvise failed (#430)

Browse Source 
`reclaimFilePageCache` did not set `errno` but `rdbSaveInternal` which
is logging the error assumed it did. This makes sure `errno` is set.

Signed-off-by: Ted Lyngmo <ted@lyncon.se>
This commit is contained in:
Ted Lyngmo 2024-05-03 20:10:33 +02:00 committed by Madelyn Olson
parent 50eefd647d
commit 2be94e68e8
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/util.c
View File

@ -1138,7 +1138,10 @@ int fsyncFileDir(const char *filename) {
int reclaimFilePageCache(int fd, size_t offset, size_t length) { int reclaimFilePageCache(int fd, size_t offset, size_t length) {
#ifdef HAVE_FADVISE #ifdef HAVE_FADVISE
int ret = posix_fadvise(fd, offset, length, POSIX_FADV_DONTNEED); int ret = posix_fadvise(fd, offset, length, POSIX_FADV_DONTNEED);
if (ret) return -1; if (ret) {
errno = ret;
return -1;
}
return 0; return 0;
#else #else
UNUSED(fd); UNUSED(fd);