malloc и свободная утечка памяти - PullRequest
Новая
       30

malloc и свободная утечка памяти

0 голосов
/ 05 февраля 2019

я пытаюсь создать класс с malloc.

  • класс имеет внутреннюю структуру.

  • у пользователя будет указатель на структуру, но он может не знатьо структуре или даже заботиться о ней.

  • он должен сохранить указатель, а некоторым функциям потребуется адрес этой структуры.

такв заголовке библиотеки я сделал следующее: 

#define EELS_MAX_SLOTS 5

class EELS
{

typedef struct{
   //struct difinition ...
}ee_slot_t;

public:
    EELS();
    uint8_t CreateSlot(uint16_t begin_addr, uint16_t length, uint8_t data_length);
    ~EELS();
protected:
private:
    void* _slot_arr[EELS_MAX_SLOTS];
    uint8_t _slot_counter;
};

и код в исполняемом файле: 

// default constructor
EELS::EELS()
{
    _slot_counter =0;
} //EELS

uint8_t EELS::CreateSlot(uint16_t begin_addr, uint16_t length, uint8_t data_length){
    if (_slot_counter > EELS_MAX_SLOTS)
        return NULL;

    ee_slot_t* slot_p;
    slot_p = malloc(sizeof(ee_slot_t))
    if (!slot_p)
        return NULL;

    slot_p->begining = begin_addr;
    slot_p->length = length;
    slot_p->counter  = 0; // TODO...init...
    slot_p->position = 0; // TODO...init...

    _slot_arr[_slot_counter] = (void*)slot_p;
    _slot_counter++;
    return _slot_counter;
}
// default destructor
EELS::~EELS()
{
    for (int i=0; i<_slot_counter; i++)
    {
        free((ee_slot_t*)_slot_arr[i]);
    }
}

, как вы видите, я возвращаю индекс массива указателей .. так(1-6) в этом случае, и я сохраняю реальный адрес внутри этого массива указателей.

но из того, что вы видите.это безопасно?свободный метод и malloc .. есть какая-то ошибка или утечка памяти?

почему бы не vector?

, потому что он для встроенной системы и текущей IDE / цепочки инструментов, которую я использую, не поддерживает std: векторы.

1 Ответ

0 голосов
/ 05 февраля 2019

Что происходит, когда _slot_counter == EELS_MAX_SLOTS.Поэтому я думаю, что вы должны изменить оператор if 

if (_slot_counter > EELS_MAX_SLOTS)
    return NULL;

на 

if (_slot_counter >= EELS_MAX_SLOTS)
    return 0; // return type is uint8_t, not a pointer
...