1

Существует такой класс:

class process
{
    HANDLE hndl;
public:
    static HANDLE attach(char*, unsigned long = PROCESS_ALL_ACCESS);
};

В нём есть статический метод attach, при вызове которого добавляется сотня килобайтов к потреблению оперативной памяти, но после отработки освобождается лишь то, что я выделил при помощи new, calloc и malloc, и освободил при помощи delete и free, и почти всё что было добавлено при вызове метода не было освобождено, т.е. уже не используется и при этом висит в памяти. Можно как-нибудь освободить всё, что было выделено методом, включая память на его обслуживание, или нужно написать по-другому?

И ещё. Базу я не знаю, учусь по видеоурокам и своим экспериментам, так что приветствуются любые правки и замечания. Спасибо.

Проблемный код:

HANDLE game::attach(char* process, unsigned long access)
    {
        {

            HANDLE handle   = CreateToolhelp32Snapshot(TH32CS_SNAPPROCESS, 0);
            PROCESSENTRY32* pentry = new PROCESSENTRY32;
            pentry->dwSize = 556;
            if (Process32First(handle, pentry))
            {

                void* ptr = calloc(256, 1);

                if (!CharToOemW(pentry->szExeFile, (char*)ptr))
                    return 0;

                if (!strcmp((char*)ptr, process))
                {

                    free(ptr);
                    goto opening;
                }
                while (Process32Next(handle, pentry))
                {
                    if (!CharToOemW(pentry->szExeFile, (char*)ptr))

                        return 0;
                    if (!strcmp((char*)ptr, process))
                    {
                        free(ptr);
                        goto opening;
                    }
                }
            }

        return 0;

    opening:

            CloseHandle(handle);
            handle = OpenProcess(access, 0, pentry->th32ProcessID);
            delete pentry;
            return handle;
        }
    }
8
  • 1
    CloseHandle msdn.microsoft.com/en-us/library/windows/desktop/…
    – user176262
    1 сен 2017 в 13:59
  • Я попробовал передать 0 и закрыть хендлы до возврата, но это не помогло. Совсем.
    – SPNN 13
    1 сен 2017 в 14:13
  • Память, о которой Вы говорите, под функции не выделяется. Для поиска утечек памяти (и ресурсов в общем) следует использовать программы, называемые профилировщиками. Список для Windows можно посмотреть тут. 1 сен 2017 в 14:13
  • Чтобы повторить проблему постарайтесь привести минимальный воспроизводимый пример в теле вопроса. 1 сен 2017 в 14:14
  • @SPNN13 "попробовал передать 0 и закрыть хендлы до возврата" - это никому ничего не говорит. Мы не видим Вашего кода и, несмотря на наши телепатические способности, не можем угадать где у Вас утекает память.
    – user176262
    1 сен 2017 в 14:22

1 ответ 1

1

я закрывая хендл, записываю новый в ту же переменную. Может из-за этого утечка?

Hет, не из-за этого.

Вы где-то выходите из функции не закрыв handle.

"Помылся - закрой душ!" (с)

Update

В Вашем коде несколько return-ов мимо вызова CloseHandle.

8
  • Чёрт. Точно! Спасибо! Как-то я об этом и не подумал.
    – SPNN 13
    1 сен 2017 в 17:03
  • @SPNN13 Пожалуйста. Вы не могли бы поместить Ваш код в вопрос - "в назидание народам древности"?
    – user176262
    1 сен 2017 в 17:06
  • Совершенно не смешно.
    – SPNN 13
    1 сен 2017 в 17:19
  • @SPNN13 это из Венедикта Ерофеева
    – user176262
    1 сен 2017 в 17:26
  • 1
    @SPNN13 "укусить" - ни в коем случае :). Я могу добавить код в вопрос, если Вы не возражаете, или Вы сами можете это сделать - ссылка "править" под вопросом.
    – user176262
    1 сен 2017 в 17:50

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.