[Gnucash-changes] r13683 - gnucash/trunk/src/register/ledger-core - Avoid infinite loop in gnc_find_split_in_trans_by_memo(). Bug #335465

[Chris Shoemaker]

Author: chris
Date: 2006-03-22 19:33:11 -0500 (Wed, 22 Mar 2006)
New Revision: 13683
Trac: http://svn.gnucash.org/trac/changeset/13683

Modified:
   gnucash/trunk/src/register/ledger-core/split-register-control.c
Log:
   Avoid infinite loop in gnc_find_split_in_trans_by_memo(). Bug #335465
   Reported by Mike Alexander.


Modified: gnucash/trunk/src/register/ledger-core/split-register-control.c
===================================================================
--- gnucash/trunk/src/register/ledger-core/split-register-control.c	2006-03-22 09:49:19 UTC (rev 13682)
+++ gnucash/trunk/src/register/ledger-core/split-register-control.c	2006-03-23 00:33:11 UTC (rev 13683)
@@ -461,6 +461,7 @@
   Split *split;
 
   while ((split = xaccTransGetSplit(trans, i)) != NULL) {
+    i++;
     if (unit_price)
     {
       gnc_numeric price = xaccSplitGetSharePrice (split);
@@ -470,7 +471,6 @@
 
     if (safe_strcmp (memo, xaccSplitGetMemo (split)) == 0)
       return split;
-    i++;
   }
 
   return NULL;