View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0004109 | Taler | merchant backend | public | 2015-12-25 14:39 | 2016-05-26 12:45 |
| Reporter | Florian Dold | Assigned To | Marcello Stanisci | Resolution deadline | |
| Implementer | Reviewer | ||||
| Release signed by | Deployed by | ||||
| Verifier | Communicator | ||||
| Data protection authority notification required | No | ||||
| CSIRT notification required | No | ||||
| Estimated effort (hours) | Actual effort (hours) | 0 | |||
| Fast-track | No | ||||
| Work package | none | ||||
| Priority | urgent | Severity | crash | Reproducibility | have not tried |
| Status | closed | Resolution | no change required | ||
| Product Version | 0.0 | ||||
| Target Version | 0.0 | Fixed in Version | 0.0 | ||
| Summary | 0004109: requesting invalid /pay twice causes merchant to crash | ||||
| Description | When refreshing the failed payment page in the wallet twice (which requests /pay from the merchant), the merchant crashes (see Additional Information). | ||||
| Additional Information | Dec 25 15:35:33-841821 taler-merchant-httpd-2745 DEBUG In mint context polling task, max_fd=11, timeout=1 Dec 25 15:35:33-845817 util-scheduler-2745 ERROR `select' failed at scheduler.c:799 with error: Bad file descriptor Dec 25 15:35:33-846644 taler-merchant-httpd-2745 ERROR Assertion failed at scheduler.c:855. ==2745== ==2745== Process terminating with default action of signal 6 (SIGABRT) ==2745== at 0x76BE5F8: raise (in /usr/lib/libc-2.22.so) ==2745== by 0x76BFA79: abort (in /usr/lib/libc-2.22.so) ==2745== by 0x628D988: GNUNET_abort_ (common_logging.c:290) ==2745== by 0x62BE7B3: GNUNET_SCHEDULER_run (scheduler.c:733) ==2745== by 0x62B9C2D: GNUNET_PROGRAM_run2 (program.c:302) ==2745== by 0x62B9F0E: GNUNET_PROGRAM_run (program.c:341) ==2745== by 0x403C54: main (taler-merchant-httpd.c:586) ==2745== ==2745== HEAP SUMMARY: ==2745== in use at exit: 388,206 bytes in 4,059 blocks ==2745== total heap usage: 382,748 allocs, 378,689 frees, 12,317,490 bytes allocated ==2745== ==2745== LEAK SUMMARY: ==2745== definitely lost: 968 bytes in 7 blocks ==2745== indirectly lost: 38,208 bytes in 388 blocks ==2745== possibly lost: 1,086 bytes in 32 blocks ==2745== still reachable: 347,944 bytes in 3,632 blocks ==2745== suppressed: 0 bytes in 0 blocks ==2745== Rerun with --leak-check=full to see details of leaked memory ==2745== ==2745== For counts of detected and suppressed errors, rerun with: -v ==2745== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0) Aborted (core dumped) | ||||
| Tags | No tags attached. | ||||
| related to | 0004070 | closed | Marcello Stanisci | need unit testcases for libtalermerchantdb |
| Probably related to the memory corruption from 0004110, seems to be fixed now. | |
| Okay, this is not gone yet, I got then crash again ... | |
| Hmm. Seems most likely that somehow we close the FD of the connection, but do not properly update the scheduler's event loop, so that one still has the old socket in the select()-set, which triggers the assertion. | |
| As this hasn't been reproduced, I'm re-assigning to Marcello, as this falls into testing the merchant (and thus is a bit with 0004070). | |
| Seems fixed for real. | |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2015-12-25 14:39 | Florian Dold | New Issue | |
| 2015-12-25 14:39 | Florian Dold | Status | new => assigned |
| 2015-12-25 14:39 | Florian Dold | Assigned To | => Marcello Stanisci |
| 2015-12-25 15:10 | Florian Dold | Note Added: 0010037 | |
| 2015-12-25 15:10 | Florian Dold | Status | assigned => informing |
| 2015-12-25 15:10 | Florian Dold | Resolution | open => no change required |
| 2015-12-25 15:10 | Florian Dold | Assigned To | Marcello Stanisci => Florian Dold |
| 2015-12-25 15:35 | Florian Dold | Note Added: 0010038 | |
| 2015-12-25 15:35 | Florian Dold | Status | informing => new |
| 2015-12-25 18:21 | Christian Grothoff | Note Added: 0010039 | |
| 2015-12-25 18:22 | Christian Grothoff | Status | new => acknowledged |
| 2015-12-25 18:22 | Christian Grothoff | Product Version | => 0.0 |
| 2015-12-25 18:22 | Christian Grothoff | Target Version | => 0.0 |
| 2016-01-22 22:53 | Christian Grothoff | Target Version | 0.0 => 0.1 |
| 2016-03-19 18:43 | Christian Grothoff | Assigned To | Florian Dold => Marcello Stanisci |
| 2016-03-19 18:43 | Christian Grothoff | Status | acknowledged => assigned |
| 2016-03-19 18:44 | Christian Grothoff | Note Added: 0010253 | |
| 2016-03-19 18:44 | Christian Grothoff | Relationship added | related to 0004070 |
| 2016-03-20 20:53 | Torsten Grothoff | Issue cloned: 0004255 | |
| 2016-05-26 12:45 | Christian Grothoff | Note Added: 0010765 | |
| 2016-05-26 12:45 | Christian Grothoff | Status | assigned => closed |
| 2016-05-26 12:45 | Christian Grothoff | Fixed in Version | => 0.0 |
| 2016-05-26 12:45 | Christian Grothoff | Target Version | 0.1 => 0.0 |
| 2025-03-22 13:09 | Christian Grothoff | Category | mechant backend => merchant backend |