Телеграмм чат группы supapro страница 33347

Size: a a a

supapro.cxx

2020 August 11

Anes

Всем привет. Почему в этом коде

   MPtArray* newarr = new MPtArray();
      newarr = LOMTOISPL(ptarr, spline->m_vid, spline->m_kolpts);
      m_nPoint = newarr->GetSize();
      pDevice->Polyline(newarr->GetHead(),m_nPoint);
      delete newarr;

анализтор кода находит уязвимость некоректного освобождения ресурсов CW-404? По идее использована соответствующая пара операторов new/delete

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

источник

13:09пожаловаться #1

Anton Kviatkovskii in supapro.cxx

Artöm Bakri Al-Sarmini

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

источник

13:09пожаловаться #2

Ilia Zviagin in supapro.cxx

Anton Kviatkovskii

delete [] newarr;

Антоха, где твоя Ава?

источник

13:09пожаловаться #3

🎄T

🎄🎊 R 🎅 Tb| ✡️ 🎊🎄... in supapro.cxx

Anes

Всем привет. Почему в этом коде

   MPtArray* newarr = new MPtArray();
      newarr = LOMTOISPL(ptarr, spline->m_vid, spline->m_kolpts);
      m_nPoint = newarr->GetSize();
      pDevice->Polyline(newarr->GetHead(),m_nPoint);
      delete newarr;

newarr = LOMTOISPL(ptarr, spline->m_vid, spline->m_kolpts);

не перезаписывает указатель?

источник

13:09пожаловаться #4

Artöm Bakri Al-Sarmi... in supapro.cxx

Перезаписывает

источник

13:10пожаловаться #5

Artöm Bakri Al-Sarmi... in supapro.cxx

Старую память освободить должен ты

источник

13:10пожаловаться #6

Anton Kviatkovskii in supapro.cxx

Ilia Zviagin

Антоха, где твоя Ава?

Вжух и появилась

источник

13:10пожаловаться #7

Ilia Zviagin in supapro.cxx

Anes

Всем привет. Почему в этом коде

   MPtArray* newarr = new MPtArray();
      newarr = LOMTOISPL(ptarr, spline->m_vid, spline->m_kolpts);
      m_nPoint = newarr->GetSize();
      pDevice->Polyline(newarr->GetHead(),m_nPoint);
      delete newarr;

Выделил память и тут же перезаписал указатель чем-то другим

источник

13:10пожаловаться #8

Anes in supapro.cxx

MPtArray* newarr = LOMTOISPL(ptarr, spline->m_vid, spline->m_kolpts);
m_nPoint = newarr->GetSize();
pDevice->Polyline(newarr->GetHead(),m_nPoint);
delete newarr;

источник

13:10пожаловаться #9

Artöm Bakri Al-Sarmi... in supapro.cxx

Сэр MSTR

удален он точно не будет, он может быть не инициализирован

Нельзя читать неинициализированные переменные

источник

13:10пожаловаться #10

Anes in supapro.cxx

Anes

MPtArray* newarr = LOMTOISPL(ptarr, spline->m_vid, spline->m_kolpts);
m_nPoint = newarr->GetSize();
pDevice->Polyline(newarr->GetHead(),m_nPoint);
delete newarr;

Типо так лучше сделать?

источник

13:10пожаловаться #11

Artöm Bakri Al-Sarmi... in supapro.cxx

Да

источник

13:10пожаловаться #12

Anes in supapro.cxx

ок спасибо

источник

13:11пожаловаться #13

Anes in supapro.cxx