diff options
| author | Michael Paquier <michael@paquier.xyz> | 2019-03-27 12:02:50 +0900 |
|---|---|---|
| committer | Michael Paquier <michael@paquier.xyz> | 2019-03-27 12:02:50 +0900 |
| commit | 1983af8e899389187026cb34c1ca9d89ea986120 (patch) | |
| tree | 9bdef33ce2dcffd83fe159032272cd94b0aa6a46 /contrib | |
| parent | 5bde1651bbba9984dc32bbeb68b329cfd3f92767 (diff) | |
| download | postgresql-1983af8e899389187026cb34c1ca9d89ea986120.tar.gz postgresql-1983af8e899389187026cb34c1ca9d89ea986120.zip | |
Switch some palloc/memset calls to palloc0
Some code paths have been doing some allocations followed by an
immediate memset() to initialize the allocated area with zeros, this is
a bit overkill as there are already interfaces to do both things in one
call.
Author: Daniel Gustafsson
Reviewed-by: Michael Paquier
Discussion: https://postgr.es/m/vN0OodBPkKs7g2Z1uyk3CUEmhdtspHgYCImhlmSxv1Xn6nY1ZnaaGHL8EWUIQ-NEv36tyc4G5-uA3UXUF2l4sFXtK_EQgLN1hcgunlFVKhA=@yesql.se
Diffstat (limited to 'contrib')
| -rw-r--r-- | contrib/tablefunc/tablefunc.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/contrib/tablefunc/tablefunc.c b/contrib/tablefunc/tablefunc.c index 02e49214101..cd02495c451 100644 --- a/contrib/tablefunc/tablefunc.c +++ b/contrib/tablefunc/tablefunc.c @@ -867,11 +867,8 @@ get_crosstab_tuplestore(char *sql, "tuple has %d columns but crosstab " \ "returns %d.", tupdesc->natts, result_ncols))); - /* allocate space */ - values = (char **) palloc(result_ncols * sizeof(char *)); - - /* and make sure it's clear */ - memset(values, '\0', result_ncols * sizeof(char *)); + /* allocate space and make sure it's clear */ + values = (char **) palloc0(result_ncols * sizeof(char *)); for (i = 0; i < proc; i++) { |