gnucash maint: Revert "Bug 740471 - Applying payment to invoice Segmentation fault"
Geert Janssens
gjanssens at code.gnucash.org
Mon Dec 29 09:35:00 EST 2014
Updated via https://github.com/Gnucash/gnucash/commit/ad09bcfd (commit)
from https://github.com/Gnucash/gnucash/commit/d5ade4cb (commit)
commit ad09bcfd4a3c32288b1ad28dccb2bf31683a5be0
Author: Geert Janssens <janssens-geert at telenet.be>
Date: Mon Dec 29 16:30:04 2014 +0100
Revert "Bug 740471 - Applying payment to invoice Segmentation fault"
This reverts commit a6d7184d830d8a8bc0c339d012ac17c249a40994.
The patch had no effect on the bug at all and needlessly clutters the code.
diff --git a/src/engine/gncOwner.c b/src/engine/gncOwner.c
index 70f41db..25fa682 100644
--- a/src/engine/gncOwner.c
+++ b/src/engine/gncOwner.c
@@ -1232,11 +1232,12 @@ void gncOwnerAutoApplyPaymentsWithLots (const GncOwner *owner, GList *lots)
* Note that due to the iterative nature of this function lots
* in the list may become empty/closed before they are evaluated as
* base lot, so we should check this for each lot. */
- if (!left_lot || qof_instance_get_destroying (left_lot))
+ if (!left_lot)
continue;
if (gnc_lot_count_splits (left_lot) == 0)
{
gnc_lot_destroy (left_lot);
+ left_iter->data = NULL;
continue;
}
if (gnc_lot_is_closed (left_lot))
@@ -1262,11 +1263,12 @@ void gncOwnerAutoApplyPaymentsWithLots (const GncOwner *owner, GList *lots)
* Note that due to the iterative nature of this function lots
* in the list may become empty/closed before they are evaluated as
* base lot, so we should check this for each lot. */
- if (!right_lot || qof_instance_get_destroying (right_lot))
+ if (!right_lot)
continue;
if (gnc_lot_count_splits (right_lot) == 0)
{
gnc_lot_destroy (right_lot);
+ right_iter->data = NULL;
continue;
}
if (gnc_lot_is_closed (right_lot))
Summary of changes:
src/engine/gncOwner.c | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
More information about the gnucash-changes
mailing list