diff options
| author | Andres Freund <andres@anarazel.de> | 2022-03-23 13:05:25 -0700 |
|---|---|---|
| committer | Andres Freund <andres@anarazel.de> | 2022-03-23 13:05:25 -0700 |
| commit | 1c6bb380e5aba195204a9c6d0b4713bd1b3dec9c (patch) | |
| tree | dc50a73f514ead5a8747663df8056d133328b4f8 /src/backend/utils/cache/relcache.c | |
| parent | e71c76fcab8d00defe9a7a608facdd9663f7bcbf (diff) | |
| download | postgresql-1c6bb380e5aba195204a9c6d0b4713bd1b3dec9c.tar.gz postgresql-1c6bb380e5aba195204a9c6d0b4713bd1b3dec9c.zip | |
Don't call fwrite() with len == 0 when writing out relcache init file.
Noticed via -fsanitize=undefined.
Backpatch to all branches, for the same reasons as 46ab07ffda9.
Discussion: https://postgr.es/m/20220323173537.ll7klrglnp4gn2um@alap3.anarazel.de
Backpatch: 10-
Diffstat (limited to 'src/backend/utils/cache/relcache.c')
| -rw-r--r-- | src/backend/utils/cache/relcache.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/backend/utils/cache/relcache.c b/src/backend/utils/cache/relcache.c index fbd11883e17..3d05297b0d9 100644 --- a/src/backend/utils/cache/relcache.c +++ b/src/backend/utils/cache/relcache.c @@ -6528,7 +6528,7 @@ write_item(const void *data, Size len, FILE *fp) { if (fwrite(&len, 1, sizeof(len), fp) != sizeof(len)) elog(FATAL, "could not write init file"); - if (fwrite(data, 1, len, fp) != len) + if (len > 0 && fwrite(data, 1, len, fp) != len) elog(FATAL, "could not write init file"); } |