Question: gda-clnt/gda-recordset.c

[Stephan Heinze <Stephan Heinze xcom de>]

Hi all,

am I wrong or should there be a check for NULL pointer in
gda-clnt/gda-recordset.c like

void
gda_recordset_close(Gda_Recordset* rs)
{
  if (NULL==rs)
  {
    g_error("invalid Gda_Recordset* in gda_recordset_close\n");
    return;
  }

  [snip code]
}

in every function like this one before accessing the stuctures member?

IMHO the check rs->open would dump a core if rs is NULL.




kind regards

stephan

-------------------------------------------------------------------
"Life is what happens to you while you're busy making other plans."
                                                        John Lennon
-------------------------------------------------------------------