- fix monitor mode on orinoco. Although WE > 14 define the standard mode

  "MONITOR", the orinoco monitor patch for 0.13e still does not use that...
- add documentation, a warning, and a runtime check for setChannel(0)

2 files changed, 12 insertions, 2 deletions

diff --git a/libopie2/opienet/onetwork.cpp b/libopie2/opienet/onetwork.cpp
index 45ee4c0..767651e 100644
--- a/libopie2/opienet/onetwork.cpp
+++ b/libopie2/opienet/onetwork.cpp
@@ -594,4 +594,10 @@ int OWirelessNetworkInterface::channel() const
 void OWirelessNetworkInterface::setChannel( int c ) const
 {
+    if ( c )
+    {
+        qWarning( "OWirelessNetworkInterface::setChannel( 0 ) called - fix your application!" );
+        return;
+    }
+
     if ( !_mon )
     {
@@ -940,5 +946,5 @@ void OWlanNGMonitoringInterface::setChannel( int c )
     QString cmd;
     cmd.sprintf( "$(which wlanctl-ng) %s lnxreq_wlansniff channel=%d enable=%s prismheader=%s",
-                 (const char*) _if->name(), c+1, (const char*) enable, (const char*) prism );
+                 (const char*) _if->name(), c, (const char*) enable, (const char*) prism );
     system( cmd );
 }
@@ -1016,6 +1022,8 @@ void OOrinocoMonitoringInterface::setEnabled( bool b )
     // IW_MODE_MONITOR was introduced in Wireless Extensions Version 15
     // Wireless Extensions < Version 15 need iwpriv commandos for monitoring
+    // However, as of recent orinoco drivers, IW_MODE_MONITOR is still not supported
 
-    #if WIRELESS_EXT > 14
+    #if 0
+    //#if WIRELESS_EXT > 14
     if ( b )
         _if->setMode( "monitor" );  // IW_MODE_MONITOR doesn't support prism header
diff --git a/libopie2/opienet/onetwork.h b/libopie2/opienet/onetwork.h
index 2553a61..00d1ebb 100644
--- a/libopie2/opienet/onetwork.h
+++ b/libopie2/opienet/onetwork.h
@@ -337,4 +337,6 @@ class OWirelessNetworkInterface : public ONetworkInterface
      * Setting the @a channel of the interface changes the radio frequency (RF)
      * of the corresponding wireless network device.
+     * @note Common channel range is within [1-14]. A value of 0 is not allowed.
+     * @see channels()
      */
     virtual void setChannel( int channel ) const;