View Issue Details

IDProjectCategoryView StatusLast Update
0002815GNUnettransport servicepublic2013-12-24 19:55
ReporterMatthias Wachs Assigned ToChristian Grothoff Due Date
PrioritynormalSeverityminorReproducibilityalways
Status closedResolutionfixed 
Product VersionGit master 
Target Version0.10.0Fixed in Version0.10.0 
Summary0002815: test_plugin_* : ERROR Assertion failed at crypto_ecc.c:160.
DescriptionECC Public key extraction fails for all plugin tests starting with rev 26035... have no clue why.

TagsNo tags attached.

Activities

Christian Grothoff

2013-03-05 09:06

manager   ~0006935

Old hostkey on disk in SVN. Fixed in 26316.

Matthias Wachs

2013-03-05 09:10

reporter   ~0006936

Does it really have to abort with assert? I would prefer to have a clean shutdown...

Christian Grothoff

2013-03-05 09:18

manager   ~0006937

Well, if your local hostkey file is 'corrupt', that's pretty bad. Now, usually this should of course never happen...

Matthias Wachs

2013-03-05 09:42

reporter   ~0006938

Of course this should never happen... but you could:
-print an error
-return NULL
- and give me a chance to clean up

because I would end badly anyway since it's impossible for me to run the test without private key...

Christian Grothoff

2013-03-05 10:00

manager   ~0006939

Well, but that's the kind of error we don't want clients to bother handling. It would make clients way to verbose. And we do print an error -- the assertion ;-).

Matthias Wachs

2013-03-05 10:14

reporter   ~0006940

> And we do print an error -- the assertion ;-).

Yeah... giving you a line number ... so I had to to figure out who is calling this line ... "extract public key" ... because it was not called directly by the test but the ecc code ... didn't help...

I was lucky since I had a core dump but in the end I didn't have a clue what causes an "assertion in ecc_crypto.c:160" ... and just assigned the bug to you :-D

Issue History

Date ModifiedUsernameFieldChange
2013-03-05 09:01Matthias WachsNew Issue
2013-03-05 09:01Matthias WachsStatusnew => assigned
2013-03-05 09:01Matthias WachsAssigned To => Christian Grothoff
2013-03-05 09:06Christian GrothoffNote Added: 0006935
2013-03-05 09:06Christian GrothoffStatusassigned => resolved
2013-03-05 09:06Christian GrothoffFixed in Version => 0.10.0
2013-03-05 09:06Christian GrothoffResolutionopen => fixed
2013-03-05 09:06Christian GrothoffProduct Version => Git master
2013-03-05 09:06Christian GrothoffTarget Version => 0.10.0
2013-03-05 09:10Matthias WachsNote Added: 0006936
2013-03-05 09:18Christian GrothoffNote Added: 0006937
2013-03-05 09:42Matthias WachsNote Added: 0006938
2013-03-05 10:00Christian GrothoffNote Added: 0006939
2013-03-05 10:14Matthias WachsNote Added: 0006940
2013-12-24 19:55Christian GrothoffStatusresolved => closed