Структура Marshal с элементом массива в C#

Если вы указываете массив в структуре, используемой в качестве параметра out, вам нужно сообщить маршалеру, какой длины будет массив. В вашем коде маршалер, вероятно, выделяет массив нулевой длины или просто использует null, что приводит к сбою. К сожалению, кажется, нет способа указать массив переменной длины out в качестве члена структуры, потому что MarshalAs.SizeParamIndex работает только для методов. Вам может сойти с рук указание большого массива постоянного размера с помощью MarshalAs.SizeConst, но обычно вам придется анализировать буфер возврата (предположительно выделенный вызываемым пользователем) следующим образом:

var count = Marshal.ReadInt32 (uList) ;
var users = new List<USER_LIST_ITEM>  () ;
var ptr   = (long)uList + 4 ;
for (int i = 0 ; i < count ; ++i)
{
    users.Add (Marshal.PtrToStructure (typeof (USER_LIST_ITEM), 
        new IntPtr (ptr))) ;
    ptr += Marshal.SizeOf (typeof (USER_LIST_ITEM)) ;
}

Вам придется уделить особое внимание выравниванию и заполнению, а также проблемам с 32/64-битными версиями.

Это потому, что List еще не выделено.

Вам нужно будет инициализировать все поля.

Еще одна проблема, которую я вижу, заключается в следующем:

IntPtr uList = Marshal.AllocHGlobal(Marshal.SizeOf(userList));
...
result = GetUsers(out uList);

Вы уверены, что out не должно быть ref? В противном случае нет смысла (тоже не уверен, что ref правильно).

Обновление: еще раз взглянув на свой код, вы должны сделать это (и избежать утечки памяти).

IntPtr uList;
var result = GetUsers(out uList);

var userlist = (USER_LIST) Marshal.PtrToStructure(ulist, typeof(USER_LIST));

Marshal.FreeHGlobal(ulist); // pray here or shoot the author of the C function

Обновите снова:

Ваша подпись p/invoke, вероятно, неверна или вы неправильно ее интерпретируете.

Я могу предположить, что это, вероятно, что-то вроде:

int GetUsers(USER_LIST* ulist);

И то, что у вас есть, это не то же самое.

Если это так, решение легко.

Измените USER_LIST на класс (но сохраните последовательную компоновку) и используйте

// pinvoke sig
int GetUsers(USER_LIST ulist);

var ulist = new USER_LIST();
// initialize fields
var r = GetUsers(ulist);

-- or --

Позвоните по ref.

// pinvoke sig
int GetUsers(ref USER_LIST ulist);

var ulist = new USER_LIST();
// initialize fields
var r = GetUsers(ref ulist);

Таким образом, вам не нужно возиться с ручным маршаллингом, и я больше не вижу потенциальных утечек памяти.

Окончательное обновление:

Учитывая опубликованную вами подпись, похоже, что GetUsers возвращает указатель на список USER_LIST с возвращаемым значением, являющимся счетчиком. Хорошая утечка памяти.

В любом случае, я бы, вероятно, поэкспериментировал с небезопасным подходом, просто просмотрел результат и убедился, что все освобождено. (Я все же думаю, что вы должны стрелять в автора).

Я думаю, что ваш исходный код, вероятно, не так уж неверен. Вероятно, вы только что использовали неправильную перегрузку Marshal.PtrToStructure.

Вы пробовали это?

[DllImport("userManager.dll")]
static extern int GetUsers(out IntPtr userList);

[DllImport("userManager.dll")]
static extern void UMFree(IntPtr userList);

static void Main()
{
    IntPtr userList;               // no need to allocate memory in managed code;
    GetUsers(out userList);        // memory is allocated by native function
    USER_LIST u = (USER_LIST)Marshal.PtrToStructure(userList, typeof(USER_LIST));
    UMFree(userList);
}

Использование небезопасного кода:

public unsafe struct USER_LIST
{
    public uint numUsers;
    public USER_LIST_ITEM* list;
}

public unsafe struct USER_LIST_ITEM
{
    public fixed byte name[260];
    public fixed byte address[256];
}

class Program
{
    [DllImport("userManager.dll")]
    static unsafe extern int GetUsers(USER_LIST** userList);

    [DllImport("userManager.dll")]
    static unsafe extern int UMFree(USER_LIST* userList);

    private static unsafe void Main()
    {
        USER_LIST* list;
        GetUsers(&list);
        UMFree(list);
    }
}