gnucash maint: [test-commodity-utils] Don't try to free the pricedb twice.

John Ralls jralls at code.gnucash.org
Mon Aug 5 13:05:02 EDT 2019


Updated	 via  https://github.com/Gnucash/gnucash/commit/bbcfc9da (commit)
	from  https://github.com/Gnucash/gnucash/commit/6252de21 (commit)



commit bbcfc9da42e2504d3d21e8dbb08cc2be515b2140
Author: John Ralls <jralls at ceridwen.us>
Date:   Mon Aug 5 10:01:04 2019 -0700

    [test-commodity-utils] Don't try to free the pricedb twice.
    
    It's freed by gnc-clear-current-session, so freeing it explicitly
    causes a double-free.

diff --git a/gnucash/report/report-system/test/test-commodity-utils.scm b/gnucash/report/report-system/test/test-commodity-utils.scm
index 3add3ead2..8a316ad3c 100644
--- a/gnucash/report/report-system/test/test-commodity-utils.scm
+++ b/gnucash/report/report-system/test/test-commodity-utils.scm
@@ -213,10 +213,7 @@
 
 
 (define (teardown)
-  (let* ((book  (gnc-get-current-book))
-         (pricedb (gnc-pricedb-get-db book)))
-    (gnc-pricedb-destroy pricedb)
-    (gnc-clear-current-session)))
+  (gnc-clear-current-session))
 
 (define (collect collector shares value)
   ((car collector) 'add shares)



Summary of changes:
 gnucash/report/report-system/test/test-commodity-utils.scm | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)



More information about the gnucash-changes mailing list