r17917 - gnucash/trunk/src/register/ledger-core - Register: Add a few safety checks to the register loading routines.

[Charles Day]

Author: cedayiv
Date: 2009-02-14 19:38:15 -0500 (Sat, 14 Feb 2009)
New Revision: 17917
Trac: http://svn.gnucash.org/trac/changeset/17917

Modified:
   gnucash/trunk/src/register/ledger-core/split-register-load.c
Log:
Register: Add a few safety checks to the register loading routines.


Modified: gnucash/trunk/src/register/ledger-core/split-register-load.c
===================================================================
--- gnucash/trunk/src/register/ledger-core/split-register-load.c	2009-02-14 23:22:18 UTC (rev 17916)
+++ gnucash/trunk/src/register/ledger-core/split-register-load.c	2009-02-15 00:38:15 UTC (rev 17917)
@@ -161,6 +161,9 @@
 {
   GList *node;
 
+  g_return_if_fail(reg);
+  g_return_if_fail(vcell_loc);
+
   if (split == find_split)
     *new_split_row = vcell_loc->virt_row;
 
@@ -271,6 +274,12 @@
   int new_split_row = -1;
   time_t present;
 
+  g_return_if_fail(reg);
+  table = reg->table;
+  g_return_if_fail(table);
+  info = gnc_split_register_get_info (reg);
+  g_return_if_fail(info);
+
   ENTER("reg=%p, slist=%p, default_account=%p", reg, slist, default_account);
 
   blank_split = xaccSplitLookup (&info->blank_split_guid,
@@ -358,8 +367,6 @@
 
   info->default_account = *xaccAccountGetGUID (default_account);
 
-  table = reg->table;
-
   // gnc_table_leave_update (table, table->current_cursor_loc);
 
   multi_line = (reg->style == REG_STYLE_JOURNAL);