SPKI signature fails in encoding of B64

Bug #701869 reported by Samu Varjonen
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
HIPL
Confirmed
Undecided
Christoph Viethen

Bug Description

debug(hipd/cert.c:108@hip_cert_spki_sign): Getting keys for HIT: 2001:001a:7139:10ad:bb56:f1d3:d3d1:83c4
error(hipd/cert.c:201@hip_cert_spki_sign): Failed to encode digest_b64
error(hipd/user.c:384@hip_handle_user_msg): Signing SPKI cert returned an error
debug(hipd/hip_socket.c:145@hip_handle_user_sock): Send response
debug(hipd/hip_socket.c:151@hip_handle_user_sock): Sending message (type=141) response to port 1001

Revision history for this message
Samu Varjonen (sklvarjo) wrote :

trunk 5458

Revision history for this message
Diego Biurrun (diego-biurrun) wrote :

What do you mean exactly by "trunk 5458"? Does the bug exist in that revision? Was it fixed in that revision?

Revision history for this message
Samu Varjonen (sklvarjo) wrote :

Tested with revision 5458.

Revision history for this message
Christoph Viethen (cviethen) wrote :

The bug turned up in revision 5091 (due to wrong processing of EVP_EncodeBlock() return code). Just changing that didn't help - some more breakage had to be removed. RSA version should be fully functional again since rev. 5662.

I'll leave this open for the moment since I suspect similar trouble (of the memory corruption kind) in the DSA portions of the code which still need to be addressed (no pun intended).

Changed in hipl:
status: New → Confirmed
assignee: nobody → Christoph Viethen (cviethen)
Revision history for this message
Miika Komu (miika-iki) wrote :

Are you using valgrind or some other tool to verify these?

Revision history for this message
Christoph Viethen (cviethen) wrote :

gdb was my weapon of choice this time around. Will take a look at valgrind & fixing the rest of this function as time permits (memory bugfixing not being my primary goal these days).

Revision history for this message
Miika Komu (miika-iki) wrote :

Have you checked out efence? Maybe it's better than gdb for the job.

Revision history for this message
Miika Komu (miika-iki) wrote :

apt-get install electric-fence

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.