mirror of
https://github.com/Unidata/netcdf-c.git
synced 2024-11-21 03:13:42 +08:00
751300ec59
This is a follow up to PR https://github.com/Unidata/netcdf-c/pull/1173 Sorry that it is so big, but leak suppression can be complex. This PR fixes all remaining memory leaks -- as determined by -fsanitize=address, and with the exceptions noted below. Unfortunately. there remains a significant leak that I cannot solve. It involves vlens, and it is unclear if the leak is occurring in the netcdf-c library or the HDF5 library. I have added a check_PROGRAM to the ncdump directory to show the problem. The program is called tst_vlen_demo.c To exercise it, build the netcdf library with -fsanitize=address enabled. Then go into ncdump and do a "make clean check". This should build tst_vlen_demo without actually executing it. Then do the command "./tst_vlen_demo" to see the output of the memory checker. Note the the lost malloc is deep in the HDF5 library (in H5Tvlen.c). I am temporarily working around this error in the following way. 1. I modified several test scripts to not execute known vlen tests that fail as described above. 2. Added an environment variable called NC_VLEN_NOTEST. If set, then those specific tests are suppressed. This should mean that the --disable-utilities option to ./configure should not need to be set to get a memory leak clean build. This should allow for detection of any new leaks. Note: I used an environment variable rather than a ./configure option to control the vlen tests. This is because it is temporary (I hope) and because it is a bit tricky for shell scripts to access ./configure options. Finally, as before, this only been tested with netcdf-4 and hdf5 support.
236 lines
4.8 KiB
Bash
Executable File
236 lines
4.8 KiB
Bash
Executable File
#!/bin/sh
|
|
|
|
if test "x$srcdir" = x ; then srcdir=`pwd`; fi
|
|
. ../test_common.sh
|
|
|
|
#set -e
|
|
echo "*** ctests.sh: testing ncgen4 -c"
|
|
|
|
KFLAG=3
|
|
|
|
# get some config.h parameters
|
|
if test -f ${top_builddir}/config.h ; then
|
|
if fgrep -e '#define USE_NETCDF4 1' ${top_builddir}/config.h >/dev/null ; then
|
|
NETCDF4=1
|
|
else
|
|
NETCDF4=0
|
|
fi
|
|
if fgrep -e '#define ENABLE_DAP 1' ${top_builddir}/config.h >/dev/null ; then
|
|
DAP=1
|
|
else
|
|
DAP=0
|
|
fi
|
|
|
|
else
|
|
echo "Cannot locate config.h"
|
|
exit 1
|
|
fi
|
|
|
|
# Locate the cdl and expected directories
|
|
cdl="${srcdir}/cdl4"
|
|
expected="${srcdir}/expected4"
|
|
|
|
# Locate various libraries, programs, etc
|
|
# The hard part is locating the HDF5 and Z libraries
|
|
|
|
NCGEN4=${NCGEN}
|
|
NCDUMP=${NCDUMP}
|
|
|
|
HDFPATH=""
|
|
ZPATH=""
|
|
SZPATH=""
|
|
CURLPATH=""
|
|
CC=cc
|
|
if test -f ${builddir}/Makefile ; then
|
|
if test "x$NETCDF4" = "x1"; then
|
|
tmp=`sed -e '/^HDF5DIR/p' -e d <${builddir}/Makefile|tr -d " "`
|
|
HDFPATH=`echo $tmp |cut -d '=' -f2`
|
|
tmp=`sed -e '/^ZLIBDIR/p' -e d <${builddir}/Makefile|tr -d " "`
|
|
ZPATH=`echo $tmp |cut -d '=' -f2`
|
|
tmp=`sed -e '/^SZLIBDIR/p' -e d <${builddir}/Makefile|tr -d " "`
|
|
SZPATH=`echo $tmp |cut -d '=' -f2`
|
|
fi
|
|
if test "x$DAP" = "x1"; then
|
|
tmp=`sed -e '/^CURLDIR/p' -e d <${builddir}/Makefile|tr -d " "`
|
|
CURLPATH=`echo $tmp |cut -d '=' -f2`
|
|
fi
|
|
tmp=`sed -e "/^CC\ /p" -e d <${builddir}/Makefile|tr -d " "`
|
|
CC=`echo $tmp |cut -d '=' -f2`
|
|
else
|
|
echo "No Makefile => cannot locate HDF5 path"
|
|
exit 1
|
|
fi
|
|
|
|
if test "x$ZPATH" = "x" ; then
|
|
ZPATH="$SZPATH"
|
|
fi
|
|
if test "x$ZPATH" = "x" ; then
|
|
echo "NO libz or libsz specified"
|
|
exit 1
|
|
fi
|
|
|
|
INCL=""
|
|
LIBLOCS=""
|
|
RPATH=""
|
|
|
|
if test "x$NETCDF4" = "x1"; then
|
|
NCLIBPATH="${builddir}/libsrc4/.libs"
|
|
NCINCLPATH="${srcdir}/libsrc4"
|
|
else
|
|
NCPATH="${builddir}/libsrc/.libs"
|
|
NCINCLPATH="${srcdir}/libsrc"
|
|
fi
|
|
|
|
INCL="-I ${NCINCLPATH}"
|
|
|
|
if test "x$NETCDF4" = "x1"; then
|
|
LIBLOCS="-L ${NCLIBPATH} -L ${HDFPATH}/lib -L ${ZPATH}/lib"
|
|
LIBS="${LIBLOCS} -lnetcdf -lhdf5_hl -lhdf5 -lz -lm"
|
|
#RPATH=-Wl,-rpath,${NCLIBPATH},-rpath,${HDFPATH}/lib,-rpath,${ZPATH}/lib
|
|
else
|
|
LIBLOCS="-L ${NCLIBPATH}"
|
|
LIBS="${LIBLOCS} -lnetcdf -lm"
|
|
#RPATH="-Wl,-rpath,${NCLIBPATH}"
|
|
fi
|
|
|
|
if test "x$DAP" = "x1"; then
|
|
LIBS="${LIBS} -L ${CURLPATH}/lib -lcurl"
|
|
#RPATH="${RPATH},-rpath,${CURLPATH}/lib"
|
|
fi
|
|
|
|
# Define the set of tests that can be
|
|
# processed with either the -k nc3 or -k nc4 flag
|
|
|
|
TESTS3="\
|
|
test0 \
|
|
nc_enddef \
|
|
ref_tst_unicode \
|
|
ref_tst_utf8 \
|
|
sfc_pres_temp \
|
|
simple_xy \
|
|
small \
|
|
fills \
|
|
c0 \
|
|
example_good \
|
|
nc_sync \
|
|
pres_temp_4D \
|
|
ref_tst_small \
|
|
ref_nctst \
|
|
ref_nctst_64bit_offset \
|
|
small2 \
|
|
tst_ncml \
|
|
ref_ctest1_nc4 \
|
|
ref_ctest1_nc4c \
|
|
ref_nctst_netcdf4 \
|
|
ref_nctst_netcdf4_classic"
|
|
|
|
# Define the set of tests that must be
|
|
# processed with the -k nc4 flag
|
|
|
|
TESTS4="\
|
|
ref_dimscope \
|
|
ref_typescope \
|
|
ref_tst_string_data \
|
|
ref_tst_comp \
|
|
ref_tst_comp2 \
|
|
ref_tst_group_data \
|
|
ref_tst_opaque_data \
|
|
ref_tst_solar_1 \
|
|
ref_tst_solar_2 \
|
|
ref_tst_enum_data \
|
|
ref_tst_nans \
|
|
ref_tst_special_atts \
|
|
ref_const_test"
|
|
|
|
if test "x$NOVLEN" = x ; then
|
|
TESTS4="$TESTS4 ref_tst_vlen_data"
|
|
fi
|
|
|
|
SPECIALTESTS="ref_tst_special_atts"
|
|
|
|
XFAILTESTS="ref_tst_special_atts"
|
|
|
|
# Following are generally not run
|
|
# Because of the size of their output
|
|
BIGTESTS3="\
|
|
bigf1 \
|
|
bigf2 \
|
|
bigf3 \
|
|
bigr1 \
|
|
bigr2"
|
|
|
|
BIGTESTS4="ref_tst_solar_1"
|
|
|
|
# This test is both big and slow
|
|
BIGBIG3="gfs1"
|
|
|
|
if test "${KFLAG}" = 1 ; then
|
|
TESTSET="${TESTS3}"
|
|
elif test "${KFLAG}" = 3 ; then
|
|
TESTSET="${TESTS3} ${TESTS4}"
|
|
else
|
|
echo "Bad KFLAG=$KFLAG"
|
|
exit 1
|
|
fi
|
|
|
|
set -e
|
|
|
|
rm -fr results_ctests
|
|
mkdir results_ctests
|
|
|
|
failcount=0
|
|
passcount=0
|
|
xfailcount=0
|
|
|
|
for x in ${TESTSET} ; do
|
|
echo "Testing: ${x}"
|
|
specflag=""
|
|
isxfail=0
|
|
for s in $SPECIALTESTS ; do
|
|
if test "$x" = "$s" ; then specflag="-s"; fi
|
|
done
|
|
for s in $XFAILTESTS ; do
|
|
if test "$x" = "$s" ; then isxfail=1; fi
|
|
done
|
|
ok=0;
|
|
cd results_ctests
|
|
if ${NCGEN4} -lc -k${KFLAG} ${cdl}/${x}.cdl >${x}.c
|
|
then ok=1; else ok=0; fi
|
|
if $CC ${INCL} -c ${x}.c
|
|
then ok=1; else ok=0; fi
|
|
if $CC -o ${x}.exe ${x}.o ${LIBS} ${RPATH}
|
|
then ok=1; else ok=0; fi
|
|
if ./${x}.exe
|
|
then ok=1; else ok=0; fi
|
|
if ${NCDUMP} ${specflag} ${x}.nc > ${x}.dmp
|
|
then ok=1; else ok=0; fi
|
|
cd ..
|
|
if test 1 = 1; then
|
|
# compare with expected
|
|
if diff -b -w ${expected}/${x}.dmp results_ctests/${x}.dmp
|
|
then ok=1; else ok=0; fi
|
|
if test "$ok" = "1" ; then
|
|
echo "*** PASS: ${x}"
|
|
passcount=`expr $passcount + 1`
|
|
elif test $isxfail = 1 ; then
|
|
echo "*** XFAIL : ${x}"
|
|
xfailcount=`expr $xfailcount + 1`
|
|
else
|
|
echo "*** FAIL: ${x}"
|
|
failcount=`expr $failcount + 1`
|
|
fi
|
|
fi
|
|
done
|
|
|
|
totalcount=`expr $passcount + $failcount + $xfailcount`
|
|
okcount=`expr $passcount + $xfailcount`
|
|
rm -fr results_ctests
|
|
|
|
echo "PASSED: ${okcount}/${totalcount} ; ${xfailcount} expected failures ; ${failcount} unexpected failures"
|
|
|
|
#if test $failcount -gt 0 ; then
|
|
# exit 1
|
|
#else
|
|
exit 0
|
|
#fi
|