crash in hdf_copy if src==NULL

[GoogleCodeExporter]

patch:

=====
static NEOERR * _copy_nodes (HDF *dest, HDF *src)
{
  NEOERR *err = STATUS_OK;
  HDF *dt, *st;
  HDF_ATTR *attr_copy;

+  if (src == NULL)
+    return STATUS_OK;
  st = src->child;
  while (st != NULL)
  {
    err = _copy_attr(&attr_copy, st->attr);
=====

Original issue reported on code.google.com by [email protected] on 26 Sep 2012 at 11:48

[GoogleCodeExporter]

[deleted comment]

[GoogleCodeExporter]

I think this is an error of end-user, not of clearsilver.

Documentation for hdf_copy says:
"hdf_copy is a deep copy of an HDF tree pointed to by
src to the named node of dest. dest and src need not be
part of the same data set"

there isnt something like "if src is NULL, ..."

Original comment by [email protected] on 8 Dec 2012 at 8:08

[GoogleCodeExporter]

If all other functions, if src is NULL - function does nothing and returns, and 
only in hdf_copy application crashed in NULL pointer dereference handler.
I think it should be consistent with other functions - if source is NULL, do 
nothing.

Original comment by [email protected] on 14 Dec 2012 at 7:36