We help IT Professionals succeed at work.

amend code.

Indarnav
Indarnav asked
on
445 Views
Last Modified: 2012-05-06
wrote code for dll that checks status of netwrok adapter and disable it. but it checks for one adapter only. suppose pc has more than one.. kindly suggest what to add so that it checks all available adapter and then disable them.
// HardDisable.cpp : Defines the exported functions for the DLL application.
//
 
#include "stdafx.h"
#include <setupapi.h>
#include <devguid.h>
 
#pragma comment (lib, "Setupapi.lib")
 
 
 extern "C" bool StateChange(DWORD NewState, DWORD SelectedItem,HDEVINFO hDevInfo) 
{ 
    SP_PROPCHANGE_PARAMS PropChangeParams = {sizeof(SP_CLASSINSTALL_HEADER)}; 
    SP_DEVINFO_DATA DeviceInfoData = {sizeof(SP_DEVINFO_DATA)}; 
    // 
    // Get a handle to the Selected Item. 
    // 
    if (!SetupDiEnumDeviceInfo(hDevInfo,SelectedItem,&DeviceInfoData)) 
        return FALSE; 
    // 
    // Set the PropChangeParams structure. 
    // 
    PropChangeParams.ClassInstallHeader.InstallFunction = DIF_PROPERTYCHANGE; 
    PropChangeParams.Scope = DICS_FLAG_GLOBAL; 
    PropChangeParams.StateChange = NewState; 
    if (!SetupDiSetClassInstallParams(hDevInfo, &DeviceInfoData,(SP_CLASSINSTALL_HEADER *)&PropChangeParams, 
        sizeof(PropChangeParams))) 
        return FALSE; 
    // 
    // Call the ClassInstaller and perform the change. 
    // 
    if (!SetupDiCallClassInstaller(DIF_PROPERTYCHANGE, hDevInfo, &DeviceInfoData)) 
        return FALSE; 
    return TRUE; 
} 
 
 extern "C" bool DisableNetAdapter() 
{ 
        HDEVINFO hdi; 
        bool     res; 
    // get a list of all devices of class 'GUID_DEVCLASS_NET' 
		//LPGUID GUID_DEVCLASS_NET = new GUID();
		//GUID_DEVCLASS_NET->Data1 = (unsigned)"4d36e974-e325-11ce-bfc1-08002be10318";
    hdi = SetupDiGetClassDevs(&GUID_DEVCLASS_NET, NULL, NULL, DIGCF_PRESENT); 
        if (hdi == INVALID_HANDLE_VALUE) 
                return FALSE; 
        res = StateChange(DICS_DISABLE, 0, hdi); 
        // release the device info list 
        SetupDiDestroyDeviceInfoList(hdi); 
        return res; 
}

Open in new window

Comment
Watch Question

>>>>    if (!SetupDiEnumDeviceInfo(hDevInfo,SelectedItem,&DeviceInfoData))
>>>>          return FALSE;

You would need to call that in a loop until all devices were returned. E. g. turn the if to a while like

    if (!SetupDiEnumDeviceInfo(hDevInfo,SelectedItem,&DeviceInfoData))
        return FALSE;
    //
    // Set the PropChangeParams structure.
    //
    PropChangeParams.ClassInstallHeader.InstallFunction = DIF_PROPERTYCHANGE;
    PropChangeParams.Scope = DICS_FLAG_GLOBAL;
    PropChangeParams.StateChange = NewState;
    while (SetupDiSetClassInstallParams(hDevInfo, &DeviceInfoData, (SP_CLASSINSTALL_HEADER *)&PropChangeParams,
        sizeof(PropChangeParams)))
    {
     //
     // Call the ClassInstaller and perform the change.
     //
     if (!SetupDiCallClassInstaller(DIF_PROPERTYCHANGE, hDevInfo, &DeviceInfoData))
        return FALSE;
    }

Author

Commented:
i made the changes,,pls confirm..
// HardDisable.cpp : Defines the exported functions for the DLL application.
//
 
#include "stdafx.h"
#include <setupapi.h>
#include <devguid.h>
 
#pragma comment (lib, "Setupapi.lib")
 
 
 extern "C" bool StateChange(DWORD NewState, DWORD SelectedItem,HDEVINFO hDevInfo) 
{ 
    SP_PROPCHANGE_PARAMS PropChangeParams = {sizeof(SP_CLASSINSTALL_HEADER)}; 
    SP_DEVINFO_DATA DeviceInfoData = {sizeof(SP_DEVINFO_DATA)}; 
    // 
    // Get a handle to the Selected Item. 
    // 
    if (!SetupDiEnumDeviceInfo(hDevInfo,SelectedItem,&DeviceInfoData)) 
        return FALSE; 
    // 
    // Set the PropChangeParams structure. 
    // 
    PropChangeParams.ClassInstallHeader.InstallFunction = DIF_PROPERTYCHANGE; 
    PropChangeParams.Scope = DICS_FLAG_GLOBAL; 
    PropChangeParams.StateChange = NewState; 
    while (!SetupDiSetClassInstallParams(hDevInfo, &DeviceInfoData,(SP_CLASSINSTALL_HEADER *)&PropChangeParams, 
        sizeof(PropChangeParams))) 
        //return FALSE; 
	{
    // 
    // Call the ClassInstaller and perform the change. 
    // 
    if (!SetupDiCallClassInstaller(DIF_PROPERTYCHANGE, hDevInfo, &DeviceInfoData)) 
        return FALSE; 
    return TRUE; 
} 
 
 extern "C" bool DisableNetAdapter() 
{ 
        HDEVINFO hdi; 
        bool     res; 
    // get a list of all devices of class 'GUID_DEVCLASS_NET' 
		//LPGUID GUID_DEVCLASS_NET = new GUID();
		//GUID_DEVCLASS_NET->Data1 = (unsigned)"4d36e974-e325-11ce-bfc1-08002be10318";
    hdi = SetupDiGetClassDevs(&GUID_DEVCLASS_NET, NULL, NULL, DIGCF_PRESENT); 
        if (hdi == INVALID_HANDLE_VALUE) 
                return FALSE; 
        res = StateChange(DICS_DISABLE, 0, hdi); 
        // release the device info list 
        SetupDiDestroyDeviceInfoList(hdi); 
        return res; 
} 
 

Open in new window

>>>> while (!SetupDiSetClassInstallParams

No. Remove the ! . The function returns TRUE while it has devices and FALSE at end or error. You could call GetLastError to verify that last is an error like NO_MORE_DATA (check winerror.h for error codes).

Author

Commented:
return true is written below

if (!SetupDiCallClassInstaller(DIF_PROPERTYCHANGE, hDevInfo, &DeviceInfoData))

has to be commented..?? and it will be replaced by getlasterror ??
CERTIFIED EXPERT
Top Expert 2012
Commented:
This one is on us!
(Get your first solution completely free - no credit card required)
UNLOCK SOLUTION
Yes jkr is right.

I mixed up the two if statements in your StateChange function by copy-pasting the wrong if statment. The while loop must be done for the enumeration (and not for the installing).

Author

Commented:
ok, let me try and come back to you

Author

Commented:
u have confused the whole, just confirm the belwo now..

have  i to replace below if statement to while...??only.

if (!SetupDiEnumDeviceInfo(hDevInfo,SelectedItem,&DeviceInfoData))
This one is on us!
(Get your first solution completely free - no credit card required)
UNLOCK SOLUTION

Author

Commented:
ok, it is working fine now. thanks
Unlock the solution to this question.
Join our community and discover your potential

Experts Exchange is the only place where you can interact directly with leading experts in the technology field. Become a member today and access the collective knowledge of thousands of technology experts.

*This site is protected by reCAPTCHA and the Google Privacy Policy and Terms of Service apply.

OR

Please enter a first name

Please enter a last name

8+ characters (letters, numbers, and a symbol)

By clicking, you agree to the Terms of Use and Privacy Policy.