|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
Есть класс, который хранит размер и буфер описывающий в общем случае строку данных. После выделения буфера, сразу прописываю размер буфера, однако 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.
... |
|||
:
Нравится:
Не нравится:
|
|||
24.10.2018, 16:27 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
Ciplusor Код: plaintext 1.
Ты разыменовываешь указатель на char и записываешь в него значение типа int. Это очень опасная дорожка. Попробуй сначала преобразовать его к указателю на int и только потом разыменовывать. ... |
|||
:
Нравится:
Не нравится:
|
|||
24.10.2018, 16:43 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
NekZПопробуй сначала преобразовать его к указателю на int и только потом разыменовывать. Не стоит. Ибо можно нарваться на забавные грабли с выравниванием и оптимизацией. Если раскладка байт в буфере совпадает с платформой, то лучше пользоваться memcpy. Posted via ActualForum NNTP Server 1.5 ... |
|||
:
Нравится:
Не нравится:
|
|||
24.10.2018, 16:50 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
Dimitry SibiryakovНе стоит. Ибо можно нарваться на забавные грабли с выравниванием и оптимизацией. Если раскладка байт в буфере совпадает с платформой, то лучше пользоваться memcpy. Насчёт оптимизации согласен, а вот по поводу выравнивания не понял. Там же нет структуры. Или POD-типов это тоже касается? ... |
|||
:
Нравится:
Не нравится:
|
|||
24.10.2018, 16:53 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
NekZпо поводу выравнивания не понял На высоких уровнях оптимизации компилятор может предполагать, что указатель на целое выровнен правильно и превратить присваивание в один movd или аналогичную команду процессора. Если это предположение неверно и команда не приспособлена работать с невыровненной памятью - будет БУМ! В данном коде такого быть не может, но лучше не привыкать к плохому. И, кстати, я только сейчас обратил внимание, что аффтар-то пишет в буфер всего один байт, а не всё целое. Хотя остаток кода предполагает иначе. И warning компилятора в данном случае это указание на жосткого бага. Posted via ActualForum NNTP Server 1.5 ... |
|||
:
Нравится:
Не нравится:
|
|||
24.10.2018, 17:01 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
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. 30. 31. 32. 33. 34. 35. 36. 37. 38. 39. 40.
... |
|||
:
Нравится:
Не нравится:
|
|||
24.10.2018, 20:45 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
OoCcи зачем делать через одно место когда можно сделать правильно? Достаточно сказать компилятору что ты хочешь.а зачем извращаться с вложенной структурой, если в класс просто нужно добавить поле для длины?)) sizt_t m_Length, например )) ну, и раз уж тут у нас C++, то фтопку malloc, нужно new[] ... |
|||
:
Нравится:
Не нравится:
|
|||
25.10.2018, 01:35 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
OoCcи зачем делать через одно место когда можно сделать правильно? Как вы можете судить о правильности, если не знаете предназначения? Моя строка - подготавливается для дальнейшей передачи через сокеты и в отличии от вашего случая - не выполняет лишних выделений памяти. Но это не суть важно. авторИ, кстати, я только сейчас обратил внимание, что аффтар-то пишет в буфер всего один байт, а не всё целоеХм, проверю, спасибо ... |
|||
:
Нравится:
Не нравится:
|
|||
25.10.2018, 06:52 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
Доработал по комментариям. Вышло обыденно и неинтересно. Но главное что работает и без варнингов, спасибо, не зря решил таки с ним разобраться :) Код: 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.
... |
|||
:
Нравится:
Не нравится:
|
|||
25.10.2018, 07:11 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
egorychа зачем извращаться с вложенной структурой, если в класс просто нужно добавить поле для длины?)) sizt_t m_Length, например )) ну, и раз уж тут у нас C++, то фтопку malloc, нужно new[] egorych ты неправ. ИМХО ТС хотел держать строку сериализованной с длинной. Вложенная структура это не изврещение - это описание сериализованного хранилища. ... |
|||
:
Нравится:
Не нравится:
|
|||
25.10.2018, 09:06 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
[quot Ciplusor] .... Моя строка - подготавливается для дальнейшей передачи через сокеты и в отличии от вашего случая - не выполняет лишних выделений памяти. Но это не суть важно. ... |
|||
:
Нравится:
Не нравится:
|
|||
25.10.2018, 09:30 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
Ciplusor Код: plaintext 1. 2.
В первой строке прописывается значение типа char приведенное из int. Это нулевой байт. 1 штука. Во второй строке что-то копируется после первых 4-х штук байт. Тут как-бы предупреждение должно быть о неявном усечении int в char с возможной потерей информации и как-бы что-то мусорное в первом, втором и третьем байтах с адреса m_Buffer. Если по первому байту программа сможет определить что дальше 3 байта мусор - то как-бы но проблем. А если нет... ... |
|||
:
Нравится:
Не нравится:
|
|||
25.10.2018, 13:58 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
Ciplusor, Вот так надо! Код: plaintext 1.
... |
|||
:
Нравится:
Не нравится:
|
|||
25.10.2018, 17:19 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
Ciplusor, структуры для описания буфера я так понимаю лучше не предлагать? только хардкор? тынц Код: plaintext 1. 2. 3. 4.
... |
|||
:
Нравится:
Не нравится:
|
|||
25.10.2018, 17:47 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
kealon(Ruslan)структуры для описания буфера я так понимаю лучше не предлагать? только хардкор? Структуры это хорошо до тех пор пока не начинается обмен между платформами с разной раскладкой байт. Posted via ActualForum NNTP Server 1.5 ... |
|||
:
Нравится:
Не нравится:
|
|||
25.10.2018, 18:14 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
kealon(Ruslan), В структуре как раз и может случится выравнивание. ... |
|||
:
Нравится:
Не нравится:
|
|||
25.10.2018, 19:02 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
А еще malloc() может вернуть NULL ... |
|||
:
Нравится:
Не нравится:
|
|||
25.10.2018, 21:08 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
Dimitry Sibiryakovkealon(Ruslan)структуры для описания буфера я так понимаю лучше не предлагать? только хардкор? Структуры это хорошо до тех пор пока не начинается обмен между платформами с разной раскладкой байт. там всё равно придётся переворачивать что пришло, и смысл париться с высчитыванием адресов? с выравниванием тоже бороться можно но создавать себе гемор на ровном месте... ... |
|||
:
Нравится:
Не нравится:
|
|||
26.10.2018, 00:33 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
CiplusorВышло обыденно и неинтересно. А какова была цель хранения размера и контента в одном сплошном массиве? ... |
|||
:
Нравится:
Не нравится:
|
|||
26.10.2018, 05:35 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
CEMbCiplusorВышло обыденно и неинтересно. А какова была цель хранения размера и контента в одном сплошном массиве?обычная практика для базовых классов, зачем два раза память выделять ... |
|||
:
Нравится:
Не нравится:
|
|||
26.10.2018, 09:35 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
kealon(Ruslan)обычная практика для базовых классов, зачем два раза память выделятьНууу... а почему два раза-то? Вот: kealon(Ruslan) Код: plaintext 1. 2. 3. 4.
выделяешь один раз для data и тут же пишешь размер в len? ... |
|||
:
Нравится:
Не нравится:
|
|||
26.10.2018, 11:53 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
CEMbвыделяешь один раз для data и тут же пишешь размер в len? А сам str_buffer выделяется автомагически? А в send() его придётся скармливать по частям?.. Posted via ActualForum NNTP Server 1.5 ... |
|||
:
Нравится:
Не нравится:
|
|||
26.10.2018, 12:06 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
CEMb, обычно ещё используют счётчики использования(refcount) что бы строки не дублировать при копировании объекта ... |
|||
:
Нравится:
Не нравится:
|
|||
26.10.2018, 12:54 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
kealon(Ruslan)CEMb, обычно ещё используют счётчики использования(refcount) что бы строки не дублировать при копировании объектаЯ обошелся без ссылок, если идет присваивание - то исходная строка берется указателем, если нет - аллоцируется буффер. Позволяет использовать как с автодеструктором, так и с сохранением значения. Главное условие - снизить минимизацию работы с памятью. Возможно глупое решение, но на текущий момент меня устраивает вполне. Код: plaintext 1.
Код: plaintext 1.
Код не претендует на оригинальность и возможно сплошной костыль, но я только начал изучать конструкторы копирования и перегрузку операторов присваивания :) Код: 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. 30. 31. 32. 33. 34. 35. 36. 37. 38. 39. 40. 41. 42. 43. 44. 45. 46. 47. 48. 49. 50. 51. 52. 53. 54. 55. 56. 57. 58. 59. 60.
... |
|||
:
Нравится:
Не нравится:
|
|||
26.10.2018, 16:59 |
|
Разыменование пустого указателя, который не пустой
|
|||
---|---|---|---|
#18+
Dimitry SibiryakovА сам str_buffer выделяется автомагически? А в send() его придётся скармливать по частям?.. Ну если на стеке, то автоматически, иначе динамически. В send она должна сама себя скармливать, по-хорошему: наружу из класса торчат методы, которые знают, как работать с данными. Внешний код не должен знать и разбираться с внутренними структурами. Иначе, в случае, если вдруг между len и data добавили smth, то весь внешний код поедет. Для меня сомнительна польза запихать всё в текстовый буфер ради send(). В том же случае, с добавлением smth в "структуру", мы получим больше проблем при обращении к "переменным". ... |
|||
:
Нравится:
Не нравится:
|
|||
29.10.2018, 05:15 |
|
|
start [/forum/topic.php?fid=57&fpage=16&tid=2017734]: |
0ms |
get settings: |
9ms |
get forum list: |
12ms |
check forum access: |
3ms |
check topic access: |
3ms |
track hit: |
22ms |
get topic data: |
11ms |
get forum data: |
3ms |
get page messages: |
64ms |
get tp. blocked users: |
2ms |
others: | 249ms |
total: | 378ms |
0 / 0 |