Opened 15 years ago

#40 closed defect (Fixed)

building surbs backtraces if first node doesn't have an ip address

Reported by: weasel Owned by: nickm
Priority: Low Milestone:
Component: Mixminion-Client Version: unspeficied
Severity: Keywords:
Cc: weasel Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

[Moved from bugzilla]
Reporter: peter@… (Peter Palfrader)

Description:
Opened: 2004-07-28 15:15



Hey,

using 0.0.7.1, when one builds a SURB, and the first node in the path does
not have an IP address, then mixminion backtraces:


weasel@galaxy:~$ mixminion generate-surb -t peter@… -o DaBlocks -P 'psycocat,*2'
Mixminion version 0.0.7.1
This software is for testing purposes only. Anonymity is not guaranteed.
Jul 28 15:14:45.582 +0200 [WARN] Directory /home is writable by group staff (mode 775)
Jul 28 15:14:45.863 +0200 [INFO] Selected path is psycocat,snorky,metacolo
Enter passphrase for keyring:
Traceback (most recent call last):

File "/usr/bin/mixminion", line 9, in ?

mixminion.Main.main(sys.argv)

File "/usr/lib/python2.3/site-packages/mixminion/Main.py", line 333, in main

func(commandStr, args[2:])

File "/usr/lib/python2.3/site-packages/mixminion/ClientMain.py", line 1724, in generateSURB

expiryTime=parser.endAt)

File "/usr/lib/python2.3/site-packages/mixminion/ClientMain.py", line 340, in generateReplyBlock

block = mixminion.BuildMessage.buildReplyBlock(

File "/usr/lib/python2.3/site-packages/mixminion/BuildMessage.py", line 300, in buildReplyBlock

seed)[0]

File "/usr/lib/python2.3/site-packages/mixminion/BuildMessage.py", line 263, in _buildReplyBlockImpl

path[0].getIPV4Info().pack(), sharedKey), secrets, tag

AttributeError: 'NoneType' object has no attribute 'pack'

psycocat only has a Hostname in the directory.



It's even marked XXX007 in the code :)

# XXXX007 switch to Host info. We need to use IPV4 for reply blocks
# XXXX007 for now, since we don't know which servers will support HOST.
# XXXX007 (Do this after all hosts have upgraded to 0.0.6 or later.)


getRoutingInfo() instead of getIPV4Info() will fix this. It looks like all
servers now support delivery by hostname. At least they all have Hostname
set in addition to IP.

[Automatically added by flyspray2trac: Operating System: All]

Child Tickets

Change History (1)

comment:1 Changed 14 years ago by nickm

flyspray2trac: bug closed.
Fixed in 0.0.8alpha1

Note: See TracTickets for help on using tickets.