Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 73549 - baselayout-1.11.7-r1/r2 macchanger failure
Summary: baselayout-1.11.7-r1/r2 macchanger failure
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] baselayout (show other bugs)
Hardware: All Linux
: High minor (vote)
Assignee: Gentoo's Team for Core System packages
URL:
Whiteboard:
Keywords: InVCS
Depends on:
Blocks: 66472
  Show dependency tree
 
Reported: 2004-12-06 04:40 UTC by Duncan
Modified: 2004-12-16 02:37 UTC (History)
0 users

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


Attachments
change the case tests to upper (macchanger.patch,978 bytes, patch)
2004-12-06 04:47 UTC, Duncan
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Duncan 2004-12-06 04:40:35 UTC
In /lib/rcscripts/net.modules.d/macchanger, the new code does a tr to uppercase (line 56), then does a
case test on the lowercase versions of the var ($mac) it just uppercased (case is lines 57-78, specifically the tests in
lines 62,65,68,71,74)!

That causes the macchanger command line to be something like "macchanger
RANDOM-FULL eth0", because the case test that would change RANDOM-FULL to -r fails, since it tests for random-full
instead, meaning the case test falls thru to the default which simply passes the var as part of the command line
unchanged.

This of course causes macchanger to fail, which in turn causes net.eth0 to fail, and I end up
without a net connection.

A patch to fix will be attached.

Reproducible: Always
Steps to Reproduce:
Comment 1 Duncan 2004-12-06 04:47:28 UTC
Created attachment 45374 [details, diff]
change the case tests to upper

The other alternative would be changing the tr to lower, and the specified mac
case test to lower as well.
Comment 2 Roy Marples (RETIRED) gentoo-dev 2004-12-06 05:43:43 UTC
I think that reversing the tr and testing a-f is the better way.

Fixed in CVS - will be in baselayout-1.11.8
Comment 3 Roy Marples (RETIRED) gentoo-dev 2004-12-16 02:37:58 UTC
baselayout-1.11.8 is out