Project

General

Profile

Bug #11136

Unclear error message in WhisperBack when failing to connect to the server

Added by alant almost 4 years ago. Updated over 3 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
-
Target version:
Start date:
02/18/2016
Due date:
% Done:

100%

Feature Branch:
Type of work:
Code
Blueprint:
Starter:
Affected tool:
WhisperBack

Description

As reported by Olivier Berger, when Tails is conected to a network but Tor cannot create a circuit to the destination WhisperBack shows an error with the "too many values to unpack" message, which ain't exactly end-user friendly.

$ whisperback 
Exception in thread Thread-6: 
Traceback (most recent call last):
[...]
File "/usr/lib/python3/dist-packages/socks.py", line 459, in
connect dest_addr, dest_port = dest_pair ValueError: too many
values to unpack (expected 2)

socks assumes the host resolves to AF_INET and triggers a ValueError if it's not the case. If a .onion address is given, it resolves to an AF_INET address and to an AF_INET6 address. If the 1st doesn't connect, the 2nd is tried, which make socks trigger the ValueError. This issue is fixed upstream in socks, in which this situation raises a socket.error (https://github.com/Anorov/PySocks/commit/4081b79)

History

#1 Updated by alant almost 4 years ago

  • Subject changed from Uncrear error message in WhisperBack when failing to connect to the server to Unclear error message in WhisperBack when failing to connect to the server

I belive I workaround this bug in WhisperBack master.

#2 Updated by alant almost 4 years ago

  • Description updated (diff)

#3 Updated by alant almost 4 years ago

  • Status changed from Confirmed to 11
  • Assignee changed from alant to anonym
  • QA Check set to Ready for QA

I tested this py patching a live Tails and it works. Please review.

#4 Updated by intrigeri almost 4 years ago

  • Status changed from 11 to In Progress
  • Target version set to Tails_2.2
  • Affected tool set to WhisperBack

#5 Updated by anonym almost 4 years ago

  • Assignee changed from anonym to alant
  • QA Check changed from Ready for QA to Dev Needed

I can see no corresponding commit in WhisperBack's master branch. Did you forget to push? Note that there's still time to get this into Tails 2.2, if you provide the fix before noon 2016-03-07 CET.

#6 Updated by alant almost 4 years ago

  • Assignee changed from alant to anonym
  • QA Check changed from Dev Needed to Ready for QA

Sorry I indeed forgot to push and then disappeared... shoudl be repaired now.

#7 Updated by intrigeri almost 4 years ago

  • Target version changed from Tails_2.2 to Tails_2.3

(Tails 2.2 was released already.)

#8 Updated by anonym over 3 years ago

  • % Done changed from 0 to 20

A package (0.7.5) has been uploaded to the feature-pre-2.3-prep APT suite.

#9 Updated by anonym over 3 years ago

  • Status changed from In Progress to 11
  • Assignee deleted (anonym)
  • % Done changed from 20 to 100
  • QA Check changed from Ready for QA to Pass

Package built and uploaded to stable's and devel's APT suites. This is the package that will end up in Tails 2.3 unless something unexpected happens.

#10 Updated by anonym over 3 years ago

  • Status changed from 11 to Resolved

Also available in: Atom PDF