diff options
| author | Quincey Koziol <koziol@hdfgroup.org> | 2005-07-17 02:46:42 (GMT) |
|---|---|---|
| committer | Quincey Koziol <koziol@hdfgroup.org> | 2005-07-17 02:46:42 (GMT) |
| commit | 96a49c0d37ea736422c2cffe4bd22f0209e4fe24 (patch) | |
| tree | 1be71f9a8081383666c91a3a5837c064fa7aaa59 /src/H5Odtype.c | |
| parent | 5e98b5bb36c9e0b1078e56d9b734657fcfe7a03d (diff) | |
| download | hdf5-96a49c0d37ea736422c2cffe4bd22f0209e4fe24.zip hdf5-96a49c0d37ea736422c2cffe4bd22f0209e4fe24.tar.gz hdf5-96a49c0d37ea736422c2cffe4bd22f0209e4fe24.tar.bz2 | |
[svn-r11078] Purpose:
Bug fix
Description:
Correct memory leak when a dataset is attempted to be opened, but turns
out to be a group or named datatype.
Also, clean up code that was leading to the leak and zero out empty
group entries to help prevent similar errors in the future.
Platforms tested:
FreebSD 4.11 (sleipnir)
Too minor to require h5committest
Diffstat (limited to 'src/H5Odtype.c')
| -rw-r--r-- | src/H5Odtype.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/src/H5Odtype.c b/src/H5Odtype.c index 02dac26..646a9b1 100644 --- a/src/H5Odtype.c +++ b/src/H5Odtype.c @@ -1182,12 +1182,8 @@ H5O_dtype_set_share (H5F_t UNUSED *f, void *_mesg/*in,out*/, assert (sh); assert (!sh->in_gh); - /* Shallow copy the symbol table entry */ - H5G_ent_copy(&(dt->ent),&(sh->u.ent),H5G_COPY_SHALLOW); - - /* Reset the names of the copied symbol table entry */ - dt->ent.user_path_r = NULL; - dt->ent.canon_path_r = NULL; + /* NULL copy here, names not appropriate */ + H5G_ent_copy(&(dt->ent),&(sh->u.ent),H5G_COPY_NULL); /* Note that the datatype is a named datatype */ dt->shared->state = H5T_STATE_NAMED; |