gnucash stable: Multiple changes pushed

John Ralls jralls at code.gnucash.org
Tue Feb 20 19:12:00 EST 2024


Updated	 via  https://github.com/Gnucash/gnucash/commit/226bfea1 (commit)
	 via  https://github.com/Gnucash/gnucash/commit/7bd97f15 (commit)
	from  https://github.com/Gnucash/gnucash/commit/8546aa97 (commit)



commit 226bfea1084646328dd3022cceee7271eb6e4ee5
Author: John Ralls <jralls at ceridwen.us>
Date:   Tue Feb 20 15:33:11 2024 -0800

    Fix a bunch of UB errors from ASAN about mismatched function types.
    
    The casts fool the compiler but not the UB sanitizer.

commit 7bd97f15d0fdbd53e752c173d616bf032810e57b
Author: John Ralls <jralls at ceridwen.us>
Date:   Tue Feb 20 15:10:05 2024 -0800

    Fix transaction delete use-after-free, take 2.
    
    The problem with take 1 was that the duplicate split uses the same lot
    and account pointers without adding itself to those lists, causing
    checks in unit tests to fail.



Summary of changes:
 libgnucash/engine/Split.cpp                  | 22 +++++++++++++++++--
 libgnucash/engine/Transaction.cpp            | 20 +++++++++--------
 libgnucash/engine/TransactionP.h             |  6 ++---
 libgnucash/engine/gnc-lot.cpp                |  2 +-
 libgnucash/engine/test/utest-Account.cpp     | 33 ++--------------------------
 libgnucash/engine/test/utest-Split.cpp       |  2 +-
 libgnucash/engine/test/utest-Transaction.cpp |  8 +++----
 po/POTFILES.in                               |  1 +
 8 files changed, 43 insertions(+), 51 deletions(-)



More information about the gnucash-patches mailing list