Fix incorrect uses of gzFile
authorPeter Eisentraut <peter_e@gmx.net>
Fri, 2 Mar 2012 20:30:01 +0000 (22:30 +0200)
committerPeter Eisentraut <peter_e@gmx.net>
Fri, 2 Mar 2012 20:30:01 +0000 (22:30 +0200)
commitd923125b77c5d698bb8107a533a21627582baa43
treedf1511da4e4e6b1edda06c57deeb282c81de5260
parent8e5f4300fdcadac1bfd72a7d1a0225030226c800
Fix incorrect uses of gzFile

gzFile is already a pointer, so code like

gzFile *handle = gzopen(...)

is wrong.

This used to pass silently because gzFile used to be defined as void*,
and you can assign a void* to a void**.  But somewhere between zlib
versions 1.2.3.4 and 1.2.6, the definition of gzFile was changed to
struct gzFile_s *, and with that new definition this usage causes
compiler warnings.

So remove all those extra pointer decorations.

There is a related issue in pg_backup_archiver.h, where

FILE       *FH;             /* General purpose file handle */

is used throughout pg_dump as sometimes a real FILE* and sometimes a
gzFile handle, which also causes warnings now.  This is not yet fixed
here, because it might need more code restructuring.
src/bin/pg_basebackup/pg_basebackup.c
src/bin/pg_dump/pg_backup_files.c
src/bin/pg_dump/pg_backup_tar.c