Приветствую. 
тема вроде избитая, но вот нашел интересный момент.
исходные данные  - VC++ 2008 SP1

код такого содержания


ошибка происходит в последней строчке, при выполнении выше  Name = L"default";
на сколько я понимаю , при компилировании строка помещается в область данных, и при условии len<=1 дается ссылка на эту область.
тут видимо и должна происходить ошибка, ведь память вроде как и не выделялась

но вот есть такой момент
запускаю этот код на WinXP SP3 хоть под отладчиком, хоть уже в релизе - все как часы работает. 
Но вот стоит запустить это на WinXp SP2  (не на всех компах срабатывает) , либо на Win7 (опять же через раз ) то выскакивает ошибка "Runtime Error! ...."
да и когда тестировал в отладчике на win7  ошибка не возникала, соберу релиз - появилась. пересоберу релиз без изменений - работает.

тут соответственно вопрос
мой код все таки некорректен?  или это уже ошибка компилятора/библиотек которые то корректно отрабатывают удаление памяти, то некорректно

Лично я считаю, что должно быть так:

Вы нигде не выделяете память и пытаетесь потом что-то удалить. И если пару раз "прокатило", это не значит, что так можно   

Это сообщение отредактировал(а) artsb - 15.1.2010, 09:45

artsb, 
спасибо за пример, я предполагал по другом это решить. но у тебя решение лучше.

ну пару раз прокатило - но оно прокатывало то интересно как  то - на разных компах по разному. у себя на компе я ни разу как не пытался не мог заставить появится ошибке. 
а уж на win7 через раз. 
у меня стойкое впечатление, что эту ситуацию по разному обрабатывают разные версии ситемных библиотек. или компилятор

Наверное код нужно подправить   

Dem_max, 

в моем случае твой вариант не вариант.
в процедуре rGetValue идет чтение строкового значения из реестра. а оно может быть по размеру любым. по этому выделение памяти  идет по факту.

У меня ни разу не прокатило 

По поводу вашего случая.
ИМХО в этом случае, компилер смотрит что строка константная и пихает её в ресурсы, а после этой операции:

в Name хранится указатель на строку, которую вы не создавали. А потом вы пытаетесь её удалить   
В этом случае, нужно убрать

и должно работать. ИМХО

ЗЫ поправьте если что 

artsb, 
все правильно говоришь. но вот работало у меня.
хотя я уже ни в чем не уверен. вечером еще раз проверю на win7

Значит в процедуре выделяется память ?



Все верно.
Для новичка эта вещь не очевидная, если он ее не поймет то дальше  можеть быть крах программы. И человек просто запарится искать свой косяк.

да, в первом посте это указано

CTapMex, ну вы понимаете, что в функции не может выделиться память для Name? Вы передаёте нулевой указатель и все. Память выделяется под указатель, который является параметром функции. Т.е.


Это сообщение отредактировал(а) artsb - 15.1.2010, 12:18

ну, вы полезли уже глубоко. 
вот сама функция

Тем не менее в тему.



амперсанд здесь не нужен.

И вы нигде не освобождаете Data.

Добавлено @ 12:39
Пробуйте так:


Добавлено @ 12:41
А юзать так:


Это сообщение отредактировал(а) artsb - 15.1.2010, 13:07

 вот про удаление - да, не заметил. спасибо. 


да, так лучше будет . спасибо 
а про бред - твой самый первый вариант решения тут подойдет

Это сообщение отредактировал(а) CTapMex - 15.1.2010, 15:20

Если вы проверяете удалось ли что-то получить после вызова rGetValue, то лучше так:

а от len можно вообще избавиться.

еще раз спасибо