Приветствую. 
тема вроде избитая, но вот нашел интересный момент.
исходные данные  - 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 можно вообще избавиться.

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

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

CTapMex, если выделять динамически память - то прячьте ее в RAII (std::auto_ptr, boost::shared_ptr и тп) *Мейерс правило 13 из "55 советов"

Так там и &, и * сразу.

artsb, и почему это Вас так пугает? именно это позволяет изменить Name в вызывающей функции и соответственно иметь в вызывающей функции доступ к считанному значению, а в последствии удалить память опять же в рамках вызывающей функции.

17dufa, 
про RAII взял на заметку. сейчас переделывать все не резон. да и надо поизучать

А вы понимаете, что * (указатели) и & (ссылки) это разные вещи? Зачем делать такую ядерную смесь? Удивительно, как оно вообще скомпилилось.

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

Всё. Я вас понял   

да это разные вещи. но если переменная является указателем, то как её передать в функцию по ссылке для изменения? 
скомпилировалось оно нормально. и работает отлично.

другой вопрос разумно ли так делать? а почему нет? не всегда есть возможность поменять функцию, как сделал ты выше. т.е. если надо передать 2 указателя и в них записать что надо

CTapMex, так делать можно и работать оно будет. а можно взять от указателя адрес и в функцию передать двойной указатель:

примерно так будет выглядеть Ваш код, если ссылку заменить двойным указателем.

по мне - разница чисто синтаксическая.

Пользуйтесь. Как сказал 17dufa, всё будет работать. Я просто изначально не совсем понял что вы делали