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:18 -0700 |
| commit | 89a94c24aadd8f40084339ea3d16e2e14fa9f629 (patch) | |
| tree | 270e19cb553762a9da08cac32bb1b87b9563631e /src/backend/utils/cache/relcache.c | |
| parent | e52e9bd57f33da1c4a915e24d0a24759a23cb5fd (diff) | |
| download | postgresql-89a94c24aadd8f40084339ea3d16e2e14fa9f629.tar.gz postgresql-89a94c24aadd8f40084339ea3d16e2e14fa9f629.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 b54c9117669..dcf56d47900 100644 --- a/src/backend/utils/cache/relcache.c +++ b/src/backend/utils/cache/relcache.c @@ -6485,7 +6485,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"); } |