summaryrefslogtreecommitdiffstats
path: root/hl/src/H5DS.c
Commit message (Collapse)AuthorAgeFilesLines
* [svn-r20061] Description:Quincey Koziol2011-02-081-164/+102
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bring changes from Coverity branch to trunk: r19930: Fix memory leaks involving VL attributes in h5repack and h5diff. The buffers in copy_attr and diff_attr were not checked for the presence of a vlen before being freed, and vlen storage was never reclaimed. Added checks and calls to H5D_vlen_reclaim(). r19933: Purpose: Fix memory leak in H5L_move_cb() Description: H5L_move_cb copied the source link using H5O_msg_copy() but freed it manually using H5MM_xfree(). Since H5O_link_copy allocates the link using H5FL_MALLOC, this causes the link to be allocated from the free list but is never put back on the free list when it is freed. This prevents the link free list from shutting down properly. Modified H5L_move_cb() and H5L_move_dest_cb() to free the link properly using H5O_msg_free(). r19973: Fix resource leaks by freeing string created by HD5f2string r19974: Issue #345: Inialize buf variable to null Tested on: Mac OS X/32 10.6.6 (amazon) w/debug & production (h5committested on Coverity branch)
* [svn-r18882] Maintenance: fixed a typo in email addressElena Pourmal2010-05-221-1/+1
|
* [svn-r18880] Bug fix: Elena Pourmal2010-05-221-44/+55
| | | | | | | | Fixed a bug (1855) in H5DSattach_scale, H5DSis_attached and H5DSdetach_scale caused by using H5Tget_native_type function to determine the native type for reading REFERENCE_LIST attribute. The bug was exposed on Mac PPC. Platforms tested: juniper (Mac PPC), jam, amani, linew
* [svn-r18818] Bug fix: Elena Pourmal2010-05-151-112/+94
| | | | | | | | | | | Fixed a bug in the H5DSdetach_scale function when 0 bytes were allocated after the last reference to a dim. scale was removed from the list of references in a VL element of the DIMENSION_LIST attribute; modified the function to comply with the Spec: DIMENSION_LIST attribute is deleted now when no dimension scales left attached. Platforms tested: jam with electric fence, amani amd linew
* [svn-r18197] Description:Quincey Koziol2010-01-301-41/+41
| | | | | | | | | Trim trailing whitespace from source code files with this command: find . \( -name "*.[ch]" -or -name "*.cpp" -or -name "*.f90" \) -print |xargs -n 1 sed -i "" 's/[[:blank:]]*$//' Tested on: None - eyeballed only
* [svn-r17566] Fix for Bug 1562. H5DSis_attached needed to account for ↵Allen Byrne2009-10-011-4/+24
| | | | | | | | | platform dependent types. Added test and test files for LE and BE dimension scales. Reworked part of test_ds file as part of debug. Added missing H5Tclose in region reference routines. H5DS.c checked for other locations which need to account for platform dependent types in dealing with ds_list_t. Added generator program for test files of LE and BE dimension scales. Tested: local linux, h5committest
* [svn-r17268] Description:Quincey Koziol2009-07-301-3/+3
| | | | | | | | | | | Bring r17266 from 1.8 branch to trunk: Clean up various compiler warnings from IRIX64 build Tested on: Mac OS X/32 (amazon) debug & production (yes, I know it's not an IRIX64 system :-) Too minor to require h5committest
* [svn-r16719] clean linux warningsPedro Vicente Nunes2009-04-091-6/+6
| | | | tested: linux
* [svn-r16706] #1538 (B2) Problems with Dim Scale APIs reported by MathworksPedro Vicente Nunes2009-04-081-16/+22
| | | | | | | ISSUE2: the scale index input/output parameter value passed to H5DSiterate_scales was not always incremented (it returns the scale index current iteration). SOLUTION FOR ISSUE2: modified the cycle in H5DSiterate_scales so that the scale index is always incremented TEST: added some test cases with calls to invalid indices and H5DSiterate_scales with return scale indices and visitor data tested: windows, linux
* [svn-r16696] #1538 (B2) Problems with Dim Scale APIs reported by MathworksPedro Vicente Nunes2009-04-081-60/+102
| | | | | | | ISSUE: When some functions are called with an invalid dimension index (for example, greater than the rank of the dataset) the function does not immediately fail, rather asserts down the code. SOLUTION: added this check for H5DSdetach_scale, H5DSset_label, H5DSget_label (the other functions have this check) . H5DSiterate_scales also added a check for an invalid dimension scale index TEST: added some test cases with calls to invalid indices DOCS: Nothing added
* [svn-r16676] #1521 (B2) H5DSdetach_scale memory leakPedro Vicente Nunes2009-04-071-3/+11
| | | | | | | | ISSUE: Purify (Windows Visual Studio 6) complains of a memory leak in H5DSdetach_scale call regarding a H5Aread call (stack is H5A_read, H5T_convert, H5T_conv_vlen, H5T_vlen_seq_mem_write, H5MM_malloc). SOLUTION: When a scale is detached from a dataset, the variable length structure length field is decreased in one entry. The associated pointer must be reallocated with the new length. DOCS AND TEST: Nothing added tested: windows with purify, linux
* [svn-r16588] Pedro Vicente Nunes2009-03-191-1574/+1671
| | | | | | 1. #1487 (B1) DS memory leaks ISSUE 1: On a "go to" error condition, previously allocated buffers were not freed. NOTE: these are "potential" memory leaks because typically the error conditions do not occur, so the potential memory leaks also do NOT occur. ISSUE 2: A function used to read dimension scales realistic data (topography of the North Atlantic, latitude and longitude) was being called without the data buffer being freed. SOLUTION FOR ISSUE 1: added "free" calls for the allocated buffers on the error sections. SOLUTION FOR ISSUE 2: added a "free" call after the read_data function, after using the buffer (on a H5Dwrite and H5Screate_simple). The read_data function reads both data and dimensions from the ASCII data files. DOCS: no docs added. Done for 1.9 and 1.8 STATUS: Closed bug tested: h5committest
* [svn-r16549] Bug #608: Memory leak in H5DSset_label. Added code to free sub ↵Allen Byrne2009-03-051-225/+258
| | | | | | | | | string ptr's belonging to buffer in H5DSset_label and H5DSget_label. Also added free of buffers in error section of both functions. Potential memory leaks may exist elsewhere, and this will not close the bug. Tested: h5committest vista 32 VS2008
* [svn-r15462] Description:Quincey Koziol2008-08-121-3/+3
| | | | | | | | Correct compiler warnings from Visual Studio. Tested on: Mac OS X/32 10.5.4 (amazon) w/FORTRAN & C++ (Too minor to require full h5committest)
* [svn-r14225] Description:Quincey Koziol2007-10-311-16/+16
| | | | | | | | | | | | | | | | | | | Change H5Oget_info -> H5Oget_info_by_name and re-add H5Oget_info in a simpler form for querying a particular object, to align with other new API routines. Tested on: FreeBSD/32 6.2 (duty) in debug mode FreeBSD/64 6.2 (liberty) w/C++ & FORTRAN, in debug mode Linux/32 2.6 (kagiso) w/PGI compilers, w/C++ & FORTRAN, w/threadsafe, in debug mode Linux/64-amd64 2.6 (smirom) w/default API=1.6.x, w/C++ & FORTRAN, in production mode Linux/64-ia64 2.6 (cobalt) w/Intel compilers, w/C++ & FORTRAN, in production mode Solaris/32 2.10 (linew) w/deprecated symbols disabled, w/C++ & FORTRAN, w/szip filter, in production mode Mac OS X/32 10.4.10 (amazon) in debug mode
* [svn-r14222] Description:Quincey Koziol2007-10-301-2/+2
| | | | | | | | | | | | | | | | | | Change H5Adelete2 -> H5Adelete_by_name and un-deprecate H5Adelete1 back to H5Adelete, for better consistency with new other API routiens. Tested on: FreeBSD/32 6.2 (duty) in debug mode FreeBSD/64 6.2 (liberty) w/C++ & FORTRAN, in debug mode Linux/32 2.6 (kagiso) w/PGI compilers, w/C++ & FORTRAN, w/threadsafe, in debug mode Linux/64-amd64 2.6 (smirom) w/default API=1.6.x, w/C++ & FORTRAN, in production mode Linux/64-ia64 2.6 (cobalt) w/Intel compilers, w/C++ & FORTRAN, in production mode Solaris/32 2.10 (linew) w/deprecated symbols disabled, w/C++ & FORTRAN, w/szip filter, in production mode Mac OS X/32 10.4.10 (amazon) in debug mode
* [svn-r14218] Description:Quincey Koziol2007-10-301-5/+5
| | | | | | | | | | | | | | | | | | | | | Changed H5Acreate2 -> H5Acreate_by_name, to be more consistent with other new API routines. Re-added simpler form of H5Acreate2, which creates attributes directly on an object. Tested on: FreeBSD/32 6.2 (duty) in debug mode FreeBSD/64 6.2 (liberty) w/C++ & FORTRAN, in debug mode Linux/32 2.6 (kagiso) w/PGI compilers, w/C++ & FORTRAN, w/threadsafe, in debug mode Linux/64-amd64 2.6 (smirom) w/default API=1.6.x, w/C++ & FORTRAN, in production mode Linux/64-ia64 2.6 (cobalt) w/Intel compilers, w/C++ & FORTRAN, in production mode Solaris/32 2.10 (linew) w/deprecated symbols disabled, w/C++ & FORTRAN, w/szip filter, in production mode Mac OS X/32 10.4.10 (amazon) in debug mode
* [svn-r14217] Description:Quincey Koziol2007-10-301-13/+13
| | | | | | | | | | | | | | | | | | | | | Change H5Aopen -> H5Aopen_by_name, in order to be more consistent with other new API routines. Re-add H5Aopen as a simpler routine, to open attributes on a particular object. (Much like the old H5Aopen_name routine). Tested on: FreeBSD/32 6.2 (duty) in debug mode FreeBSD/64 6.2 (liberty) w/C++ & FORTRAN, in debug mode Linux/32 2.6 (kagiso) w/PGI compilers, w/C++ & FORTRAN, w/threadsafe, in debug mode Linux/64-amd64 2.6 (smirom) w/default API=1.6.x, w/C++ & FORTRAN, in production mode Linux/64-ia64 2.6 (cobalt) w/Intel compilers, w/C++ & FORTRAN, in production mode Solaris/32 2.10 (linew) w/deprecated symbols disabled, w/C++ & FORTRAN, w/szip filter, in production mode Mac OS X/32 10.4.10 (amazon) in debug mode
* [svn-r14198] formatted some commentsPedro Vicente Nunes2007-10-101-17/+17
| | | | tested: linux
* [svn-r14187] Description:Quincey Koziol2007-10-041-42/+38
| | | | | | | | | | | | | | | | | | | | Put H5Acreate() under API versioning, with all internal usage shifted to H5Acreate2(). Add regression tests for H5Acreate1(). Tested on: FreeBSD/32 6.2 (duty) in debug mode FreeBSD/64 6.2 (liberty) w/C++ & FORTRAN, in debug mode Linux/32 2.6 (kagiso) w/PGI compilers, w/C++ & FORTRAN, w/threadsafe, in debug mode Linux/64-amd64 2.6 (smirom) w/default API=1.6.x, w/C++ & FORTRAN, in production mode Linux/64-ia64 2.6 (cobalt) w/Intel compilers, w/C++ & FORTRAN, in production mode Solaris/32 2.10 (linew) w/deprecated symbols disabled, w/C++ & FORTRAN, w/szip filter, in production mode Mac OS X/32 10.4.10 (amazon) in debug mode
* [svn-r14185] Description:Quincey Koziol2007-10-041-229/+229
| | | | | | | | | | | | | | | | | | | | | Move H5Aopen_name() routine to deprecated symbol section and replace internal usage with H5Aopen(). Add simple regression test for H5Aopen_name() to deprecated routine test. Tested on: FreeBSD/32 6.2 (duty) in debug mode FreeBSD/64 6.2 (liberty) w/C++ & FORTRAN, in debug mode Linux/32 2.6 (kagiso) w/PGI compilers, w/C++ & FORTRAN, w/threadsafe, in debug mode Linux/64-amd64 2.6 (smirom) w/default API=1.6.x, w/C++ & FORTRAN, in production mode Linux/64-ia64 2.6 (cobalt) w/Intel compilers, w/C++ & FORTRAN, in production mode Solaris/32 2.10 (linew) w/deprecated symbols disabled, w/C++ & FORTRAN, w/szip filter, in production mode Mac OS X/32 10.4.10 (amazon) in debug mode
* [svn-r14162] Description:Quincey Koziol2007-09-271-2/+2
| | | | | | | | | | | | | | | | | | Make H5Adelete versioned and switch internal library use to H5Adelete2. Add regression test for H5Adelete1 Tested on: FreeBSD/32 6.2 (duty) in debug mode FreeBSD/64 6.2 (liberty) w/C++ & FORTRAN, in debug mode Linux/32 2.6 (kagiso) w/PGI compilers, w/C++ & FORTRAN, w/threadsafe, in debug mode Linux/64 2.6 (smirom) w/default API=1.6.x, w/C++ & FORTRAN, in production mode Solaris/32 2.10 (linew) w/deprecated symbols disabled, w/C++ & FORTRAN, w/szip filter, in production mode AIX/32 5.3 (copper) w/FORTRAN, w/parallel, in production mode Mac OS X/32 10.4.10 (amazon) in debug mode
* [svn-r14144] Description:Quincey Koziol2007-09-131-45/+32
| | | | | | | | | | | | | | | | Move H5Gget_objinfo() to deprecated symbols section and retarget internal usage to H5Lget_info()/H5Oget_info(). Misc. other code cleanups... Tested on: FreeBSD/32 6.2 (duty) FreeBSD/64 6.2 (liberty) Linux/32 2.6 (kagiso) Linux/64 2.6 (smirom) AIX/32 5.3 (copper) Solaris/32 2.10 (linew) Mac OS X/32 10.4.10 (amazon)
* [svn-r13253] Updated all C and C++ style source code files with the THG ↵Albert Cheng2007-02-071-2/+3
| | | | | | | | | copyright notice. Tested platform: Kagiso only since it is only a comment block change. If it works in one machine, it should work in all, I hope. Still need to check the parallel build on copper.
* [svn-r12440] Purpose:Quincey Koziol2006-06-271-25/+25
| | | | | | | | | | | Code cleanup Description: Trim trailing whitespace in Makefile.am and C/C++ source files to make diffing changes easier. Platforms tested: None necessary, whitespace only change
* [svn-r12314] Purpose:Quincey Koziol2006-04-291-42/+40
| | | | | | | | | | | | | | Code cleanup Description: Big clean up on high-level library code, including: - Removing include of HDF5 private header file(!) - Reduce number of compiler warnings - Untangle header files, etc. Platforms tested: FreeBSD 4.11 (sleipnir) w/C++ Linux 2.4/64 (mir) w/C++ & FORTRAN
* [svn-r12254] Purpose:Quincey Koziol2006-04-141-8/+10
| | | | | | | | | | | | | | Anti-feature Description: Revert changes to H5G_stat_t struct, to make it compatible with the 1.6.x branch again. The information that was added to the H5G_stat_t struct will be reported through other API routines. Platforms tested: FreeBSD 4.11 (sleipnir) w/C++ Linux 2.4/64 (mir) w/C++ & Fortran Solaris 2.9 (shanti)
* [svn-r12229] Purpose:Pedro Vicente Nunes2006-04-121-14/+13
| | | | | | | | | | | | | | | | | | | | | new feature Description: 1) separated the HL library into "public" and "private" header files, with the same caracteristics as the basic library 2) added the public headers to hdf5.h (with a conditional include macro, defined in configure.in) 3) added the path to HL in all Makefile.am 's , because of the inclusion in hdf5.h Solution: Platforms tested: linux 32, 64 AIX solaris with fortran and c++ (one packet table example fails) Misc. update:
* [svn-r11740] Purpose:Pedro Vicente Nunes2005-11-181-12/+14
| | | | | | | | | | | | | | code check Description: changed the name of a few variables that were misleading in its name Solution: Platforms tested: linux Misc. update:
* [svn-r11708] Purpose:Pedro Vicente Nunes2005-11-111-394/+404
| | | | | | | | | | | | | | | documentation improve Description: added a more complete description for each function in the comments that matches the online documentation Solution: Platforms tested: solaris Misc. update:
* [svn-r11537] Purpose:Albert Cheng2005-10-111-11/+13
| | | | | | Fixed the copyright notice. Tested in heping.
* [svn-r11304] Purpose:Quincey Koziol2005-08-271-8/+8
| | | | | | | | | | | | | Code cleanup (sorta) Description: Adjust H5G_stat_t to prepare for eventual "external link" information. This also puts the information for regular objects and soft links into more obviously separate places. Platforms tested: FreeBSD 4.11 (sleipnir) Linux 2.4
* [svn-r11245] Purpose:Quincey Koziol2005-08-131-257/+257
| | | | | | | | | | | | | | | | | | | | Code cleanup Description: Trim trailing whitespace, which is making 'diff'ing the two branches difficult. Solution: Ran this script in each directory: foreach f (*.[ch] *.cpp) sed 's/[[:blank:]]*$//' $f > sed.out && mv sed.out $f end Platforms tested: FreeBSD 4.11 (sleipnir) Too minor to require h5committest
* [svn-r11091] Purpose:Pedro Vicente Nunes2005-07-201-20/+20
| | | | | | | | | | | | | | | improve documentation by comments changed the return value of H5DSget_num_scales for a case when there are no scales attached from FAIL (-1) to 0 Description: Solution: Platforms tested: linux solaris Misc. update:
* [svn-r10446] Purpose:Pedro Vicente Nunes2005-03-251-3/+7
| | | | | | | | | | | | | | | | | new test Description: added a test for a scalar scale Solution: Platforms tested: linux solaris AIX IRIX Misc. update:
* [svn-r10416] Purpose:Pedro Vicente Nunes2005-03-251-28/+29
| | | | | | | | | | | | | | | | | | | bug fix Description: when exiting due to a error condition on the goto out instruction there was an attempt to call H5Dvlen_reclaim without checking if buf was null Solution: checket it Platforms tested: linux solaris IRIX64 AIX Misc. update:
* [svn-r10403] Purpose:Pedro Vicente Nunes2005-03-241-8/+30
| | | | | | | | | | | | | | | | | new tests bug fix Description: added new tests for iterartors (on many scales, on group, on deleted scales) fixed a bug on H5DSiterate, an ID of the referenced scale was not being closed on the cycle Solution: Platforms tested: linux solaris Misc. update:
* [svn-r10386] Purpose:Pedro Vicente Nunes2005-03-231-3/+0
| | | | | | | | | | | | | | new tests for is scale, detach scales and iterate scales Description: Solution: Platforms tested: linux solaris Misc. update:
* [svn-r10248] Purpose:Pedro Vicente Nunes2005-03-211-0/+86
| | | | | | | | | | | | | | | | new tests for DS Description: add a test that ckecks if a scale being attached itself has scales attached (error) add a test for the dataset being attached to is a reserved High Level class (image, palette, table) Solution: Platforms tested: linux solaris Misc. update:
* [svn-r10111] Purpose:Robert E. McGrath2005-03-011-2/+2
| | | | | | | | | | | | | | | | bug fix Description: H5DSget_scale_name, H5DSget_lablel return wrong value when truncating to 1 character array. Solution: fix it. Platforms tested: verbena,shanti,copper Misc. update:
* [svn-r10090] Purpose:Robert E. McGrath2005-02-251-9/+9
| | | | | | | | | | | | | | | change Description: Minor changes to Dimension Scale API, to bring into alignment with specification. Solution: Platforms tested: verbena,shanti, copper64 Misc. update:
* [svn-r10068] Purpose:Pedro Vicente Nunes2005-02-231-1/+4
| | | | | | | | | | | | | | news tests (test several ranks for the datasets and scales ) Description: Solution: Platforms tested: linux solaris Misc. update:
* [svn-r10065] Purpose:Pedro Vicente Nunes2005-02-221-57/+63
| | | | | | | | | | | | | | | new definition for H5DSget_label (returns the label size) more tests Description: Solution: Platforms tested: linux solaris Misc. update:
* [svn-r10063] Purpose:Pedro Vicente Nunes2005-02-221-21/+78
| | | | | | | | | | | | | | | changed the function H5DSget_scale_name to return the size of the name buffer added a test Description: Solution: Platforms tested: linux solaris Misc. update:
* [svn-r10059] Purpose:Pedro Vicente Nunes2005-02-211-38/+103
| | | | | | | | | | | | | | | new features Description: added a new function H5DS_is_attached, more tests Solution: Platforms tested: linux solaris Misc. update:
* [svn-r10043] Purpose:Pedro Vicente Nunes2005-02-181-22/+355
| | | | | | | | | | | | | | | | | new features/tests Description: add a couple more tests for the attach/detach pair of functions together with a verify function for it (used only for test purposes) avoided addind the same scale twice to a dimension all seems to be working very well Solution: Platforms tested: linux solaris Misc. update:
* [svn-r10042] Purpose:Pedro Vicente Nunes2005-02-181-4/+12
| | | | | | | | | | | | | | | | bug fix Description: a scale was deleted twice in one case Solution: check for the dimension , delete only for it Platforms tested: linux solaris Misc. update:
* [svn-r10037] Purpose:Pedro Vicente Nunes2005-02-171-8/+122
| | | | | | | | | | | | | | more error checking tests Description: Solution: Platforms tested: linux solaris Misc. update:
* [svn-r10032] Purpose:Pedro Vicente Nunes2005-02-171-186/+151
| | | | | | | | | | | | | | | new tests Description: added a new test for detach_scale, some minor changes in the the detach_scale function Solution: Platforms tested: linux solaris Misc. update:
* [svn-r10019] Purpose:Pedro Vicente Nunes2005-02-161-282/+8
| | | | | | | | | | | | | | | | new test Description: added a test for the iterator function that verifies if the scales meet certain criteria (having the same size as the dimensions sizes of the dataset and they are not empty datasets) Solution: Platforms tested: linux solaris Misc. update: