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:13:33 -0700 |
| commit | c5b60a68cc08400dd2de5e2ca2908a204e6bef54 (patch) | |
| tree | 3c10b0e290ce169c59039af4e7bdbab79ad0bb28 /src/backend/utils/cache | |
| parent | 6a767bc2e71254f74f0b8c576fa584b789cda00e (diff) | |
| download | postgresql-c5b60a68cc08400dd2de5e2ca2908a204e6bef54.tar.gz postgresql-c5b60a68cc08400dd2de5e2ca2908a204e6bef54.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')
| -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 87651111ee3..bad672f51b3 100644 --- a/src/backend/utils/cache/relcache.c +++ b/src/backend/utils/cache/relcache.c @@ -6083,7 +6083,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"); } |