rapid7/metasploit-framework — pull request #1871 by zeroSteiner (2013-05-28 18:59:28+00:00)
Title: fix an undefined variable bug in the DTP module

There's been a bug in the DTP module for a while now, not sure when it originated. Basically the smac function fails if netifaces_implemented? evaluates to true because 'interface' is undefined.

```
msf > use auxiliary/spoof/cisco/dtp
msf auxiliary(dtp) > show options

Module options (auxiliary/spoof/cisco/dtp):

Name Current Setting Required Description
---- --------------- -------- -----------
FILTER no The filter string for capturing traffic
INTERFACE no The name of the interface
SMAC no The spoofed mac (if unset, derived from netifaces)
SNAPLEN 65535 yes The number of bytes to capture
TIMEOUT 500 yes The number of seconds to wait for new data

msf auxiliary(dtp) > set INTERFACE em1
INTERFACE => em1
msf auxiliary(dtp) > exploit
[*] Auxiliary module execution completed

[-] Auxiliary failed: NameError undefined local variable or method `interface' for #
[-] Call stack:
[-] /home/user/repos/msf3-git/modules/auxiliary/spoof/cisco/dtp.rb:57:in `smac'
[-] /home/user/repos/msf3-git/modules/auxiliary/spoof/cisco/dtp.rb:62:in `run'
msf auxiliary(dtp) >
```

This pull request fixes the bug and removes my email address.

After the fix:
```
msf > use auxiliary/spoof/cisco/dtp
msf auxiliary(dtp) > show options

Module options (auxiliary/spoof/cisco/dtp):

Name Current Setting Required Description
---- --------------- -------- -----------
FILTER no The filter string for capturing traffic
INTERFACE no The name of the interface
SMAC no The spoofed mac (if unset, derived from netifaces)
SNAPLEN 65535 yes The number of bytes to capture
TIMEOUT 500 yes The number of seconds to wait for new data

msf auxiliary(dtp) > set INTERFACE em1
INTERFACE => em1
msf auxiliary(dtp) > exploit
[*] Auxiliary module execution completed

[*] Starting DTP spoofing service...
msf auxiliary(dtp) >
```