Почему не получается преобразовать __int128 в BigInteger в своей функции?

Question

[Popou]

я далеко не силен в c и особо не силен в p/invoke, и для практики я решил проверить себя...... подозреваю что нужно использовать malloc(кажется просто возвращается адрес памяти или читает лишние байты), но тогда стоит вопрос об использование free.
код на C:

#include <stdint.h>

__int128  fact(int  n)
{
    if (n == 0) return 1;
    return n * fact(n - 1);
}

__int128* ptr_fact(int n)
{
    __int128 val = fact(n);
    __int128* pfact = &val;
    return pfact;
}

Код на c#:

[DllImport("Test")] static extern IntPtr ptr_fact(int n);

BigInteger ToBigInteger(IntPtr ptr)
{
    Span<byte> span;
    unsafe
    {
        span = new(ptr.ToPointer(), 16);
    }
    return new(span);
}

а вот результат:

Answer 1

[Василий Банников]

1. Ты берёшь указатель на переменную на стеке, которая после выхода из него уже не факт, что будет иметь то значение, которое тебе нужно.
2. Тебе нужно выделить память заранее (или в куче) и в эту область памяти уже записывать результат.
3. Лучше, чтобы память для результата выделялась на стороне C#, чтобы не было проблем с освобождением памяти.
4. Сишный int не всегда равен шарповому int.
5. В .NET уже есть структура для Int128, так что тебе не нужно байты перекладывать.

6.

public BigInteger (ReadOnlySpan<byte> value, bool isUnsigned = false, bool isBigEndian = false);

принимает байты от длинного целого. Так что у тебя твой ToBigInteger работает неправильно, если ты в него подаёшь указатель, на __int128

ptr_fact должно выглядеть как-то так:

void ptr_fact(int32_t n, __int128* result)
{
    *result = fact(n);
}

[DllImport("Test")] static extern void ptr_fact(int n, out Int128 result);

Но смысла использовать ptr_fact я не вижу, тк обычный fact ты тоже легко можешь вызывать

Comments

[Popou]

1,2) тогда придаться еще и с free работать если выделять память вроде бы, и на стороне "c" я должен буду создать отдельную функцию которая будет освобождать ее?
3) Хорошо учту, но не всегда есть такая возможность(
4) И че мне делать тогда(
5)Int128 есть только в Net 7 вроде бы
6)Я подаю не указатель , а Span байтов(16 байтов), и эта часть вроде как работает прекрасно

Смысл использовать ptr_fact нету, это просто попытка ответить на вопрос, а как это сделать?

[Василий Банников]

Popou,
3. Пример? Иначе придётся 1,2
4. Использовать int32_t
5. Никто не запрещает скопипастить код из исходников net7. Там же по сути просто структура нужного размера и всё. Ну и почему бы не обновиться?)
6. Ну если работает, то ладно. Почему-то беспокоился, что BigInteger может не совпадать с i128

[Popou]

Василий Банников, больше спасибо!

Answer 2

[Popou]

просто использовал malloc, но теперь беда с утечкой памяти)
код на c

__int128* ptr_fact(int n)
{
    __int128* pfact = malloc(sizeof(__int128));
    *pfact = fact(n);
    return pfact;
}

Comments

[Popou]

насчет утечки памяти, теоретически это должно решаться вот так, по крайнем мере в blazor Wasm не вызывает ошибку

BigInteger ToBigInteger(IntPtr ptr)
{
    Span<byte> span;
    unsafe
    {
         span = new(ptr.ToPointer(), 16);
    }
    var bigInt = new BigInteger(span);

    Marshal.FreeHGlobal(ptr);

    return bigInt;
}

[freeExec]

Popou, Решаться это должно, как тебе уже говорили, выделением памяти на стороне c#

var ptr = Marshal.AllocHGlobal(16);
exterFactInt128(n, ptr);
...
Marshal.FreeHGlobal(ptr);

[Василий Банников]

Popou, теперь у тебя вместо утечки потенциальный double free и use after free

[Popou]

freeExec, спасибо большое!

[Popou]

Василий Банников, спасибо!

Answer 3

[mayton2019]

Проще на Сишарпе факториал посчитать.

Comments

[Василий Банников]

Это понятно, но всё-таки вопрос про pinvoke, а не про то где лучше факториалы считать