View Issue Details

IDProjectCategoryView StatusLast Update
0005542Talerobsolete componentpublic2019-12-20 18:12
ReporterChristian Grothoff 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
PriorityurgentSeveritycrashReproducibilityhave not tried
Status closedResolutionfixed 
Platformi7OSDebian GNU/LinuxOS Versionsqueeze
Product Versiongit (master) 
Target Version0.6Fixed in Version0.6 
Summary0005542: bank assertion failure reported on twitter
Descriptionhttps://twitter.com/davidak/status/1086430279911256064
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

Marcello Stanisci

2019-02-05 13:39

reporter   ~0013602

Theoretically, it's all "okay": with numbers too big a assert failure happens, and that's what happened here. OF COURSE, this is not the right way to react to it, and needs to be handled properly.

Christian Grothoff

2019-02-05 14:46

manager   ~0013603

Yes, we should (1) test first and generate a nice(r) error message, and (2) deploy the bank without infinite negative money (is this because 0.5 didn't have the limit implemented?) so that such extreme balances can't easily be produced.

Marcello Stanisci

2019-02-05 15:35

reporter   ~0013604

e5e6d7c1c4852757e29481 hot-fixes (1). It's already deployed, so you can try. As of (2), we deliberately opted for infinite negative balance for the bank (0.5 does have limit implemented), I suspect because we award 100 kudos to new users and it was just easier to do this way.

Of course we can award the 100 kudos to the first N users who register after a new release, and avoid the infinite limit for the bank; but I'm not sure that's worth doing this ("thanks" to those infinite limits we can discover new bugs.)

Marcello Stanisci

2019-02-05 15:42

reporter   ~0013605

Changing to "solved" by the way, as this strictly speaking is.

Issue History

Date ModifiedUsernameFieldChange
2019-02-04 21:29Christian GrothoffNew Issue
2019-02-04 21:29Christian GrothoffStatusnew => assigned
2019-02-04 21:29Christian GrothoffAssigned To => Marcello Stanisci
2019-02-05 13:39Marcello StanisciNote Added: 0013602
2019-02-05 14:46Christian GrothoffNote Added: 0013603
2019-02-05 15:35Marcello StanisciNote Added: 0013604
2019-02-05 15:42Marcello StanisciNote Added: 0013605
2019-02-05 15:43Marcello StanisciStatusassigned => informing
2019-02-05 15:43Marcello StanisciResolutionopen => fixed
2019-02-12 22:18Christian GrothoffFixed in Version => 0.6
2019-12-20 18:12Christian GrothoffStatusinforming => closed
2022-08-23 18:26Christian GrothoffCategorybank (demonstrator) => py bank (demonstrator, obsolete)
2023-12-03 00:23Christian GrothoffCategorypy bank (demonstrator, obsolete) => obsolete componet
2023-12-11 19:08Florian DoldCategoryobsolete componet => obsolete component