14

I am attempting to enumerate connected portable devices on Windows using the Windows Portable Devices API and the PortableDeviceManager provided by this API.

I have implemented enumeration of device IDs following the MSDN documentation link and various blogs link, but they all result in the same issue - I can only get it to give me the ID of one device when there are several connected.

Here's the snippet of C# code I am using:

PortableDeviceManagerClass deviceManager = new PortableDeviceManagerClass();
deviceManager.RefreshDeviceList();  

uint numberOfDevices = 1;            
deviceManager.GetDevices(null, ref numberOfDevices);

if (numberOfDevices == 0)
{
    return new string[0];
}

string [] deviceIds = new string[numberOfDevices];
deviceManager.GetDevices(ref deviceIds[0], ref numberOfDevices);

return deviceIds;

I have two devices connected to my computer, one Removable USB memory stick and one digital camera. When both are active, only the device ID of my camera will be returned. When I deactivate the camera, the device ID of the removable USB stick is returned.

Is there anyone with experience with this API which can point me in the direction of what I am doing wrong?

1
  • For me, I try use PortableDeviceApiLib (COM) but not compiles neither I have found any full source sample working. I try too WMI with Win32ext_WPD but it was difficult for me. Sep 3, 2015 at 17:38

6 Answers 6

19

Jaran,

Take a look at the following post by the WPD team, it mentions how you can fix the interop assembly.

http://blogs.msdn.com/b/dimeby8/archive/2006/12/05/enumerating-wpd-devices-in-c.aspx

Just to be complete, I'll mention the answer here as well:

This is due to a marshalling restriction. This sample code will only detect one device. You need to manually fix the interop assembly.

  • Disassemble the PortableDeviceApi Interop assembly using the command:

    ildasm Interop.PortableDeviceApiLib.dll /out:pdapi.il

  • Open the IL in Notepad and search for the following string:

    instance void GetDevices([in][out] string& marshal( lpwstr) pPnPDeviceIDs,

  • Replace all instances of the string above with the following string:

    instance void GetDevices([in][out] string[] marshal([]) pPnPDeviceIDs,

  • Save the IL and reassemble the interop using the command:

    ilasm pdapi.il /dll /output=Interop.PortableDeviceApiLib.dll

Rebuild your project. You can now first call GetDevices with a NULL parameter to get the count of devices and then call it again with an array to get the device IDs.

Hope this helps.

9
  • Thank you Christophe! I'm not sure how I missed that paragraph - I've been through the blog post before! I have gotten it to report the correct number of devices now, however I am having problems with the application hanging on the second call to GetDevices().
    – Jaran
    May 29, 2011 at 13:37
  • 6
    To solve the issue with call to GetDevices() hanging, I followed the instructions here: codeproject.com/Messages/3187340/… and replaced with "instance void GetDevices([in][out] string[] marshal(lpwstr[]) pPnPDeviceIDs," instead of "instance void GetDevices([in][out] string[] marshal([]) pPnPDeviceIDs,".
    – Jaran
    May 29, 2011 at 14:04
  • 2
    @Jaran 's comment helped me solve my "Marshaler restriction: Excessively long string." error. I've posted the IL I ended up here: gist.github.com/4327544
    – rikkit
    Dec 18, 2012 at 12:25
  • 1
    What's about WMI Win32ext_WPD? Not well document googleing. Only reference: squadratechnologies.wordpress.com/2013/07/24/…
    – Kiquenet
    Aug 26, 2014 at 8:40
  • 1
    This might be helpful for further understanding: msdn.microsoft.com/en-us/library/ek1fb3c6(v=vs.100).aspx It describes common changes in signatures when marshalling. Also: social.msdn.microsoft.com/Forums/vstudio/en-US/…
    – bash.d
    Dec 30, 2016 at 7:35
4

Just an update on the accepted answer.

The correct replacement is as follows.

GetDevices([in][out] string& marshal( lpwstr) pPnPDeviceIDs,

to

GetDevices([in][out] string[] marshal( lpwstr[]) pPnPDeviceIDs,

As per Andrew Trevarrow.

1
  • 1
    Just because @bash.d's comment on the accepted answer gets buried for me, make sure to have a look at Marshalling Changes for other similar replacements.
    – Bondolin
    Sep 7, 2021 at 15:44
1

The one line of Power-shell script below un-mounts a USB cable attached Windows Portable Device (WPD) from the Windows Operating System (XP thru W8/2012)

And just in case you have not yet started playing with Powershell, here is the equivalent VBScript (maybe be can port to C#):

Set objWMIService = GetObject ("winmgmts:\\.\root\cimv2")
Set colItems = objWMIService.ExecQuery ("Select * from Win32ext_WPD Where strFriendlyName = 'SAMSUNG-SGH-I747'")
For Each objItem in colItems
Set objWMIWPDStatic = objWMIService.Get("Win32ext_WPD")
Set objInParam = objWMIWPDStatic.Methods_("EjectDevice").inParameters.SpawnInstance_()
objInParam.Properties_.Item("strObjectDeviceId") =  objItem.strId
Set objOutParams = objWMIService.ExecMethod("Win32ext_WPD", "EjectDevice", objInParam)
Exit For
Next

Note change ‘SAMSUNG-SGH-I747′ to the phone/tablet name you see in Windows Explorer

About Win32ext_WPD

"Select * from Win32ext_WPD Where strFriendlyName = 'SAMSUNG-SGH-I747'"

Not well document googleing, not found more 2 references.

Maybe port to C# using:

var oScope = new ManagementScope(@"\\" + MachineName + @"\root\cimv2");

Reference:

http://squadratechnologies.wordpress.com/2013/07/24/windows-powershellvbscript-to-un-mount-a-smart-phone-or-tablet/

0

WPD-.NET-Wrapper on github - Can enumerate devices and copy files successfully. so that you don't have to waste time coding. you can just open, connect and copy...

https://github.com/kumushoq/WPD-.NET-Wrapper/blob/2d502d883b981b8bc57d32389e8280877632f6de/WindowsPortableDeviceNet/Utility.cs

1
  • For me, not compiles, PortableDeviceApiLib; is a COM ? what I need install in my Windows 7 x64 ? Sep 3, 2015 at 17:34
0

I am late to the party, but this is what worked for me:

  1. Download this NuGet package: PortableDevices

  2. Add references to these 4 COM libraries:

    • PortableDeviceClassExtension
    • PortableDeviceConnectApi
    • PortableDeviceTypes
    • PortableDeviceApi
  3. Take the dll's under obj\Debug and put them into bin\Debug:

    • Interop.PortableDeviceClassExtension.dll
    • Interop.PortableDeviceConnectApiLib.dll
    • Interop.PortableDeviceTypesLib.dll
    • Interop.PortableDeviceApiLib.dll

Now you can use the following function to list all devices, although FriendlyName does not seem to be working (it returns an empty string):

    private IDictionary<string, string> GetDeviceIds()
    {
        var deviceIds = new Dictionary<string, string>();
        var devices = new PortableDeviceCollection();
        devices.Refresh();
        foreach (var device in devices)
        {
            device.Connect();
            deviceIds.Add(device.FriendlyName, device.DeviceId);
            Console.WriteLine(@"DeviceId: {0}, FriendlyName: {1}", device.DeviceId, device.FriendlyName);
            device.Disconnect();
        }
        return deviceIds;
    }

The next step is getting the contents from the device, which is done like so:

var contents = device.GetContents();
-1

Maybe I'm wrong, but the line deviceManager.GetDevices(ref deviceIds[0], ref numberOfDevices); does not makes sense to me. If you want fill the array of string, you should pass whole array, not only one string.

I found a post on blgs.msdn.com where is passed the whole array.

string[] deviceIDs = new string[cDevices];
devMgr.GetDevices(deviceIDs, ref cDevices);

However it's only guess. I hope, it will help you.

EDIT: I found several code samples, where is passed whole array instead of first item: For example: http://social.msdn.microsoft.com/forums/en-US/vbgeneral/thread/22288487-caf3-4da5-855f-6447ad9fa48d

And I found the Importing PortableDeviceApiLib generates incorrect Interop in VB.NET issue - it looks like the default interop assembly is not correct. Maybe the successfull way is getting correct interop assembly from someone and use it. Or use some another bridge between managed code (C#/VB.NET/...) and native code. The C++\CLI could be good way if you are good enough in C++.

I found Portable Device Lib project on CodePlex - maybe it is the correct bridge.

4
  • Normally I would agree with you, but the API does not offer a method like that. It requires you to pass a string pointer. My understanding was that you pass a pointer to the head of the array and the GetDevices method will populate the array with cDevices number of device IDs, starting from that address.
    – Jaran
    May 28, 2011 at 14:10
  • @Jaran: The problem is, the "string" is reference type in .NET, so if you pass reference to first string from array, there is no relation to the second string in array. Image - one string (one reference) can be in 0-N arrays of string. So, which string (reference) should be the next from the "first"?
    – TcKs
    May 28, 2011 at 16:25
  • Why do they pass pointers? They just could return the result. I'm just curious.
    – OneWorld
    Jan 25, 2013 at 12:43
  • In general, the string value can be large. So passing only the reference is much more cheaper. In this case, you must ask the authors of device manager API.
    – TcKs
    Jan 26, 2013 at 15:59

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Not the answer you're looking for? Browse other questions tagged or ask your own question.