Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 234514 - wicd-1.5.1 breaks connecting to hidden essid containing '.'
Summary: wicd-1.5.1 breaks connecting to hidden essid containing '.'
Status: RESOLVED UPSTREAM
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High minor (vote)
Assignee: Jeremy Olexa (darkside) (RETIRED)
URL:
Whiteboard: Upstream; Fixed in next release.
Keywords:
Depends on:
Blocks:
 
Reported: 2008-08-12 00:15 UTC by Stephen Ziemba
Modified: 2008-09-17 23:24 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Stephen Ziemba 2008-08-12 00:15:21 UTC
After installing 1.5.1 I am no longer able to connect to the essid 'PAL2.0'.

This is due to changes within wnettools.py and most specifically line 78 of the 1.5.1 release. The following function is the problem:

def _sanitize_string(string):
# blacklist = [';', ' ', '`']
# new_string = []
# for c in string:
# if c in blacklist:
# new_string.append("\\" + c)
# else:
# new_string.append(c)
# return ''.join(new_string)
    try:
        return re.escape(string)
    except TypeError:
        return None

I am unsure of the full intent of _sanitize_string (it was added between 1.5.0 and 1.5.1 and is called several places). However, re.escape(essid) changes 'PAL2.0' to 'PAL2\.0'. Changing return re.escape(string) to return string fixes this behavior (but loses whatever functionality which was added)

Reproducible: Always

Steps to Reproduce:
1. Connect to a hidden essid containing '.' 

Actual Results:  
No connection is made.  iwconfig essid PAL2\.0 is called instead of iwconfig essid PAL2.0
Comment 1 Jeremy Olexa (darkside) (RETIRED) archtester gentoo-dev Security 2008-08-12 01:06:27 UTC
Talked to upstream and it will be fixed in the next release. thanks for reporting.
Comment 2 Jeremy Olexa (darkside) (RETIRED) archtester gentoo-dev Security 2008-09-14 14:56:40 UTC
Version bump in cvs now, should hit your mirrors in an hour or so. Please test.
Comment 3 Stephen Ziemba 2008-09-17 23:24:16 UTC
(In reply to comment #2)
> Version bump in cvs now, should hit your mirrors in an hour or so. Please test.
> 

1.52 corrects the problem.  Can be marked as fixed.