Network

 
AuthorCommitMessageCommit Date
Vailin ChoiVailin Choi
36bb80baebd[svn-r20675] Fix for coverity #1714 due to the fix for #810. Use HDfree() instead of H5MM_xfree().
Allen ByrneAllen Byrne
68730696767[svn-r20668] Add test_configuration setting
Allen ByrneAllen Byrne
3444fc118a5[svn-r20664] Remove extraneous memcheck parameters
Gerd HeberGerd Heber
02614dec682[svn-r20614] Use HDsnprintf. --gh
Larry KnoxLarry Knox
b470fed3b54[svn-r20612] Changed string functions to versions with string length to fix coverity issues 922, 942 and 943. Teted on jam.
Gerd HeberGerd Heber
bcda15ae15c[svn-r20611] Use HDstrncpy. --gh
Gerd HeberGerd Heber
8ce468af77c[svn-r20609] Use HDstrncpy and HDstrncat. --gh
Larry KnoxLarry Knox
99b31e64185[svn-r20607] Changed string function calls to use versions that specify the string length to fix coverity issues 832 and 839. Tested on jam.
Neil FortnerNeil Fortner
7a8ac70ecb6[svn-r20606] Purpose: Fix coverity issue 807 Description: Modified long_compact in stab.c to keep track of whether objname is allocated by setting it to NULL when it is not. Added code to free objname on error if it is not NULL. Tested: Fedora
Gerd HeberGerd Heber
fde41d30512[svn-r20605] Use HDstrncpy and HDstrncat. --gh
Gerd HeberGerd Heber
3e1c77db9fd[svn-r20604] Use HDstrncpy. --gh
Neil FortnerNeil Fortner
ea7e3799c51[svn-r20603] Purpose: Fix coverity issues 808-809 Description: Modified test_core in vfd.c to check the returns from malloc, and keep track of whether points and check are allocated by setting them to NULL when they are not. Added code to free points and check on error if they are not NULL. Also fixed unrelated warnings in vfd.c. Tested: Fedora
Gerd HeberGerd Heber
05a07a1dbf3[svn-r20602] Use HDsnprintf and HDstrncat --gh
Neil FortnerNeil Fortner
1422f5bc125[svn-r20601] Purpose: Fix coverity issues 1703-1705 Description: Modified the cleanup code in test_free in accum.c to reset allocated buffers to NULL after they are freed, and modified the error cleanup code to check if these buffers are NULL before freeing them. Also fixed some unrelated warnings in accum.c. Tested: Fedora
Neil FortnerNeil Fortner
886f9727323[svn-r20528] Purpose: Fix coverity issue 1372 Description: Rewrite file open secition of H5FD_stdio_open to avoid TOCTUO condition. No longer calls access, and uses a tentative open in "rb" or "rb+" mode to check for existence of the file. Tested: Fedora
Gerd HeberGerd Heber
8ff955cdf6e[svn-r20524] Check H5Z_SZIP->encoder_present < 1 assuming 0 represents absence. --gh
Peter CaoPeter Cao
a5fbfbaa866[svn-r20523] fixed coverity issues 68, 1120, 1116i
John MainzerJohn Mainzer
94a04f0a584[svn-r20522] Addressed coverity issues 930-933, 850, 836, 835, 1307. All minor potential buffer overwrite bugs, or coverity errors. Fixed by replacing strcpy and sprintf with strncpy and snprintf. Tested on wren.
Gerd HeberGerd Heber
c89d53ba806[svn-r20516] Added else branch to the if (ret_value < 0) check. --gh
Gerd HeberGerd Heber
966e33e61fa[svn-r20514] Initialize h5fid = -1 and predicate HDclose call on h5fid != -1 --gh
Gerd HeberGerd Heber
2ece1fae1fb[svn-r20512] Initialize ifid = -1 and predicate HDclose call on ifid != -1 --gh
Neil FortnerNeil Fortner
9b3da7dd207[svn-r20511] Purpose: Fix coverity issue 1715 Description: Free "file" and nested data on failure in H5FD_core_open. Tested: Fedora
Gerd HeberGerd Heber
494dc89b76e[svn-r20510] Initialize ufid = -1 and predicate HDclose call on ufid != -1 --gh
Allen ByrneAllen Byrne
00bdb46f5a0[svn-r20457] Coverity issue 691: return of H5duo could be negative. Fixed by using STDOUT_FILENO and redesign parse_command_line and main to cleanup file allocations. The output_file var is null when using stdout. In cleanup do not close output_file if NULL. Tested: local linux
Peter CaoPeter Cao
b81336db9ef[svn-r20456] fixed coverity 678, 679
Neil FortnerNeil Fortner
f78da50bec6[svn-r20455] Purpose: Cleanup fix for coverity issue 816 Description: Removed redundant check for "fa" in H5FD_core_open. Tested: Fedora
Neil FortnerNeil Fortner
c74a8ad3ddf[svn-r20454] Purpose: Fix coverity issue 816 Description: Modified various file drivers to always check the return value from H5P_get_driver_info. Tested: Fedora
Peter CaoPeter Cao
61350ed450a[svn-r20451] Fixed coverity 810
Peter CaoPeter Cao
b7a29a17145[svn-r20450] fixed coverity 813
Neil FortnerNeil Fortner
1e5622ce2e6[svn-r20449] Purpose: Fix coverity issue 82 Description: Modified H5E_walk2_cb to check return value of H5I_object_verify. Tested: Fedora
Jonathan KimJonathan Kim
607c10e607c[svn-r20414] This is related to the previous checkin r20399. There were incorrect updates which caused incorrect behavior when no file was given. Also possible segfault when handling hyperslab options. Simplify the code changes.
Larry KnoxLarry Knox
3e193cb378f[svn-r20402] Added #includes for h5tools.y or h5tools_utils.h as required to remedy implicit function declarations which caused compiler warnings and coverity issues 703-4 and 708-11. Tested on jam.
Gerd HeberGerd Heber
f0c6b9a212f[svn-r20401] Added free(src_sizes) and free(src_offset). --gh
Neil FortnerNeil Fortner
27e145fb4bc[svn-r20400] Purpose: Fix coverity issue 592 Description: Added void cast to call to H5F_sfile_assert_num() in H5F_term)interface(). There is no need to check the return value of this function in general, and there would be nothing to do even if it did fail. Tested: Fedora
Allen ByrneAllen Byrne
3bacf1a72b7[svn-r20399] Change parse_command_line to free hand structure on error and only "leave()" at end of main. Tested by: JKM
Gerd HeberGerd Heber
e0229ea4813[svn-r20398] Added free(src_offset). --gh
Gerd HeberGerd Heber
c7ac1ea7472[svn-r20397] Added free(src_sizes) --gh
Gerd HeberGerd Heber
26234fc6541[svn-r20396] Should be H5MM_calloc(count * sizeof(void*)). Fixed. --gh
Gerd HeberGerd Heber
664d90224a4[svn-r20395] Should be H5MM_calloc(count * sizeof(void*)). Fixed --gh
Neil FortnerNeil Fortner
66a500271cf[svn-r20394] Purpose: Fix coverity issues 1674 and 1675 Description: Modified H5Epush2() and H5E_printf_stack() to keep track of whether va_start() has been called, and to call va_end() at the end if so. Tested: Fedora
Gerd HeberGerd Heber
69009f5c674[svn-r20393] Move initialization ocrt_info.new_obj = NULL; before FUNC_ENTER_NOAPI -- gh
Neil FortnerNeil Fortner
39ef9f07c6b[svn-r20392] Purpose: Fix coverity issues 1677 and 1678 Description: Added return value checking for 4 funciton calls in H5tools_get_symlink_info(). Tested: Fedora
Allen ByrneAllen Byrne
12a4d5640a9[svn-r20360] Remove testfiles not in coverity branch
Allen ByrneAllen Byrne
534b3460cbd[svn-r20356] Remove executable file
Allen ByrneAllen Byrne
44c2c55860d[svn-r20352] Update Valgrind ignore list
Larry KnoxLarry Knox
7151d3ef511[svn-r20348] Addressed Coverity issue 705 - added prototype for resize_configs_are_equal to cache_common.h.
Neil FortnerNeil Fortner
c7b655e119f[svn-r20347] Purpose: Fix memory leak in overhead.c Description: Fixed leak in test() in overhead.c where array "had" was never freed. Tested: Fedora (valgrind)
Allen ByrneAllen Byrne
bd683abe175[svn-r20346] Merge cmake changes from 1.8 to coverity to correct valgrind issues already fixed in trunk/branch
Neil FortnerNeil Fortner
6f3c8cad180[svn-r20345] Purpose: Fix valgrind issues in tvlstr.cpp and H5CommonFG.cpp Description: Fixed bug in test_vlstring_dataset() where a c string was created and used without a null terminator, potentially causing memory errors. Fixed leak in H5::CommonFG::getComment() where a temporary variable would not be freed in case of error. Tested: Fedora
Gerd HeberGerd Heber
523725e620b[svn-r20344] Added initialization udata.path = NULL; to prevent uninitialized used after done: label