View Issue Details

IDProjectCategoryView StatusLast Update
0005187Talermechant backendpublic2024-01-12 14:04
ReporterFlorian Dold Assigned ToChristian Grothoff  
PriorityhighSeveritymajorReproducibilityhave not tried
Status closedResolutionfixed 
Product Version0.4 
Target Version0.5Fixed in Version0.5 
Summary0005187: fix uninitialized memory accesses in merchant (refunds)
DescriptionLooks like in refunds, we access uninitialized memory.

Maybe we should include an option / environment variable to run the tests under valgrind.
Steps To ReproduceChange test_merchant_api so it runs the merchant under valgrind.
Additional Information==9942== Conditional jump or move depends on uninitialised value(s)
==9942== at 0x4C3014E: memcpy@GLIBC_2.2.5 (vg_replace_strmem.c:1017)
==9942== by 0x6C08AB9: ??? (in /lib/x86_64-linux-gnu/libgcrypt.so.20.1.6)
==9942== by 0x6B4E588: gcry_mpi_print (in /lib/x86_64-linux-gnu/libgcrypt.so.20.1.6)
==9942== by 0x60C7ACC: GNUNET_CRYPTO_mpi_print_unsigned (crypto_mpi.c:103)
==9942== by 0x60C13DB: GNUNET_CRYPTO_eddsa_sign (crypto_ecc.c:878)
==9942== by 0x12322B: MH_handler_refund_increase (taler-merchant-httpd_refund.c:271)
==9942== by 0x10D24F: url_handler (taler-merchant-httpd.c:279)
==9942== by 0x586EAAE: call_connection_handler (connection.c:2281)
==9942== by 0x586FDEF: MHD_connection_handle_idle (connection.c:3483)
==9942== by 0x5871AF0: call_handlers (daemon.c:1113)
==9942== by 0x5873F34: internal_run_from_select (daemon.c:3256)
==9942== by 0x58759A4: MHD_select (daemon.c:3517)
TagsNo tags attached.

Activities

Christian Grothoff

2017-11-30 11:22

manager   ~0012619

Fixed in d5cde48..78f0bae

Related Changesets

merchant: master 78f0bae5

2017-11-30 12:22

Christian Grothoff


Details Diff
fix 0005187 Affected Issues
0005187
mod - src/backend/taler-merchant-httpd_refund.c Diff File

Issue History

Date Modified Username Field Change
2017-11-30 03:01 Florian Dold New Issue
2017-11-30 03:01 Florian Dold Status new => assigned
2017-11-30 03:01 Florian Dold Assigned To => Christian Grothoff
2017-11-30 11:22 Christian Grothoff Status assigned => resolved
2017-11-30 11:22 Christian Grothoff Resolution open => fixed
2017-11-30 11:22 Christian Grothoff Fixed in Version => 0.5
2017-11-30 11:22 Christian Grothoff Note Added: 0012619
2017-11-30 11:22 Christian Grothoff Product Version => 0.4
2017-11-30 11:22 Christian Grothoff Target Version => 0.5
2018-04-15 20:34 Christian Grothoff Status resolved => closed
2021-09-02 18:22 Christian Grothoff Changeset attached => Taler-merchant master 78f0bae5
2024-01-12 14:04 Christian Grothoff Category merchant backend API (C) => mechant backend