ReSQL.ru
     
powered by simpleCommunicator - 2.0.59     © 2025 Programmizd 02
Мобильная версия    Контакт    Правила    FAQ    Помощь
Целевая тема:
Создать новую тему:
Автор:
Закрыть
Цитировать
Доб. в избранное | Игнор. тему | Прикреп. тему | Пометить прочит. / непрочит. | Фильтр
Форумы / C++ [игнор отключен] [закрыт для гостей] / Разыменование пустого указателя, который не пустой
23 сообщений из 48, страница 2 из 2
  2  все
Разыменование пустого указателя, который не пустой
    #39724323
Фотография MasterZiv
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
MasterZiv
Участник
CiplusorЕсть класс, который хранит размер и буфер описывающий в общем случае строку данных. После выделения буфера, сразу прописываю размер буфера, однако vs2017 выдает warning. Я так полагаю из за того, что конкретно для *m_buffer не выделяется память. Как правильно обойти этот варнинг?

Код: plaintext
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
16.
17.
18.
19.
20.
21.
22.
23.
24.
25.
26.
27.
28.
29.
class string
{
private:
    char * m_Buffer = nullptr;
public:
    string() noexcept
    {
    }
    ~string()
    {
        free( m_Buffer);
    }
    bool Copy(const void * aBuffer, int aLength)
    {
        m_Buffer = (char*)malloc(sizeof(int) + aLength);

        * m_Buffer = aLength; // Разыменование пустого указателя "m_Buffer"

        return memcpy( m_Buffer + sizeof(int), aBuffer, aLength);
    }
    char * String()
    {
        return m_Buffer + sizeof(int);
    }
    int Length()
    {
        return * m_Buffer;
    }
};



Ну, говнокод во всей своей красе.
Даже смотреть противно.

Странно, что только warning и только один...


Вот это вообще как у тебя компилируется?

Код: plaintext
1.
2.
3.
4.
 int Length()
    {
        return * m_Buffer;
    }
...
Рейтинг: 0 / 0
29.10.2018, 10:47
| Ответить | Цитировать | Написать  
Разыменование пустого указателя, который не пустой
    #39724324
Фотография MasterZiv
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
MasterZiv
Участник
CiplusorЕсть класс, который хранит размер и буфер описывающий в общем случае строку данных. После выделения буфера, сразу прописываю размер буфера, однако vs2017 выдает warning. Я так полагаю из за того, что конкретно для *m_buffer не выделяется память. Как правильно обойти этот варнинг?

Да и самое главное -- НАФИГА?
...
Рейтинг: 0 / 0
29.10.2018, 10:48
| Ответить | Цитировать | Написать  
Разыменование пустого указателя, который не пустой
    #39724333
Фотография NekZ
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
NekZ
Участник
MasterZivВот это вообще как у тебя компилируется?

Код: plaintext
1.
2.
3.
4.
 int Length()
    {
        return * m_Buffer;
    }



А что не так? Тебя пробел смущает? Разыменовывается указатель на char, конвертируется в int и привет.
...
Рейтинг: 0 / 0
29.10.2018, 11:02
| Ответить | Цитировать | Написать  
Разыменование пустого указателя, который не пустой
    #39724347
Фотография Anatoly Moskovsky
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Anatoly Moskovsky
Участник
NekZА что не так?
Тут return умножается на m_Buffer
...
Рейтинг: 0 / 0
29.10.2018, 11:16
| Ответить | Цитировать | Написать  
Разыменование пустого указателя, который не пустой
    #39724356
Фотография NekZ
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
NekZ
Участник
Anatoly MoskovskyNekZА что не так?
Тут return умножается на m_Buffer
Спасибо Бьярне за перегруженную семантику * и &. В итоге так и смотрится, вызывая когнитивный диссонанс.
...
Рейтинг: 0 / 0
29.10.2018, 11:24
| Ответить | Цитировать | Написать  
Разыменование пустого указателя, который не пустой
    #39724627
Ciplusor
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Ciplusor
Участник
>> Странно, что только warning и только один...

Ну.. скажем так с нуля писать НЕ говнокод на сях наверное мало кому под силу :) Язык очень верткий, то что делается одним способом оказывается неправильным и делается еще пятью которые правильные. Все в процессе ;)
...
Рейтинг: 0 / 0
29.10.2018, 16:48
| Ответить | Цитировать | Написать  
Разыменование пустого указателя, который не пустой
    #39724635
Ciplusor
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Ciplusor
Участник
А нафига? Ну с одной стороны действительно нафига когда есть готовые Qt, Net библиотеки в которых все разжевано и положено. А с другой - наверное зависимость. Разбирать по винтикам =D
...
Рейтинг: 0 / 0
29.10.2018, 16:53
| Ответить | Цитировать | Написать  
Разыменование пустого указателя, который не пустой
    #39724659
kealon(Ruslan)
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
kealon(Ruslan)
Участник
CiplusorА нафига? Ну с одной стороны действительно нафига когда есть готовые Qt, Net библиотеки в которых все разжевано и положено. А с другой - наверное зависимость. Разбирать по винтикам =Dкаждый тру-программер должен написать свой CString!!!!!!!!
...
Рейтинг: 0 / 0
29.10.2018, 17:35
| Ответить | Цитировать | Написать  
Разыменование пустого указателя, который не пустой
    #39724682
Фотография MasterZiv
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
MasterZiv
Участник
CiplusorА нафига? Ну с одной стороны действительно нафига когда есть готовые Qt, Net библиотеки в которых все разжевано и положено. А с другой - наверное зависимость. Разбирать по винтикам =D


Нафига хранить размер в нескольких последовательных элементах массива char[], а не в отдельном поле?
...
Рейтинг: 0 / 0
29.10.2018, 18:35
| Ответить | Цитировать | Написать  
Разыменование пустого указателя, который не пустой
    #39724686
Dima T
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Dima T
Участник
CiplusorА нафига? Ну с одной стороны действительно нафига когда есть готовые Qt, Net библиотеки в которых все разжевано и положено. А с другой - наверное зависимость. Разбирать по винтикам =D
Тут "изобретен" недо std::string, который без зависимостей.
...
Рейтинг: 0 / 0
29.10.2018, 18:50
| Ответить | Цитировать | Написать  
Разыменование пустого указателя, который не пустой
    #39724690
Dima T
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Dima T
Участник
MasterZiv Нафига хранить размер в нескольких последовательных элементах массива char[], а не в отдельном поле?
Наверно чтобы прога вылетела на таком коде
Код: plaintext
1.
2.
string s;
int l = s.Length();


а потом баг искать долго и упорно.
...
Рейтинг: 0 / 0
29.10.2018, 18:56
| Ответить | Цитировать | Написать  
Разыменование пустого указателя, который не пустой
    #39724709
Ciplusor
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Ciplusor
Участник
Dima TТут "изобретен" недо std::string, который без зависимостей.Зависимость имелась ввиду - разбирать с нуля как устроен мир. Вот вы написали что бред хранить длину строки в одном с ней буфере. Я подумал - действительно, смысла нет. Да, строка пишется в Send() но в составе общего блока команды, поэтому в принципе можно и вынести размер в поле. Как будет окончательный вариант, обязательно выложу :)
...
Рейтинг: 0 / 0
29.10.2018, 20:01
| Ответить | Цитировать | Написать  
Разыменование пустого указателя, который не пустой
    #39724711
kealon(Ruslan)
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
kealon(Ruslan)
Участник
CiplusorВот вы написали что бред хранить длину строки в одном с ней буфере. никто такого не писал
...
Рейтинг: 0 / 0
29.10.2018, 20:11
| Ответить | Цитировать | Написать  
Разыменование пустого указателя, который не пустой
    #39724712
Dima T
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Dima T
Участник
CiplusorДа, строка пишется в Send() но в составе общего блока команды, поэтому в принципе можно и вынести размер в поле.
Ты попытался создать строку заточенную под сериализацию. Это излишне. Просто сделай функцию отправки строки в поток: сначала размер, потом содержимое. А строку оставь std::string.

Если ты думаешь что в TCP быстрее уйдет одни кусок чем два поменьше, то заблуждаешься. В общем случае в итоге они скорее всего уйдут вместе одним IP-пакетом.
...
Рейтинг: 0 / 0
29.10.2018, 20:12
| Ответить | Цитировать | Написать  
Разыменование пустого указателя, который не пустой
    #39724713
Dima T
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Dima T
Участник
kealon(Ruslan)CiplusorВот вы написали что бред хранить длину строки в одном с ней буфере. никто такого не писал
Он буфер выделяет в куче, отсюда грабли 21718321
...
Рейтинг: 0 / 0
29.10.2018, 20:16
| Ответить | Цитировать | Написать  
Разыменование пустого указателя, который не пустой
    #39724714
Dima T
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Dima T
Участник
CiplusorЗависимость имелась ввиду - разбирать с нуля как устроен мир.
Если так, то советую начать с ассемблера, это тот самый ноль. Если потом захочется опуститься ниже нуля, то микросхемотехника.
...
Рейтинг: 0 / 0
29.10.2018, 20:22
| Ответить | Цитировать | Написать  
Разыменование пустого указателя, который не пустой
    #39724718
kealon(Ruslan)
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
kealon(Ruslan)
Участник
Dima Tkealon(Ruslan)пропущено...
никто такого не писал
Он буфер выделяет в куче, отсюда грабли 21718321 он его там вообще не выделяет
...
Рейтинг: 0 / 0
29.10.2018, 20:27
| Ответить | Цитировать | Написать  
Разыменование пустого указателя, который не пустой
    #39724721
Dima T
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Dima T
Участник
kealon(Ruslan)Dima Tпропущено...

Он буфер выделяет в куче, отсюда грабли 21718321 он его там вообще не выделяет
Глубоко не вникал, но есть такие строки в коде
Код: plaintext
1.
2.
3.
    bool Copy(const void * aBuffer, int aLength)    {
                   m_Buffer = (char*)malloc(sizeof(int) + aLength);
...


т.е. выделяет, но руки бы оторвать за такое выделение
...
Рейтинг: 0 / 0
29.10.2018, 20:33
| Ответить | Цитировать | Написать  
Разыменование пустого указателя, который не пустой
    #39724752
Фотография CEMb
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
CEMb
Участник
NekZСпасибо Бьярне за перегруженную семантику * и &. В итоге так и смотрится, вызывая когнитивный диссонанс.
Пользуясь случаем, позанудствую, что указатели и прочие ссылки надо писать слева, потому что они накладываются справа налево.

int*& p - ссылка на указатель на int
CiplusorНу.. скажем так с нуля писать НЕ говнокод на сях наверное мало кому под силу :)я думаю: в жизни надо попробовать всё
Главное понимать, что ты пишешь и почему именно так ты пишешь. Гораздо хуже, когда человек бездумно учит best practics и перестаёт включать мозги. Да, у него "идеальный код", но в нестандартной ситуации, которой в книжках нет, он быстро садится в лужу. Даже хрен с ней, с лужей - идеальный код не всегда оптимален и безопасен.
Лучше один раз пробухаться и осознать гармонию в жизни, чем читать 100 книжек о вреде алкоголя в больших дозах, а потом ещё всех донимать здоровым образом жизни.
Dima Tт.е. выделяет, но руки бы оторвать за такое выделениеиногда надо от души самозабвенно поскакать по граблям, чтобы проникнуться
...
Рейтинг: 0 / 0
30.10.2018, 05:45
| Ответить | Цитировать | Написать  
Разыменование пустого указателя, который не пустой
    #39724772
Ciplusor
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Ciplusor
Участник
Dima Tт.е. выделяет, но руки бы оторвать за такое выделениеЯ правильно понимаю - за то что не используется static_cast? :)
...
Рейтинг: 0 / 0
30.10.2018, 09:02
| Ответить | Цитировать | Написать  
Разыменование пустого указателя, который не пустой
    #39724775
Dima T
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Dima T
Участник
CiplusorDima Tт.е. выделяет, но руки бы оторвать за такое выделениеЯ правильно понимаю - за то что не используется static_cast? :)
За то что m_Buffer никак не проверяется на NULL.
...
Рейтинг: 0 / 0
30.10.2018, 09:09
| Ответить | Цитировать | Написать  
Разыменование пустого указателя, который не пустой
    #39724776
Dima T
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Dima T
Участник
http://www.cplusplus.com/reference/cstdlib/malloc/?kw=malloc malloc()
If the function failed to allocate the requested block of memory, a null pointer is returned.
...
Рейтинг: 0 / 0
30.10.2018, 09:10
| Ответить | Цитировать | Написать  
Разыменование пустого указателя, который не пустой
    #39725432
Фотография CEMb
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
CEMb
Участник
http://www.cplusplus.com/reference/cstdlib/malloc/?kw=malloc If the function failed to allocate the requested block of memory
Причём она может failed не только потому что память кончилась.
...
Рейтинг: 0 / 0
31.10.2018, 05:11
| Ответить | Цитировать | Написать  
23 сообщений из 48, страница 2 из 2
  2  все
Форумы / C++ [игнор отключен] [закрыт для гостей] / Разыменование пустого указателя, который не пустой
Найденые пользователи ...
Разблокировать пользователей ...
Читали тему (0):
Читали форум (0):
Пользователи онлайн (0):
Польз. соглашение
NoSQL.ru       Новости, Чат       Фотоальбом: Участники, Встречи
Закрыть
start [/forum/topic.php?fid=57&msg=39724721&tid=2017734]:
 0ms
get settings:
 10ms
get forum list:
 13ms
check forum access:
 3ms
check topic access:
 3ms
track hit:
 34ms
get topic data:
 9ms
get forum data:
 2ms
get page messages:
 53ms
get tp. blocked users:
 1ms
others: 13ms
total:  141ms
созд. / загр.: 141ms
x
x
Закрыть


Просмотр
0 / 0
Close
Debug Console [Select Text]