View Issue Details

IDProjectCategoryView StatusLast Update
0004482Talermerchant backendpublic2016-10-11 15:28
ReporterFlorian Dold Assigned ToMarcello Stanisci Resolution deadline
ImplementerReviewer 
Release signed byDeployed by 
VerifierCommunicator 
Data protection authority notification requiredNo  
CSIRT notification requiredNo  
Estimated effort (hours)Actual effort (hours)0 
Fast-trackNo
Work packagenone
PriorityurgentSeverityblockReproducibilityhave not tried
Status closedResolutionwon't fix 
Product Version0.0 
Target Version0.1Fixed in Version0.1 
Summary0004482: merchant sometimes crashes with SIGSEVG
DescriptionThe full coredumps are on test-blue@tripwire.

From the stack trace I assume it has to do with the key fetching logic ...
Additional Information#0 strlen () at ../sysdeps/x86_64/strlen.S:106
#1 0x000000000040e6cb in TMH_KS_handler_keys (rh=0x61b068 <handlers+168>, connection=0x7f8ed800a1b0, connection_cls=0x7f8ed800a208,
    upload_data=0x0, upload_data_size=0x7f8ede0b0c58) at taler-exchange-httpd_keystate.c:1002
#2 0x0000000000404370 in handle_mhd_request (cls=0x0, connection=0x7f8ed800a1b0, url=0x7f8ed8000a84 "/keys", method=0x7f8ed8000a80 "GET",
    version=0x7f8ed8000a8a "HTTP/1.0", upload_data=0x0, upload_data_size=0x7f8ede0b0c58, con_cls=0x7f8ed800a208) at taler-exchange-httpd.c:368
#3 0x00007f8ee4a6c1c2 in call_connection_handler (connection=connection@entry=0x7f8ed800a1b0) at connection.c:1613
#4 0x00007f8ee4a6d641 in MHD_connection_handle_idle (connection=0x7f8ed800a1b0) at connection.c:2541
#5 0x00007f8ee4a72ac2 in MHD_run_from_select (daemon=daemon@entry=0x1fc1810, read_fd_set=read_fd_set@entry=0x7f8ede0b0d90,
    write_fd_set=write_fd_set@entry=0x7f8ede0b0e10, except_fd_set=except_fd_set@entry=0x7f8ede0b0e90) at daemon.c:2339
#6 0x00007f8ee4a72d63 in MHD_select (daemon=daemon@entry=0x1fc1810, may_block=<optimized out>, may_block@entry=1) at daemon.c:2482
#7 0x00007f8ee4a72f92 in MHD_select_thread (cls=0x1fc1810) at daemon.c:3043
#8 0x00007f8ee41c8454 in start_thread (arg=0x7f8ede0b1700) at pthread_create.c:334
#9 0x00007f8ee3f05edd in clone () at ../sysdeps/unix/sysv/linux/x86_64/clone.S:109
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

Marcello Stanisci

2016-05-02 12:48

reporter   ~0010607

might not be related, but if the config contains a non-existent exchange and CTRL-C gets pressed after the first failing attempt to get /keys, then the merchant segfaults.

Christian Grothoff

2016-05-03 03:50

manager   ~0010611

You reported this against the merchant, but the stack trace is for the exchange: taler-exchange-httpd_keystate.c:1002 Something is fishy here...

Christian Grothoff

2016-05-03 03:54

manager   ~0010612

Added assertions to help debug this in 025fbdb..eabbafc

Florian Dold

2016-05-03 09:21

manager   ~0010616

Oops, I posted the wrong stack trace. But still, there are also merchant stack traces in there:

test-blue@tripwire:~$ ls *core*
core.libmicrohttpd.23610.tripwire.1461759725 core.taler-merchant-.23613.tripwire.1461759760
core.taler-merchant-.15342.tripwire.1462263261 core.taler-merchant-.3716.tripwire.1462207199

Christian Grothoff

2016-05-31 06:39

manager   ~0010799

Is this issue still current?

Marcello Stanisci

2016-06-21 14:57

reporter   ~0010923

That was impossible to reproduce, and might also have "accidentally" fixed. So I suggest to close, and if the problem is still there, it will come back. Ok?

Issue History

Date ModifiedUsernameFieldChange
2016-05-02 11:18Florian DoldNew Issue
2016-05-02 11:18Florian DoldStatusnew => assigned
2016-05-02 11:18Florian DoldAssigned To => Marcello Stanisci
2016-05-02 12:48Marcello StanisciNote Added: 0010607
2016-05-03 03:50Christian GrothoffNote Added: 0010611
2016-05-03 03:54Christian GrothoffNote Added: 0010612
2016-05-03 09:21Florian DoldNote Added: 0010616
2016-05-05 13:24Christian GrothoffProduct Version => 0.0
2016-05-05 13:24Christian GrothoffTarget Version => 0.1
2016-05-31 06:39Christian GrothoffNote Added: 0010799
2016-05-31 06:39Christian GrothoffStatusassigned => feedback
2016-06-21 14:57Marcello StanisciNote Added: 0010923
2016-06-21 14:57Marcello StanisciStatusfeedback => informing
2016-06-21 14:57Marcello StanisciResolutionopen => won't fix
2016-06-21 18:24Christian GrothoffFixed in Version => 0.1
2016-10-11 15:28Christian GrothoffStatusinforming => closed
2025-03-22 13:09Christian GrothoffCategorymechant backend => merchant backend